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): indent compat.c with tabs instead of s...



details:   https://anonhg.NetBSD.org/src/rev/a465556d077c
branches:  trunk
changeset: 1016557:a465556d077c
user:      rillig <rillig%NetBSD.org@localhost>
date:      Mon Nov 23 19:35:10 2020 +0000

description:
make(1): indent compat.c with tabs instead of spaces

diffstat:

 usr.bin/make/compat.c |  663 +++++++++++++++++++++++++------------------------
 1 files changed, 337 insertions(+), 326 deletions(-)

diffs (truncated from 785 to 300 lines):

diff -r 88aa7e8cc4e8 -r a465556d077c usr.bin/make/compat.c
--- a/usr.bin/make/compat.c     Mon Nov 23 19:27:20 2020 +0000
+++ b/usr.bin/make/compat.c     Mon Nov 23 19:35:10 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: compat.c,v 1.185 2020/11/23 19:27:20 rillig Exp $      */
+/*     $NetBSD: compat.c,v 1.186 2020/11/23 19:35:10 rillig Exp $      */
 
 /*
  * Copyright (c) 1988, 1989, 1990 The Regents of the University of California.
@@ -96,7 +96,7 @@
 #include "pathnames.h"
 
 /*     "@(#)compat.c   8.2 (Berkeley) 3/19/94" */
-MAKE_RCSID("$NetBSD: compat.c,v 1.185 2020/11/23 19:27:20 rillig Exp $");
+MAKE_RCSID("$NetBSD: compat.c,v 1.186 2020/11/23 19:35:10 rillig Exp $");
 
 static GNode *curTarg = NULL;
 static pid_t compatChild;
@@ -109,13 +109,13 @@
 static void
 CompatDeleteTarget(GNode *gn)
 {
-    if (gn != NULL && !Targ_Precious(gn)) {
-       const char *file = GNode_VarTarget(gn);
+       if (gn != NULL && !Targ_Precious(gn)) {
+               const char *file = GNode_VarTarget(gn);
 
-       if (!opts.noExecute && eunlink(file) != -1) {
-           Error("*** %s removed", file);
+               if (!opts.noExecute && eunlink(file) != -1) {
+                       Error("*** %s removed", file);
+               }
        }
-    }
 }
 
 /* Interrupt the creation of the current target and remove it if it ain't
@@ -129,34 +129,34 @@
 static void
 CompatInterrupt(int signo)
 {
-    CompatDeleteTarget(curTarg);
+       CompatDeleteTarget(curTarg);
 
-    if (curTarg != NULL && !Targ_Precious(curTarg)) {
-       /*
-        * Run .INTERRUPT only if hit with interrupt signal
-        */
-       if (signo == SIGINT) {
-           GNode *gn = Targ_FindNode(".INTERRUPT");
-           if (gn != NULL) {
-               Compat_Make(gn, gn);
-           }
+       if (curTarg != NULL && !Targ_Precious(curTarg)) {
+               /*
+                * Run .INTERRUPT only if hit with interrupt signal
+                */
+               if (signo == SIGINT) {
+                       GNode *gn = Targ_FindNode(".INTERRUPT");
+                       if (gn != NULL) {
+                               Compat_Make(gn, gn);
+                       }
+               }
        }
-    }
 
-    if (signo == SIGQUIT)
-       _exit(signo);
+       if (signo == SIGQUIT)
+               _exit(signo);
 
-    /*
-     * If there is a child running, pass the signal on.
-     * We will exist after it has exited.
-     */
-    compatSigno = signo;
-    if (compatChild > 0) {
-       KILLPG(compatChild, signo);
-    } else {
-       bmake_signal(signo, SIG_DFL);
-       kill(myPid, signo);
-    }
+       /*
+        * If there is a child running, pass the signal on.
+        * We will exist after it has exited.
+        */
+       compatSigno = signo;
+       if (compatChild > 0) {
+               KILLPG(compatChild, signo);
+       } else {
+               bmake_signal(signo, SIG_DFL);
+               kill(myPid, signo);
+       }
 }
 
 static void
@@ -193,272 +193,278 @@
 int
 Compat_RunCommand(const char *cmdp, GNode *gn)
 {
-    char *cmdStart;            /* Start of expanded command */
-    char *bp;
-    Boolean silent;            /* Don't print command */
-    Boolean doIt;              /* Execute even if -n */
-    volatile Boolean errCheck; /* Check errors */
-    int reason;                        /* Reason for child's death */
-    int status;                        /* Description of child's death */
-    pid_t cpid;                        /* Child actually found */
-    pid_t retstat;             /* Result of wait */
-    StringListNode *cmdNode;   /* Node where current command is located */
-    const char **volatile av;  /* Argument vector for thing to exec */
-    char **volatile mav;       /* Copy of the argument vector for freeing */
-    Boolean useShell;          /* TRUE if command should be executed
+       char *cmdStart;         /* Start of expanded command */
+       char *bp;
+       Boolean silent;         /* Don't print command */
+       Boolean doIt;           /* Execute even if -n */
+       volatile Boolean errCheck; /* Check errors */
+       int reason;             /* Reason for child's death */
+       int status;             /* Description of child's death */
+       pid_t cpid;             /* Child actually found */
+       pid_t retstat;          /* Result of wait */
+       StringListNode *cmdNode; /* Node where current command is located */
+       const char **volatile av; /* Argument vector for thing to exec */
+       char **volatile mav;    /* Copy of the argument vector for freeing */
+       Boolean useShell;       /* TRUE if command should be executed
                                 * using a shell */
-    const char *volatile cmd = cmdp;
+       const char *volatile cmd = cmdp;
 
-    silent = (gn->type & OP_SILENT) != 0;
-    errCheck = !(gn->type & OP_IGNORE);
-    doIt = FALSE;
+       silent = (gn->type & OP_SILENT) != 0;
+       errCheck = !(gn->type & OP_IGNORE);
+       doIt = FALSE;
 
-    /* Luckily the commands don't end up in a string pool, otherwise
-     * this comparison could match too early, in a dependency using "..."
-     * for delayed commands, run in parallel mode, using the same shell
-     * command line more than once; see JobPrintCommand.
-     * TODO: write a unit-test to protect against this potential bug. */
-    cmdNode = Lst_FindDatum(gn->commands, cmd);
-    (void)Var_Subst(cmd, gn, VARE_WANTRES, &cmdStart);
-    /* TODO: handle errors */
+       /* Luckily the commands don't end up in a string pool, otherwise
+        * this comparison could match too early, in a dependency using "..."
+        * for delayed commands, run in parallel mode, using the same shell
+        * command line more than once; see JobPrintCommand.
+        * TODO: write a unit-test to protect against this potential bug. */
+       cmdNode = Lst_FindDatum(gn->commands, cmd);
+       (void)Var_Subst(cmd, gn, VARE_WANTRES, &cmdStart);
+       /* TODO: handle errors */
 
-    if (cmdStart[0] == '\0') {
-       free(cmdStart);
-       return 0;
-    }
-    cmd = cmdStart;
-    LstNode_Set(cmdNode, cmdStart);
+       if (cmdStart[0] == '\0') {
+               free(cmdStart);
+               return 0;
+       }
+       cmd = cmdStart;
+       LstNode_Set(cmdNode, cmdStart);
 
-    if (gn->type & OP_SAVE_CMDS) {
-       GNode *endNode = Targ_GetEndNode();
-       if (gn != endNode) {
-           Lst_Append(endNode->commands, cmdStart);
-           return 0;
+       if (gn->type & OP_SAVE_CMDS) {
+               GNode *endNode = Targ_GetEndNode();
+               if (gn != endNode) {
+                       Lst_Append(endNode->commands, cmdStart);
+                       return 0;
+               }
        }
-    }
-    if (strcmp(cmdStart, "...") == 0) {
-       gn->type |= OP_SAVE_CMDS;
-       return 0;
-    }
+       if (strcmp(cmdStart, "...") == 0) {
+               gn->type |= OP_SAVE_CMDS;
+               return 0;
+       }
 
-    for (;;) {
-       if (*cmd == '@')
-           silent = !DEBUG(LOUD);
-       else if (*cmd == '-')
-           errCheck = FALSE;
-       else if (*cmd == '+') {
-           doIt = TRUE;
-           if (!shellName)     /* we came here from jobs */
-               Shell_Init();
-       } else
-           break;
-       cmd++;
-    }
+       for (;;) {
+               if (*cmd == '@')
+                       silent = !DEBUG(LOUD);
+               else if (*cmd == '-')
+                       errCheck = FALSE;
+               else if (*cmd == '+') {
+                       doIt = TRUE;
+                       if (!shellName) /* we came here from jobs */
+                               Shell_Init();
+               } else
+                       break;
+               cmd++;
+       }
 
-    while (ch_isspace(*cmd))
-       cmd++;
+       while (ch_isspace(*cmd))
+               cmd++;
 
-    /*
-     * If we did not end up with a command, just skip it.
-     */
-    if (cmd[0] == '\0')
-       return 0;
+       /*
+        * If we did not end up with a command, just skip it.
+        */
+       if (cmd[0] == '\0')
+               return 0;
 
 #if !defined(MAKE_NATIVE)
-    /*
-     * In a non-native build, the host environment might be weird enough
-     * that it's necessary to go through a shell to get the correct
-     * behaviour.  Or perhaps the shell has been replaced with something
-     * that does extra logging, and that should not be bypassed.
-     */
-    useShell = TRUE;
+       /*
+        * In a non-native build, the host environment might be weird enough
+        * that it's necessary to go through a shell to get the correct
+        * behaviour.  Or perhaps the shell has been replaced with something
+        * that does extra logging, and that should not be bypassed.
+        */
+       useShell = TRUE;
 #else
-    /*
-     * Search for meta characters in the command. If there are no meta
-     * characters, there's no need to execute a shell to execute the
-     * command.
-     *
-     * Additionally variable assignments and empty commands
-     * go to the shell. Therefore treat '=' and ':' like shell
-     * meta characters as documented in make(1).
-     */
+       /*
+        * Search for meta characters in the command. If there are no meta
+        * characters, there's no need to execute a shell to execute the
+        * command.
+        *
+        * Additionally variable assignments and empty commands
+        * go to the shell. Therefore treat '=' and ':' like shell
+        * meta characters as documented in make(1).
+        */
 
-    useShell = needshell(cmd);
+       useShell = needshell(cmd);
 #endif
 
-    /*
-     * Print the command before echoing if we're not supposed to be quiet for
-     * this one. We also print the command if -n given.
-     */
-    if (!silent || !GNode_ShouldExecute(gn)) {
-       printf("%s\n", cmd);
-       fflush(stdout);
-    }
+       /*
+        * Print the command before echoing if we're not supposed to be quiet
+        * for this one. We also print the command if -n given.
+        */
+       if (!silent || !GNode_ShouldExecute(gn)) {
+               printf("%s\n", cmd);
+               fflush(stdout);
+       }
 
-    /*
-     * If we're not supposed to execute any commands, this is as far as
-     * we go...
-     */
-    if (!doIt && !GNode_ShouldExecute(gn))
-       return 0;
-
-    DEBUG1(JOB, "Execute: '%s'\n", cmd);
-
-    if (useShell) {
        /*
-        * We need to pass the command off to the shell, typically



Home | Main Index | Thread Index | Old Index