use const qualifiers where possible
[paraslash.git] / exec.c
diff --git a/exec.c b/exec.c
index 34643b8..1a302c1 100644 (file)
--- a/exec.c
+++ b/exec.c
  * \param args the argument array for the command
  * \param fds a pointer to a value-result array
  *
- * This function uses fork/exec to create a new process.  \a fds must be a
- * pointer to three integers, corresponding to stdin, stdout and stderr
- * respectively. It specifies how to deal with fd 0, 1, 2 in the child.  The
- * contents of \a fds are interpreted as follows:
- *
- *     - fd[i] < 0: leave fd \a i alone
- *     - fd[i] = 0: dup fd \a i to /dev/null
- *     - fd[i] > 0: create a pipe and dup i to one end of that pipe.
- *     Upon return, fd[i] contains the file descriptor of the pipe.
- *
- *     In any case, all unneeded filedescriptors are closed.
- *
  * \return Negative on errors, positive on success.
  *
  * \sa null(4), pipe(2), dup2(2), fork(2), exec(3)
  */
-int para_exec(pid_t *pid, const char *file, char *const args[], int *fds)
+static int para_exec(pid_t *pid, const char *file, char *const args[], int *fds)
 {
        int ret, in[2] = {-1, -1}, out[2] = {-1, -1}, err[2] = {-1, -1},
                null = -1; /* ;) */
@@ -134,16 +122,25 @@ err_out:
 /**
  * exec the given command
  *
- * \param pid the same meaning as in para_exec()
+ * \param pid will hold the pid of the created process upon return
  * \param cmdline holds the command and its arguments, seperated by spaces
- * \param fds the same meaning as in para_exec()
+ * \param fds a pointer to a value-result array
+ *
+ * This function uses fork/exec to create a new process. \a fds must be a
+ * pointer to three integers, corresponding to stdin, stdout and stderr
+ * respectively. It specifies how to deal with fd 0, 1, 2 in the child. The
+ * contents of \a fds are interpreted as follows:
+ *
+ *     - fd[i] < 0: leave fd \a i alone
+ *     - fd[i] = 0: dup fd \a i to /dev/null
+ *     - fd[i] > 0: create a pipe and dup i to one end of that pipe.
+ *     Upon return, fd[i] contains the file descriptor of the pipe.
  *
- * A wrapper for para_exec() which calls split_args() to seperate
- * the command line arguments.
+ *     In any case, all unneeded filedescriptors are closed.
  *
  * \return positive on success, negative on errors
  */
-int para_exec_cmdline_pid(pid_t *pid, char *cmdline, int *fds)
+int para_exec_cmdline_pid(pid_t *pid, const char *cmdline, int *fds)
 {
        int argc, ret;
        char **argv, *tmp = para_strdup(cmdline);