commit 66e5c33dd2cbd5fcb26ef6cc60f483ee82bfa9d2
Author: sin <[email protected]>
Date:   Fri Feb 27 21:43:11 2015 +0000

    time(1) cleanup

diff --git a/time.c b/time.c
index 71bf04d..a628021 100644
--- a/time.c
+++ b/time.c
@@ -1,11 +1,14 @@
-#include <stdio.h>
-#include <unistd.h>
+/* See LICENSE file for copyright and license details. */
 #include <sys/times.h>
 #include <sys/wait.h>
 
+#include <errno.h>
+#include <stdio.h>
+#include <unistd.h>
+
 #include "util.h"
 
-void
+static void
 usage(void)
 {
        eprintf("usage: %s [-p] utility [argument ...]\n", argv0);
@@ -35,19 +38,12 @@ main(int argc, char *argv[])
        if ((ticks = sysconf(_SC_CLK_TCK)) < 0)
                eprintf("sysconf() failed to retrieve clock ticks per second:");
 
-       if ((rbeg = times(&tms)) < 0) /* POSIX doesn't say NULL is ok... */
+       if ((rbeg = times(&tms)) < 0)
                eprintf("times() failed to retrieve start times:");
 
        if (!(pid = fork())) { /* child */
                execvp(*argv, argv);
-               /* FIXME: check errno for POSIX exit status
-                * 126: found could not be invoked
-                * 127: could not be found
-                * problem is some like EACCESS can mean either...
-                * worth doing manual path search for correct value? also gives 
more
-                * accurate time as the patch search wouldn't be included in 
child's
-                * user/sys times... */
-               enprintf(127, "failed to exec %s:", *argv);
+               enprintf(errno == ENOENT ? 127 : 126, "failed to exec %s:", 
*argv);
        }
        waitpid(pid, &status, 0);
 

Reply via email to