]> ruderich.org/simon Gitweb - coloredstderr/coloredstderr.git/commitdiff
Remove unnecessary calls to update_environment() in exec*().
authorSimon Ruderich <simon@ruderich.org>
Thu, 13 Jun 2013 19:50:51 +0000 (21:50 +0200)
committerSimon Ruderich <simon@ruderich.org>
Thu, 13 Jun 2013 19:50:51 +0000 (21:50 +0200)
src/coloredstderr.c

index 7c4a5f5010cc42163ca10e8de75a34cb1ea32da1..0d76b5aee7dd0799c40e1ccf82b313e3f4002221 100644 (file)
@@ -531,23 +531,24 @@ int execve(char const *filename, char * const argv[], char * const env[]) {
 int execl(char const *path, char const *arg, ...) {
     EXECL_COPY_VARARGS(args);
 
-    update_environment();
+    /* execv() updates the environment. */
     return execv(path, args);
 }
-
 int execlp(char const *file, char const *arg, ...) {
     EXECL_COPY_VARARGS(args);
 
-    update_environment();
+    /* execvp() updates the environment. */
     return execvp(file, args);
 }
-
 int execle(char const *path, char const *arg, ... /*, char * const envp[] */) {
+    char * const *envp;
+
     EXECL_COPY_VARARGS_START(args);
     /* Get envp[] located after arguments. */
-    char * const *envp = va_arg(ap, char * const *);
+    envp = va_arg(ap, char * const *);
     EXECL_COPY_VARARGS_END(args);
 
+    /* execve() updates the environment. */
     return execve(path, args, envp);
 }
 
@@ -573,6 +574,7 @@ int execvpe(char const *file, char * const argv[], char * const envp[]) {
     /* Fake the environment so we can reuse execvp(). */
     environ = (char **)envp;
 
+    /* execvp() updates the environment. */
     return execvp(file, argv);
 }
 #endif