Changeset 9f5cf68 in mainline

Timestamp:

2012-08-16T18:51:52Z (12 years ago)

Author:

Martin Decky <martin@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

b6a088f

Parents:

dbf2ca4

Message:

implement va_list variant of task_spawn()

Location:

uspace/lib/c

Files:

• generic/task.c (modified) (3 diffs)
• include/task.h (modified) (2 diffs)

uspace/lib/c/generic/task.c

@@ -201 +201 @@
  *
  * This is really just a convenience wrapper over the more complicated
+ * loader API. Arguments are passed in a va_list.
+ *
+ * @param id   If not NULL, the ID of the task is stored here on success.
+ * @param path Pathname of the binary to execute.
+ * @param cnt  Number of arguments.
+ * @param ap   Command-line arguments.
+ *
+ * @return Zero on success or negative error code.
+ *
+ */
+int task_spawn(task_id_t *task_id, const char *path, int cnt, va_list ap)
+{
+        /* Allocate argument list. */
+        const char **arglist = malloc(cnt * sizeof(const char *));
+        if (arglist == NULL)
+                return ENOMEM;
+        
+        /* Fill in arguments. */
+        const char *arg;
+        cnt = 0;
+        do {
+                arg = va_arg(ap, const char *);
+                arglist[cnt++] = arg;
+        } while (arg != NULL);
+        
+        /* Spawn task. */
+        int rc = task_spawnv(task_id, path, arglist);
+        
+        /* Free argument list. */
+        free(arglist);
+        return rc;
+}
+
+/** Create a new task by running an executable from the filesystem.
+ *
+ * This is really just a convenience wrapper over the more complicated
  * loader API. Arguments are passed as a null-terminated list of arguments.
  *
@@ -216 +252 @@
         va_list ap;
         const char *arg;
-        const char **arglist;
         int cnt = 0;
         
@@ -226 +261 @@
         va_end(ap);
         
-        /* Allocate argument list. */
-        arglist = malloc(cnt * sizeof(const char *));
-        if (arglist == NULL)
-                return ENOMEM;
-        
-        /* Fill in arguments. */
-        cnt = 0;
         va_start(ap, path);
-        do {
-                arg = va_arg(ap, const char *);
-                arglist[cnt++] = arg;
-        } while (arg != NULL);
+        int rc = task_spawn(task_id, path, cnt, ap);
         va_end(ap);
         
-        /* Spawn task. */
-        int rc = task_spawnv(task_id, path, arglist);
-        
-        /* Free argument list. */
-        free(arglist);
         return rc;
 }

uspace/lib/c/include/task.h

@@ -38 +38 @@
 #include <sys/types.h>
 #include <abi/proc/task.h>
+#include <stdarg.h>
 
 typedef enum {
@@ -48 +49 @@
 extern int task_kill(task_id_t);
 
-extern task_id_t task_spawn(const char *, const char *const[], int *);
 extern int task_spawnv(task_id_t *, const char *path, const char *const []);
 extern int task_spawnvf(task_id_t *, const char *path, const char *const [],
     int *const []);
+extern int task_spawn(task_id_t *, const char *path, int, va_list ap);
 extern int task_spawnl(task_id_t *, const char *path, ...);