Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/usr.bin/make make(1): clean up Cmd_Exec stylistically



details:   https://anonhg.NetBSD.org/src/rev/864d79d97696
branches:  trunk
changeset: 956734:864d79d97696
user:      rillig <rillig%NetBSD.org@localhost>
date:      Fri Nov 06 23:04:20 2020 +0000

description:
make(1): clean up Cmd_Exec stylistically

diffstat:

 usr.bin/make/main.c |  33 ++++++++++-----------------------
 1 files changed, 10 insertions(+), 23 deletions(-)

diffs (74 lines):

diff -r b5dbde5f3db3 -r 864d79d97696 usr.bin/make/main.c
--- a/usr.bin/make/main.c       Fri Nov 06 22:39:10 2020 +0000
+++ b/usr.bin/make/main.c       Fri Nov 06 23:04:20 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: main.c,v 1.429 2020/11/06 21:20:31 rillig Exp $        */
+/*     $NetBSD: main.c,v 1.430 2020/11/06 23:04:20 rillig Exp $        */
 
 /*
  * Copyright (c) 1988, 1989, 1990, 1993
@@ -118,7 +118,7 @@
 #include "trace.h"
 
 /*     "@(#)main.c     8.3 (Berkeley) 3/19/94" */
-MAKE_RCSID("$NetBSD: main.c,v 1.429 2020/11/06 21:20:31 rillig Exp $");
+MAKE_RCSID("$NetBSD: main.c,v 1.430 2020/11/06 23:04:20 rillig Exp $");
 #if defined(MAKE_NATIVE) && !defined(lint)
 __COPYRIGHT("@(#) Copyright (c) 1988, 1989, 1990, 1993 "
            "The Regents of the University of California.  "
@@ -1721,10 +1721,7 @@
      */
     switch (cpid = vFork()) {
     case 0:
-       /*
-        * Close input side of pipe
-        */
-       (void)close(fds[0]);
+       (void)close(fds[0]);    /* Close input side of pipe */
 
        /*
         * Duplicate the output stream to the shell's output, then
@@ -1745,36 +1742,26 @@
        goto bad;
 
     default:
-       /*
-        * No need for the writing half
-        */
-       (void)close(fds[1]);
+       (void)close(fds[1]);    /* No need for the writing half */
 
        savederr = 0;
        Buf_Init(&buf, 0);
 
        do {
-           char   result[BUFSIZ];
+           char result[BUFSIZ];
            bytes_read = read(fds[0], result, sizeof result);
            if (bytes_read > 0)
                Buf_AddBytes(&buf, result, (size_t)bytes_read);
-       }
-       while (bytes_read > 0 || (bytes_read == -1 && errno == EINTR));
+       } while (bytes_read > 0 || (bytes_read == -1 && errno == EINTR));
        if (bytes_read == -1)
            savederr = errno;
 
-       /*
-        * Close the input side of the pipe.
-        */
-       (void)close(fds[0]);
+       (void)close(fds[0]);    /* Close the input side of the pipe. */
 
-       /*
-        * Wait for the process to exit.
-        */
-       while(((pid = waitpid(cpid, &status, 0)) != cpid) && (pid >= 0)) {
+       /* Wait for the process to exit. */
+       while((pid = waitpid(cpid, &status, 0)) != cpid && pid >= 0)
            JobReapChild(pid, status, FALSE);
-           continue;
-       }
+
        res_len = Buf_Len(&buf);
        res = Buf_Destroy(&buf, FALSE);
 



Home | Main Index | Thread Index | Old Index