 @@ -1,2252 +1,2254 @@
-/*	$NetBSD: main.c,v 1.408 2020/10/27 19:16:46 rillig Exp $	*/
+/*	$NetBSD: main.c,v 1.409 2020/10/28 03:21:25 rillig Exp $	*/
 /*
  * Copyright (c) 1988, 1989, 1990, 1993
  *	The Regents of the University of California.  All rights reserved.
+ *
  * This code is derived from software contributed to Berkeley by
  * Adam de Boor.
+ *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
  * 3. Neither the name of the University nor the names of its contributors
  *    may be used to endorse or promote products derived from this software
  *    without specific prior written permission.
+ *
  * THIS SOFTWARE IS PROVIDED BY THE REGENTS AND CONTRIBUTORS ``AS IS'' AND
  * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
  * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
  * ARE DISCLAIMED.  IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE
  * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
  * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
  * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
  * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
  * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
  * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
  * SUCH DAMAGE.
  */
 /*
  * Copyright (c) 1989 by Berkeley Softworks
  * All rights reserved.
+ *
  * This code is derived from software contributed to Berkeley by
  * Adam de Boor.
+ *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
  * 3. All advertising materials mentioning features or use of this software
  *    must display the following acknowledgement:
  *	This product includes software developed by the University of
  *	California, Berkeley and its contributors.
  * 4. Neither the name of the University nor the names of its contributors
  *    may be used to endorse or promote products derived from this software
  *    without specific prior written permission.
+ *
  * THIS SOFTWARE IS PROVIDED BY THE REGENTS AND CONTRIBUTORS ``AS IS'' AND
  * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
  * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
  * ARE DISCLAIMED.  IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE
  * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
  * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
  * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
  * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
  * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
  * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
  * SUCH DAMAGE.
  */
 /*-
  * main.c --
  *	The main file for this entire program. Exit routines etc
  *	reside here.
+ *
  * Utility functions defined in this file:
  *	Main_ParseArgLine	Takes a line of arguments, breaks them and
  *				treats them as if they were given when first
  *				invoked. Used by the parse module to implement
  *				the .MFLAGS target.
+ *
  *	Error			Print a tagged error message. The global
  *				MAKE variable must have been defined. This
  *				takes a format string and optional arguments
  *				for it.
+ *
  *	Fatal			Print an error message and exit. Also takes
  *				a format string and arguments for it.
+ *
  *	Punt			Aborts all jobs and exits with a message. Also
  *				takes a format string and arguments for it.
+ *
  *	Finish			Finish things up by printing the number of
  *				errors which occurred, as passed to it, and
  *				exiting.
  */
 #include <sys/types.h>
 #include <sys/time.h>
 #include <sys/param.h>
 #include <sys/resource.h>
 #include <sys/stat.h>
 #ifdef MAKE_NATIVE
 #include <sys/sysctl.h>
 #endif
 #include <sys/utsname.h>
 #include <sys/wait.h>
 #include <errno.h>
 #include <signal.h>
 #include <stdarg.h>
 #include <time.h>
 #include "make.h"
 #include "dir.h"
 #include "job.h"
 #include "pathnames.h"
 #include "trace.h"
 /*	"@(#)main.c	8.3 (Berkeley) 3/19/94"	*/
-MAKE_RCSID("$NetBSD: main.c,v 1.408 2020/10/27 19:16:46 rillig Exp $");
+MAKE_RCSID("$NetBSD: main.c,v 1.409 2020/10/28 03:21:25 rillig Exp $");
 #if defined(MAKE_NATIVE) && !defined(lint)
 __COPYRIGHT("@(#) Copyright (c) 1988, 1989, 1990, 1993 "
 	    "The Regents of the University of California.  "
 	    "All rights reserved.");
 #endif
 #ifndef	DEFMAXLOCAL
 #define	DEFMAXLOCAL DEFMAXJOBS
 #endif
 CmdOpts opts;
 time_t			now;		/* Time at start of make */
 GNode			*DEFAULT;	/* .DEFAULT node */
 Boolean			allPrecious;	/* .PRECIOUS given on line by itself */
 Boolean			deleteOnError;	/* .DELETE_ON_ERROR: set */
 static int		maxJobTokens;	/* -j argument */
 Boolean			enterFlagObj;	/* -w and objdir != srcdir */
 Boolean			oldVars;	/* variable substitution style */
 static int jp_0 = -1, jp_1 = -1;	/* ends of parent job pipe */
 Boolean			doing_depend;	/* Set while reading .depend */
 static Boolean		jobsRunning;	/* TRUE if the jobs might be running */
 static const char *	tracefile;
 static int		ReadMakefile(const char *);
 static void		usage(void) MAKE_ATTR_DEAD;
 static void		purge_cached_realpaths(void);
 static Boolean		ignorePWD;	/* if we use -C, PWD is meaningless */
 static char objdir[MAXPATHLEN + 1];	/* where we chdir'ed to */
 char curdir[MAXPATHLEN + 1];		/* Startup directory */
 char *progname;				/* the program name */
 char *makeDependfile;
 pid_t myPid;
 int makelevel;
 Boolean forceJobs = FALSE;
 extern SearchPath *parseIncPath;
 /*
  * For compatibility with the POSIX version of MAKEFLAGS that includes
  * all the options with out -, convert flags to -f -l -a -g -s.
  */
 static char *
 explode(const char *flags)
+{
     size_t len;
     char *nf, *st;
     const char *f;
     if (flags == NULL)
 	return NULL;
     for (f = flags; *f; f++)
 	if (!ch_isalpha(*f))
 	    break;
     if (*f)
 	return bmake_strdup(flags);
     len = strlen(flags);
     st = nf = bmake_malloc(len * 3 + 1);
     while (*flags) {
 	*nf++ = '-';
 	*nf++ = *flags++;
 	*nf++ = ' ';
+    }
     *nf = '\0';
     return st;
+}
 static void
 parse_debug_option_F(const char *modules)
+{
     const char *mode;
     size_t len;
     char *fname;
     if (opts.debug_file != stdout && opts.debug_file != stderr)
 	fclose(opts.debug_file);
     if (*modules == '+') {
 	modules++;
 	mode = "a";
     } else
 	mode = "w";
     if (strcmp(modules, "stdout") == 0) {
 	opts.debug_file = stdout;
 	return;
+    }
     if (strcmp(modules, "stderr") == 0) {
 	opts.debug_file = stderr;
 	return;
+    }
     len = strlen(modules);
     fname = bmake_malloc(len + 20);
     memcpy(fname, modules, len + 1);
     /* Let the filename be modified by the pid */
     if (strcmp(fname + len - 3, ".%d") == 0)
 	snprintf(fname + len - 2, 20, "%d", getpid());
     opts.debug_file = fopen(fname, mode);
     if (!opts.debug_file) {
 	fprintf(stderr, "Cannot open debug file %s\n",
 		fname);
 	usage();
+    }
     free(fname);
+}
 static void
 parse_debug_options(const char *argvalue)
+{
 	const char *modules;
 	for (modules = argvalue; *modules; ++modules) {
 		switch (*modules) {
 		case '0':	/* undocumented, only intended for tests */
 			opts.debug &= DEBUG_LINT;
 			break;
 		case 'A':
 			opts.debug = ~(0|DEBUG_LINT);
 			break;
 		case 'a':
 			opts.debug |= DEBUG_ARCH;
 			break;
 		case 'C':
 			opts.debug |= DEBUG_CWD;
 			break;
 		case 'c':
 			opts.debug |= DEBUG_COND;
 			break;
 		case 'd':
 			opts.debug |= DEBUG_DIR;
 			break;
 		case 'e':
 			opts.debug |= DEBUG_ERROR;
 			break;
 		case 'f':
 			opts.debug |= DEBUG_FOR;
 			break;
 		case 'g':
 			if (modules[1] == '1') {
 				opts.debug |= DEBUG_GRAPH1;
 				++modules;
+			}
 			else if (modules[1] == '2') {
 				opts.debug |= DEBUG_GRAPH2;
 				++modules;
+			}
 			else if (modules[1] == '3') {
 				opts.debug |= DEBUG_GRAPH3;
 				++modules;
+			}
 			break;
 		case 'h':
 			opts.debug |= DEBUG_HASH;
 			break;
 		case 'j':
 			opts.debug |= DEBUG_JOB;
 			break;
 		case 'L':
 			opts.debug |= DEBUG_LINT;
 			break;
 		case 'l':
 			opts.debug |= DEBUG_LOUD;
 			break;
 		case 'M':
 			opts.debug |= DEBUG_META;
 			break;
 		case 'm':
 			opts.debug |= DEBUG_MAKE;
 			break;
 		case 'n':
 			opts.debug |= DEBUG_SCRIPT;
 			break;
 		case 'p':
 			opts.debug |= DEBUG_PARSE;
 			break;
 		case 's':
 			opts.debug |= DEBUG_SUFF;
 			break;
 		case 't':
 			opts.debug |= DEBUG_TARG;
 			break;
 		case 'V':
 			opts.debugVflag = TRUE;
 			break;
 		case 'v':
 			opts.debug |= DEBUG_VAR;
 			break;
 		case 'x':
 			opts.debug |= DEBUG_SHELL;
 			break;
 		case 'F':
 			parse_debug_option_F(modules + 1);
 			goto debug_setbuf;
 		default:
 			(void)fprintf(stderr,
 			    "%s: illegal argument to d option -- %c\n",
 			    progname, *modules);
 			usage();
+		}
+	}
 debug_setbuf:
 	/*
 	 * Make the debug_file unbuffered, and make
 	 * stdout line buffered (unless debugfile == stdout).
 	 */
 	setvbuf(opts.debug_file, NULL, _IONBF, 0);
 	if (opts.debug_file != stdout) {
 		setvbuf(stdout, NULL, _IOLBF, 0);
+	}
+}
 /*
  * does path contain any relative components
  */
 static Boolean
 is_relpath(const char *path)
+{
 	const char *cp;
 	if (path[0] != '/')
 		return TRUE;
 	cp = path;
 	while ((cp = strstr(cp, "/.")) != NULL) {
 		cp += 2;
 		if (cp[0] == '/' || cp[0] == '\0')
 			return TRUE;
 		else if (cp[0] == '.') {
 			if (cp[1] == '/' || cp[1] == '\0')
 				return TRUE;
+		}
+	}
 	return FALSE;
+}
 static void
 MainParseArgChdir(const char *argvalue)
+{
 	struct stat sa, sb;
 	if (chdir(argvalue) == -1) {
 		(void)fprintf(stderr, "%s: chdir %s: %s\n",
 		    progname, argvalue, strerror(errno));
 		exit(1);
+	}
 	if (getcwd(curdir, MAXPATHLEN) == NULL) {
 		(void)fprintf(stderr, "%s: %s.\n", progname, strerror(errno));
 		exit(2);
+	}
 	if (!is_relpath(argvalue) &&
 	    stat(argvalue, &sa) != -1 &&
 	    stat(curdir, &sb) != -1 &&
 	    sa.st_ino == sb.st_ino &&
 	    sa.st_dev == sb.st_dev)
 		strncpy(curdir, argvalue, MAXPATHLEN);
 	ignorePWD = TRUE;
+}
 static void
 MainParseArgJobsInternal(const char *argvalue)
+{
 	if (sscanf(argvalue, "%d,%d", &jp_0, &jp_1) != 2) {
 		(void)fprintf(stderr,
 			      "%s: internal error -- J option malformed (%s)\n",
 			      progname, argvalue);
 		usage();
+	}
 	if ((fcntl(jp_0, F_GETFD, 0) < 0) ||
 	    (fcntl(jp_1, F_GETFD, 0) < 0)) {
 #if 0
 		(void)fprintf(stderr,
 		    "%s: ###### warning -- J descriptors were closed!\n",
 		    progname);
 		exit(2);
 #endif
 		jp_0 = -1;
 		jp_1 = -1;
 		opts.compatMake = TRUE;
 	} else {
 		Var_Append(MAKEFLAGS, "-J", VAR_GLOBAL);
 		Var_Append(MAKEFLAGS, argvalue, VAR_GLOBAL);
+	}
+}
 static void
 MainParseArgJobs(const char *argvalue)
+{
 	char *p;
 	forceJobs = TRUE;
 	opts.maxJobs = (int)strtol(argvalue, &p, 0);
 	if (*p != '\0' || opts.maxJobs < 1) {
 		(void)fprintf(stderr,
 		    "%s: illegal argument to -j -- must be positive integer!\n",
 		    progname);
 		exit(1);	/* XXX: why not 2? */
+	}
 	Var_Append(MAKEFLAGS, "-j", VAR_GLOBAL);
 	Var_Append(MAKEFLAGS, argvalue, VAR_GLOBAL);
 	Var_Set(".MAKE.JOBS", argvalue, VAR_GLOBAL);
 	maxJobTokens = opts.maxJobs;
+}
 static void
 MainParseArgSysInc(const char *argvalue)
+{
 	/* look for magic parent directory search string */
 	if (strncmp(".../", argvalue, 4) == 0) {
 		char *found_path = Dir_FindHereOrAbove(curdir, argvalue + 4);
 		if (found_path == NULL)
 			return;
 		(void)Dir_AddDir(sysIncPath, found_path);
 		free(found_path);
 	} else {
 		(void)Dir_AddDir(sysIncPath, argvalue);
+	}
 	Var_Append(MAKEFLAGS, "-m", VAR_GLOBAL);
 	Var_Append(MAKEFLAGS, argvalue, VAR_GLOBAL);
+}
 static Boolean
 MainParseArg(char c, const char *argvalue)
+{
 	switch (c) {
 	case '\0':
 		break;
 	case 'B':
 		opts.compatMake = TRUE;
 		Var_Append(MAKEFLAGS, "-B", VAR_GLOBAL);
 		Var_Set(MAKE_MODE, "compat", VAR_GLOBAL);
 		break;
 	case 'C':
 		MainParseArgChdir(argvalue);
 		break;
 	case 'D':
 		if (argvalue[0] == '\0') return FALSE;
 		Var_Set(argvalue, "1", VAR_GLOBAL);
 		Var_Append(MAKEFLAGS, "-D", VAR_GLOBAL);
 		Var_Append(MAKEFLAGS, argvalue, VAR_GLOBAL);
 		break;
 	case 'I':
 		Parse_AddIncludeDir(argvalue);
 		Var_Append(MAKEFLAGS, "-I", VAR_GLOBAL);
 		Var_Append(MAKEFLAGS, argvalue, VAR_GLOBAL);
 		break;
 	case 'J':
 		MainParseArgJobsInternal(argvalue);
 		break;
 	case 'N':
 		opts.noExecute = TRUE;
 		opts.noRecursiveExecute = TRUE;
 		Var_Append(MAKEFLAGS, "-N", VAR_GLOBAL);
 		break;
 	case 'S':
 		opts.keepgoing = FALSE;
 		Var_Append(MAKEFLAGS, "-S", VAR_GLOBAL);
 		break;
 	case 'T':
 		tracefile = bmake_strdup(argvalue);
 		Var_Append(MAKEFLAGS, "-T", VAR_GLOBAL);
 		Var_Append(MAKEFLAGS, argvalue, VAR_GLOBAL);
 		break;
 	case 'V':
 	case 'v':
 		opts.printVars = c == 'v' ? EXPAND_VARS : COMPAT_VARS;
 		Lst_Append(opts.variables, bmake_strdup(argvalue));
 		/* XXX: Why always -V? */
 		Var_Append(MAKEFLAGS, "-V", VAR_GLOBAL);
 		Var_Append(MAKEFLAGS, argvalue, VAR_GLOBAL);
 		break;
 	case 'W':
 		opts.parseWarnFatal = TRUE;
 		break;
 	case 'X':
 		opts.varNoExportEnv = TRUE;
 		Var_Append(MAKEFLAGS, "-X", VAR_GLOBAL);
 		break;
 	case 'd':
 		/* If '-d-opts' don't pass to children */
 		if (argvalue[0] == '-')
 			argvalue++;
 		else {
 			Var_Append(MAKEFLAGS, "-d", VAR_GLOBAL);
 			Var_Append(MAKEFLAGS, argvalue, VAR_GLOBAL);
+		}
 		parse_debug_options(argvalue);
 		break;
 	case 'e':
 		opts.checkEnvFirst = TRUE;
 		Var_Append(MAKEFLAGS, "-e", VAR_GLOBAL);
 		break;
 	case 'f':
 		Lst_Append(opts.makefiles, bmake_strdup(argvalue));
 		break;
 	case 'i':
 		opts.ignoreErrors = TRUE;
 		Var_Append(MAKEFLAGS, "-i", VAR_GLOBAL);
 		break;
 	case 'j':
 		MainParseArgJobs(argvalue);
 		break;
 	case 'k':
 		opts.keepgoing = TRUE;
 		Var_Append(MAKEFLAGS, "-k", VAR_GLOBAL);
 		break;
 	case 'm':
 		MainParseArgSysInc(argvalue);
 		break;
 	case 'n':
 		opts.noExecute = TRUE;
 		Var_Append(MAKEFLAGS, "-n", VAR_GLOBAL);
 		break;
 	case 'q':
 		opts.queryFlag = TRUE;
 		/* Kind of nonsensical, wot? */
 		Var_Append(MAKEFLAGS, "-q", VAR_GLOBAL);
 		break;
 	case 'r':
 		opts.noBuiltins = TRUE;
 		Var_Append(MAKEFLAGS, "-r", VAR_GLOBAL);
 		break;
 	case 's':
 		opts.beSilent = TRUE;
 		Var_Append(MAKEFLAGS, "-s", VAR_GLOBAL);
 		break;
 	case 't':
 		opts.touchFlag = TRUE;
 		Var_Append(MAKEFLAGS, "-t", VAR_GLOBAL);
 		break;
 	case 'w':
 		opts.enterFlag = TRUE;
 		Var_Append(MAKEFLAGS, "-w", VAR_GLOBAL);
 		break;
 	default:
 	case '?':
 		usage();
+	}
 	return TRUE;
+}
 /* Parse the given arguments.  Called from main() and from
  * Main_ParseArgLine() when the .MAKEFLAGS target is used.
+ *
  * The arguments must be treated as read-only and will be freed after the
  * call.
+ *
  * XXX: Deal with command line overriding .MAKEFLAGS in makefile */
 static void
 MainParseArgs(int argc, char **argv)
+{
 	char c;
 	int arginc;
 	char *argvalue;
 	char *optscan;
 	Boolean inOption, dashDash = FALSE;
 	const char *optspecs = "BC:D:I:J:NST:V:WXd:ef:ij:km:nqrstv:w";
 /* Can't actually use getopt(3) because rescanning is not portable */
 rearg:
 	inOption = FALSE;
 	optscan = NULL;
 	while (argc > 1) {
 		const char *optspec;
 		if (!inOption)
 			optscan = argv[1];
 		c = *optscan++;
 		arginc = 0;
 		if (inOption) {
 			if (c == '\0') {
 				++argv;
 				--argc;
 				inOption = FALSE;
 				continue;
+			}
 		} else {
 			if (c != '-' || dashDash)
 				break;
 			inOption = TRUE;
 			c = *optscan++;
+		}
 		/* '-' found at some earlier point */
 		optspec = strchr(optspecs, c);
 		if (c != '\0' && optspec != NULL && optspec[1] == ':') {
 			/* -<something> found, and <something> should have an arg */
 			inOption = FALSE;
 			arginc = 1;
 			argvalue = optscan;
 			if (*argvalue == '\0') {
 				if (argc < 3)
 					goto noarg;
 				argvalue = argv[2];
 				arginc = 2;
+			}
 		} else {
 			argvalue = NULL;
+		}
 		switch (c) {
 		case '\0':
 			arginc = 1;
 			inOption = FALSE;
 			break;
 		case '-':
 			dashDash = TRUE;
 			break;
 		default:
 			if (!MainParseArg(c, argvalue))
 				goto noarg;
+		}
 		argv += arginc;
 		argc -= arginc;
+	}
 	oldVars = TRUE;
 	/*
 	 * See if the rest of the arguments are variable assignments and
 	 * perform them if so. Else take them to be targets and stuff them
 	 * on the end of the "create" list.
 	 */
 	for (; argc > 1; ++argv, --argc) {
 		VarAssign var;
 		if (Parse_IsVar(argv[1], &var)) {
 			Parse_DoVar(&var, VAR_CMD);
 		} else {
 			if (!*argv[1])
 				Punt("illegal (null) argument.");
 			if (*argv[1] == '-' && !dashDash)
 				goto rearg;
 			Lst_Append(opts.create, bmake_strdup(argv[1]));
+		}
+	}
 	return;
 noarg:
 	(void)fprintf(stderr, "%s: option requires an argument -- %c\n",
 	    progname, c);
 	usage();
+}
 /* Break a line of arguments into words and parse them.
+ *
  * Used when a .MFLAGS or .MAKEFLAGS target is encountered during parsing and
  * by main() when reading the MAKEFLAGS environment variable. */
 void
 Main_ParseArgLine(const char *line)
+{
 	Words words;
 	char *p1;
 	const char *argv0 = Var_Value(".MAKE", VAR_GLOBAL, &p1);
 	char *buf;
 	if (line == NULL)
 		return;
 	for (; *line == ' '; ++line)
 		continue;
 	if (!*line)
 		return;
 	buf = str_concat3(argv0, " ", line);
 	free(p1);
 	words = Str_Words(buf, TRUE);
 	if (words.words == NULL) {
 		Error("Unterminated quoted string [%s]", buf);
 		free(buf);
 		return;
+	}
 	free(buf);
 	MainParseArgs((int)words.len, words.words);
 	Words_Free(words);
+}
 Boolean
 Main_SetObjdir(const char *fmt, ...)
+{
 	struct stat sb;
 	char *path;
 	char buf[MAXPATHLEN + 1];
 	char buf2[MAXPATHLEN + 1];
 	Boolean rc = FALSE;
 	va_list ap;
 	va_start(ap, fmt);
 	vsnprintf(path = buf, MAXPATHLEN, fmt, ap);
 	va_end(ap);
 	if (path[0] != '/') {
 		snprintf(buf2, MAXPATHLEN, "%s/%s", curdir, path);
 		path = buf2;
+	}
 	/* look for the directory and try to chdir there */
 	if (stat(path, &sb) == 0 && S_ISDIR(sb.st_mode)) {
 		if (chdir(path)) {
 			(void)fprintf(stderr, "make warning: %s: %s.\n",
 				      path, strerror(errno));
 		} else {
 			snprintf(objdir, sizeof objdir, "%s", path);
 			Var_Set(".OBJDIR", objdir, VAR_GLOBAL);
 			setenv("PWD", objdir, 1);
 			Dir_InitDot();
 			purge_cached_realpaths();
 			rc = TRUE;
 			if (opts.enterFlag && strcmp(objdir, curdir) != 0)
 				enterFlagObj = TRUE;
+		}
+	}
 	return rc;
+}
 static Boolean
 Main_SetVarObjdir(const char *var, const char *suffix)
+{
 	char *path_freeIt;
 	const char *path = Var_Value(var, VAR_CMD, &path_freeIt);
 	const char *xpath;
 	char *xpath_freeIt;
 	if (path == NULL || path[0] == '\0') {
 		bmake_free(path_freeIt);
 		return FALSE;
+	}
 	/* expand variable substitutions */
 	xpath = path;
 	xpath_freeIt = NULL;
 	if (strchr(path, '$') != 0) {
 		(void)Var_Subst(path, VAR_GLOBAL, VARE_WANTRES, &xpath_freeIt);
 		/* TODO: handle errors */
 		xpath = xpath_freeIt;
+	}
 	(void)Main_SetObjdir("%s%s", xpath, suffix);
 	bmake_free(xpath_freeIt);
 	bmake_free(path_freeIt);
 	return TRUE;
+}
 /* Read and parse the makefile.
  * Return TRUE if reading the makefile succeeded. */
 static int
 ReadMakefileSucceeded(void *fname, void *unused)
+{
 	return ReadMakefile(fname) == 0;
+}
 int
 str2Lst_Append(StringList *lp, char *str, const char *sep)
+{
     char *cp;
     int n;
     if (!sep)
 	sep = " \t";
     for (n = 0, cp = strtok(str, sep); cp; cp = strtok(NULL, sep)) {
 	Lst_Append(lp, cp);
 	n++;
+    }
     return n;
+}
 #ifdef SIGINFO
 /*ARGSUSED*/
 static void
 siginfo(int signo MAKE_ATTR_UNUSED)
+{
 	char dir[MAXPATHLEN];
 	char str[2 * MAXPATHLEN];
 	int len;
 	if (getcwd(dir, sizeof(dir)) == NULL)
 		return;
 	len = snprintf(str, sizeof(str), "%s: Working in: %s\n", progname, dir);
 	if (len > 0)
 		(void)write(STDERR_FILENO, str, (size_t)len);
+}
 #endif
 /*
  * Allow makefiles some control over the mode we run in.
  */
 void
 MakeMode(const char *mode)
+{
     char *mode_freeIt = NULL;
     if (mode == NULL) {
 	(void)Var_Subst("${" MAKE_MODE ":tl}",
 		  VAR_GLOBAL, VARE_WANTRES, &mode_freeIt);
 	/* TODO: handle errors */
 	mode = mode_freeIt;
+    }
     if (mode[0] != '\0') {
 	if (strstr(mode, "compat")) {
 	    opts.compatMake = TRUE;
 	    forceJobs = FALSE;
+	}
 #if USE_META
 	if (strstr(mode, "meta"))
 	    meta_mode_init(mode);
 #endif
+    }
     free(mode_freeIt);
+}
 static void
 PrintVar(const char *varname, Boolean expandVars)
+{
 	if (strchr(varname, '$')) {
 		char *evalue;
 		(void)Var_Subst(varname, VAR_GLOBAL, VARE_WANTRES, &evalue);
 		/* TODO: handle errors */
 		printf("%s\n", evalue);
 		bmake_free(evalue);
 	} else if (expandVars) {
 		char *expr = str_concat3("${", varname, "}");
 		char *evalue;
 		(void)Var_Subst(expr, VAR_GLOBAL, VARE_WANTRES, &evalue);
 		/* TODO: handle errors */
 		free(expr);
 		printf("%s\n", evalue);
 		bmake_free(evalue);
 	} else {
 		char *freeIt;
 		const char *value = Var_Value(varname, VAR_GLOBAL, &freeIt);
 		printf("%s\n", value ? value : "");
 		bmake_free(freeIt);
+	}
+}
 static void
 doPrintVars(void)
+{
 	StringListNode *ln;
 	Boolean expandVars;
 	if (opts.printVars == EXPAND_VARS)
 		expandVars = TRUE;
 	else if (opts.debugVflag)
 		expandVars = FALSE;
 	else
 		expandVars = getBoolean(".MAKE.EXPAND_VARIABLES", FALSE);
 	for (ln = opts.variables->first; ln != NULL; ln = ln->next) {
 		const char *varname = ln->datum;
 		PrintVar(varname, expandVars);
+	}
+}
 static Boolean
 runTargets(void)
+{
 	GNodeList *targs; /* target nodes to create -- passed to Make_Init */
 	Boolean outOfDate;	/* FALSE if all targets up to date */
 	/*
 	 * Have now read the entire graph and need to make a list of
 	 * targets to create. If none was given on the command line,
 	 * we consult the parsing module to find the main target(s)
 	 * to create.
 	 */
 	if (Lst_IsEmpty(opts.create))
 		targs = Parse_MainName();
 	else
 		targs = Targ_FindList(opts.create);
 	if (!opts.compatMake) {
 		/*
 		 * Initialize job module before traversing the graph
 		 * now that any .BEGIN and .END targets have been read.
 		 * This is done only if the -q flag wasn't given
 		 * (to prevent the .BEGIN from being executed should
 		 * it exist).
 		 */
 		if (!opts.queryFlag) {
 			Job_Init();
 			jobsRunning = TRUE;
+		}
 		/* Traverse the graph, checking on all the targets */
 		outOfDate = Make_Run(targs);
 	} else {
 		/*
 		 * Compat_Init will take care of creating all the
 		 * targets as well as initializing the module.
 		 */
 		Compat_Run(targs);
 		outOfDate = FALSE;
+	}
 	Lst_Free(targs);
 	return outOfDate;
+}
 /*
  * Set up the .TARGETS variable to contain the list of targets to be
  * created. If none specified, make the variable empty -- the parser
  * will fill the thing in with the default or .MAIN target.
  */
 static void
 InitVarTargets(void)
+{
 	StringListNode *ln;
 	if (Lst_IsEmpty(opts.create)) {
 		Var_Set(".TARGETS", "", VAR_GLOBAL);
 		return;
+	}
 	for (ln = opts.create->first; ln != NULL; ln = ln->next) {
 		char *name = ln->datum;
 		Var_Append(".TARGETS", name, VAR_GLOBAL);
+	}
+}
 static void
 InitRandom(void)
+{
 	struct timeval tv;
 	gettimeofday(&tv, NULL);
 	srandom((unsigned int)(tv.tv_sec + tv.tv_usec));
+}
 static const char *
 init_machine(const struct utsname *utsname)
+{
 	const char *machine = getenv("MACHINE");
 	if (machine != NULL)
 		return machine;
 #ifdef MAKE_NATIVE
 	return utsname->machine;
 #else
 #ifdef MAKE_MACHINE
 	return MAKE_MACHINE;
 #else
 	return "unknown";
 #endif
 #endif
+}
 static const char *
 init_machine_arch(void)
+{
 	const char *env = getenv("MACHINE_ARCH");
 	if (env != NULL)
 		return env;
 #ifdef MAKE_NATIVE
+	{
 		struct utsname utsname;
 		static char machine_arch_buf[sizeof(utsname.machine)];
 		const int mib[2] = { CTL_HW, HW_MACHINE_ARCH };
 		size_t len = sizeof(machine_arch_buf);
 		if (sysctl(mib, __arraycount(mib), machine_arch_buf,
 			&len, NULL, 0) < 0) {
 		    (void)fprintf(stderr, "%s: sysctl failed (%s).\n", progname,
 			strerror(errno));
 		    exit(2);
+		}
 		return machine_arch_buf;
+	}
 #else
 #ifndef MACHINE_ARCH
 #ifdef MAKE_MACHINE_ARCH
 	return MAKE_MACHINE_ARCH;
 #else
 	return "unknown";
 #endif
 #else
 	return MACHINE_ARCH;
 #endif
 #endif
+}
 #ifndef NO_PWD_OVERRIDE
 /*
  * All this code is so that we know where we are when we start up
  * on a different machine with pmake.
+ *
  * Overriding getcwd() with $PWD totally breaks MAKEOBJDIRPREFIX
  * since the value of curdir can vary depending on how we got
  * here.  Ie sitting at a shell prompt (shell that provides $PWD)
  * or via subdir.mk in which case its likely a shell which does
  * not provide it.
+ *
  * So, to stop it breaking this case only, we ignore PWD if
  * MAKEOBJDIRPREFIX is set or MAKEOBJDIR contains a variable expression.
  */
 static void
 HandlePWD(const struct stat *curdir_st)
+{
 	char *pwd;
 	char *prefix_freeIt, *makeobjdir_freeIt;
 	const char *makeobjdir;
 	struct stat pwd_st;
 	if (ignorePWD || (pwd = getenv("PWD")) == NULL)
 		return;
 	if (Var_Value("MAKEOBJDIRPREFIX", VAR_CMD, &prefix_freeIt) != NULL) {
 		bmake_free(prefix_freeIt);
 		return;
+	}
 	makeobjdir = Var_Value("MAKEOBJDIR", VAR_CMD, &makeobjdir_freeIt);
 	if (makeobjdir != NULL && strchr(makeobjdir, '$') != NULL)
 		goto ignore_pwd;
 	if (stat(pwd, &pwd_st) == 0 &&
 	    curdir_st->st_ino == pwd_st.st_ino &&
 	    curdir_st->st_dev == pwd_st.st_dev)
 		(void)strncpy(curdir, pwd, MAXPATHLEN);
 ignore_pwd:
 	bmake_free(makeobjdir_freeIt);
+}
 #endif
 /*
  * Find the .OBJDIR.  If MAKEOBJDIRPREFIX, or failing that,
  * MAKEOBJDIR is set in the environment, try only that value
  * and fall back to .CURDIR if it does not exist.
+ *
  * Otherwise, try _PATH_OBJDIR.MACHINE-MACHINE_ARCH, _PATH_OBJDIR.MACHINE,
  * and * finally _PATH_OBJDIRPREFIX`pwd`, in that order.  If none
  * of these paths exist, just use .CURDIR.
  */
 static void
 InitObjdir(const char *machine, const char *machine_arch)
+{
 	Dir_InitDir(curdir);
 	(void)Main_SetObjdir("%s", curdir);
 	if (!Main_SetVarObjdir("MAKEOBJDIRPREFIX", curdir) &&
 	    !Main_SetVarObjdir("MAKEOBJDIR", "") &&
 	    !Main_SetObjdir("%s.%s-%s", _PATH_OBJDIR, machine, machine_arch) &&
 	    !Main_SetObjdir("%s.%s", _PATH_OBJDIR, machine) &&
 	    !Main_SetObjdir("%s", _PATH_OBJDIR))
 		(void)Main_SetObjdir("%s%s", _PATH_OBJDIRPREFIX, curdir);
+}
 /* get rid of resource limit on file descriptors */
 static void
 UnlimitFiles(void)
+{
 #if defined(MAKE_NATIVE) || (defined(HAVE_SETRLIMIT) && defined(RLIMIT_NOFILE))
 	struct rlimit rl;
 	if (getrlimit(RLIMIT_NOFILE, &rl) != -1 &&
 	    rl.rlim_cur != rl.rlim_max) {
 		rl.rlim_cur = rl.rlim_max;
 		(void)setrlimit(RLIMIT_NOFILE, &rl);
+	}
 #endif
+}
 static void
 CmdOpts_Init(void)
+{
 	opts.compatMake = FALSE;	/* No compat mode */
 	opts.debug = 0;			/* No debug verbosity, please. */
 	/* opts.debug_file has been initialized earlier */
 	opts.debugVflag = FALSE;
 	opts.checkEnvFirst = FALSE;
 	opts.makefiles = Lst_New();
 	opts.ignoreErrors = FALSE;	/* Pay attention to non-zero returns */
 	opts.maxJobs = DEFMAXLOCAL;	/* Set default local max concurrency */
 	opts.keepgoing = FALSE;		/* Stop on error */
 	opts.noRecursiveExecute = FALSE; /* Execute all .MAKE targets */
 	opts.noExecute = FALSE;		/* Execute all commands */
 	opts.queryFlag = FALSE;		/* This is not just a check-run */
 	opts.noBuiltins = FALSE;	/* Read the built-in rules */
 	opts.beSilent = FALSE;		/* Print commands as executed */
 	opts.touchFlag = FALSE;		/* Actually update targets */
 	opts.printVars = 0;
 	opts.variables = Lst_New();
 	opts.parseWarnFatal = FALSE;
 	opts.enterFlag = FALSE;
 	opts.varNoExportEnv = FALSE;
 	opts.create = Lst_New();
+}
 /* Initialize MAKE and .MAKE to the path of the executable, so that it can be
  * found by execvp(3) and the shells, even after a chdir.
+ *
  * If it's a relative path and contains a '/', resolve it to an absolute path.
  * Otherwise keep it as is, assuming it will be found in the PATH. */
 static void
 InitVarMake(const char *argv0)
+{
 	const char *make = argv0;
 	if (argv0[0] != '/' && strchr(argv0, '/') != NULL) {
 		char pathbuf[MAXPATHLEN];
 		const char *abs = cached_realpath(argv0, pathbuf);
 		struct stat st;
 		if (abs != NULL && abs[0] == '/' && stat(make, &st) == 0)
 			make = abs;
+	}
 	Var_Set("MAKE", make, VAR_GLOBAL);
 	Var_Set(".MAKE", make, VAR_GLOBAL);
+}
 static void
-InitDefIncPath(char *syspath)
+InitDefSysIncPath(char *syspath)
+{
 	static char defsyspath[] = _PATH_DEFSYSPATH;
 	char *start, *cp;
 	/*
 	 * If no user-supplied system path was given (through the -m option)
 	 * add the directories from the DEFSYSPATH (more than one may be given
 	 * as dir1:...:dirn) to the system include path.
 	 */
 	/* XXX: mismatch: the -m option sets sysIncPath, not syspath */
 	if (syspath == NULL || syspath[0] == '\0')
 		syspath = defsyspath;
 	else
 		syspath = bmake_strdup(syspath);
 	for (start = syspath; *start != '\0'; start = cp) {
 		for (cp = start; *cp != '\0' && *cp != ':'; cp++)
 			continue;
 		if (*cp == ':') {
 			*cp++ = '\0';
+		}
 		/* look for magic parent directory search string */
 		if (strncmp(".../", start, 4) != 0) {
-			(void)Dir_AddDir(defIncPath, start);
+			(void)Dir_AddDir(defSysIncPath, start);
 		} else {
 			char *dir = Dir_FindHereOrAbove(curdir, start + 4);
 			if (dir != NULL) {
-				(void)Dir_AddDir(defIncPath, dir);
+				(void)Dir_AddDir(defSysIncPath, dir);
 				free(dir);
+			}
+		}
+	}
 	if (syspath != defsyspath)
 		free(syspath);
+}
 static void
 ReadBuiltinRules(void)
+{
 	StringList *sysMkPath = Lst_New();
 	Dir_Expand(_PATH_DEFSYSMK,
-		   Lst_IsEmpty(sysIncPath) ? defIncPath : sysIncPath,
+		   Lst_IsEmpty(sysIncPath) ? defSysIncPath : sysIncPath,
 		   sysMkPath);
 	if (Lst_IsEmpty(sysMkPath))
 		Fatal("%s: no system rules (%s).", progname, _PATH_DEFSYSMK);
 	if (!Lst_ForEachUntil(sysMkPath, ReadMakefileSucceeded, NULL))
 		Fatal("%s: cannot open %s.", progname,
 		      (char *)sysMkPath->first->datum);
 	/* XXX: sysMkPath is not freed */
+}
 static void
 InitMaxJobs(void)
+{
 	char *value;
 	int n;
 	if (forceJobs || opts.compatMake ||
 	    !Var_Exists(".MAKE.JOBS", VAR_GLOBAL))
 		return;
 	(void)Var_Subst("${.MAKE.JOBS}", VAR_GLOBAL, VARE_WANTRES, &value);
 	/* TODO: handle errors */
 	n = (int)strtol(value, NULL, 0);
 	if (n < 1) {
 		(void)fprintf(stderr,
 			      "%s: illegal value for .MAKE.JOBS "
 			      "-- must be positive integer!\n",
 			      progname);
 		exit(1);
+	}
 	if (n != opts.maxJobs) {
 		Var_Append(MAKEFLAGS, "-j", VAR_GLOBAL);
 		Var_Append(MAKEFLAGS, value, VAR_GLOBAL);
+	}
 	opts.maxJobs = n;
 	maxJobTokens = opts.maxJobs;
 	forceJobs = TRUE;
 	free(value);
+}
 /*
  * For compatibility, look at the directories in the VPATH variable
  * and add them to the search path, if the variable is defined. The
  * variable's value is in the same format as the PATH environment
  * variable, i.e. <directory>:<directory>:<directory>...
  */
 static void
 InitVpath(void)
+{
 	char *vpath, savec, *path;
 	if (!Var_Exists("VPATH", VAR_CMD))
 		return;
 	(void)Var_Subst("${VPATH}", VAR_CMD, VARE_WANTRES, &vpath);
 	/* TODO: handle errors */
 	path = vpath;
 	do {
 		char *cp;
 		/* skip to end of directory */
 		for (cp = path; *cp != ':' && *cp != '\0'; cp++)
 			continue;
 		/* Save terminator character so know when to stop */
 		savec = *cp;
 		*cp = '\0';
 		/* Add directory to search path */
 		(void)Dir_AddDir(dirSearchPath, path);
 		*cp = savec;
 		path = cp + 1;
 	} while (savec == ':');
 	free(vpath);
+}
 static void
 ReadMakefiles(void)
+{
 	if (opts.makefiles->first != NULL) {
 		StringListNode *ln;
 		for (ln = opts.makefiles->first; ln != NULL; ln = ln->next) {
 			if (ReadMakefile(ln->datum) != 0)
 				Fatal("%s: cannot open %s.",
 				      progname, (char *)ln->datum);
+		}
 	} else {
 		char *p1;
 		(void)Var_Subst("${" MAKEFILE_PREFERENCE "}",
 				VAR_CMD, VARE_WANTRES, &p1);
 		/* TODO: handle errors */
 		(void)str2Lst_Append(opts.makefiles, p1, NULL);
 		(void)Lst_ForEachUntil(opts.makefiles,
 				       ReadMakefileSucceeded, NULL);
 		free(p1);
+	}
+}
 static void
 CleanUp(void)
+{
 #ifdef CLEANUP
 	Lst_Destroy(opts.variables, free);
 	Lst_Free(opts.makefiles);	/* don't free, may be used in GNodes */
 	Lst_Destroy(opts.create, free);
 #endif
 	/* print the graph now it's been processed if the user requested it */
 	if (DEBUG(GRAPH2))
 		Targ_PrintGraph(2);
 	Trace_Log(MAKEEND, 0);
 	if (enterFlagObj)
 		printf("%s: Leaving directory `%s'\n", progname, objdir);
 	if (opts.enterFlag)
 		printf("%s: Leaving directory `%s'\n", progname, curdir);
 #ifdef USE_META
 	meta_finish();
 #endif
 	Suff_End();
 	Targ_End();
 	Arch_End();
 	Var_End();
 	Parse_End();
 	Dir_End();
 	Job_End();
 	Trace_End();
+}
 /*-
  * main --
  *	The main function, for obvious reasons. Initializes variables
  *	and a few modules, then parses the arguments give it in the
  *	environment and on the command line. Reads the system makefile
  *	followed by either Makefile, makefile or the file given by the
  *	-f argument. Sets the .MAKEFLAGS PMake variable based on all the
  *	flags it has received by then uses either the Make or the Compat
  *	module to create the initial list of targets.
+ *
  * Results:
  *	If -q was given, exits -1 if anything was out-of-date. Else it exits
  *	0.
+ *
  * Side Effects:
  *	The program exits when done. Targets are created. etc. etc. etc.
  */
 int
 main(int argc, char **argv)
+{
 	Boolean outOfDate;	/* FALSE if all targets up to date */
 	struct stat sa;
 	char *p1;
 	const char *machine;
 	const char *machine_arch;
 	char *syspath = getenv("MAKESYSPATH");
 	struct utsname utsname;
 	/* default to writing debug to stderr */
 	opts.debug_file = stderr;
 #ifdef SIGINFO
 	(void)bmake_signal(SIGINFO, siginfo);
 #endif
 	InitRandom();
 	if ((progname = strrchr(argv[0], '/')) != NULL)
 		progname++;
 	else
 		progname = argv[0];
 	UnlimitFiles();
 	if (uname(&utsname) == -1) {
 	    (void)fprintf(stderr, "%s: uname failed (%s).\n", progname,
 		strerror(errno));
 	    exit(2);
+	}
 	/*
 	 * Get the name of this type of MACHINE from utsname
 	 * so we can share an executable for similar machines.
 	 * (i.e. m68k: amiga hp300, mac68k, sun3, ...)
+	 *
 	 * Note that both MACHINE and MACHINE_ARCH are decided at
 	 * run-time.
 	 */
 	machine = init_machine(&utsname);
 	machine_arch = init_machine_arch();
 	myPid = getpid();		/* remember this for vFork() */
 	/*
 	 * Just in case MAKEOBJDIR wants us to do something tricky.
 	 */
 	Var_Init();		/* Initialize the lists of variables for
 				 * parsing arguments */
 	Var_Set(".MAKE.OS", utsname.sysname, VAR_GLOBAL);
 	Var_Set("MACHINE", machine, VAR_GLOBAL);
 	Var_Set("MACHINE_ARCH", machine_arch, VAR_GLOBAL);
 #ifdef MAKE_VERSION
 	Var_Set("MAKE_VERSION", MAKE_VERSION, VAR_GLOBAL);
 #endif
 	Var_Set(".newline", "\n", VAR_GLOBAL); /* handy for :@ loops */
 	/*
 	 * This is the traditional preference for makefiles.
 	 */
 #ifndef MAKEFILE_PREFERENCE_LIST
 # define MAKEFILE_PREFERENCE_LIST "makefile Makefile"
 #endif
 	Var_Set(MAKEFILE_PREFERENCE, MAKEFILE_PREFERENCE_LIST,
 		VAR_GLOBAL);
 	Var_Set(MAKE_DEPENDFILE, ".depend", VAR_GLOBAL);
 	CmdOpts_Init();
 	allPrecious = FALSE;		/* Remove targets when interrupted */
 	deleteOnError = FALSE;		/* Historical default behavior */
 	jobsRunning = FALSE;
 	maxJobTokens = opts.maxJobs;
 	ignorePWD = FALSE;
 	/*
 	 * Initialize the parsing, directory and variable modules to prepare
 	 * for the reading of inclusion paths and variable settings on the
 	 * command line
 	 */
 	/*
 	 * Initialize various variables.
 	 *	MAKE also gets this name, for compatibility
 	 *	.MAKEFLAGS gets set to the empty string just in case.
 	 *	MFLAGS also gets initialized empty, for compatibility.
 	 */
 	Parse_Init();
 	InitVarMake(argv[0]);
 	Var_Set(MAKEFLAGS, "", VAR_GLOBAL);
 	Var_Set(MAKEOVERRIDES, "", VAR_GLOBAL);
 	Var_Set("MFLAGS", "", VAR_GLOBAL);
 	Var_Set(".ALLTARGETS", "", VAR_GLOBAL);
 	/* some makefiles need to know this */
 	Var_Set(MAKE_LEVEL ".ENV", MAKE_LEVEL_ENV, VAR_CMD);
 	/*
 	 * Set some other useful macros
 	 */
+	{
 	    char tmp[64], *ep;
 	    makelevel = ((ep = getenv(MAKE_LEVEL_ENV)) && *ep) ? atoi(ep) : 0;
 	    if (makelevel < 0)
 		makelevel = 0;
 	    snprintf(tmp, sizeof(tmp), "%d", makelevel);
 	    Var_Set(MAKE_LEVEL, tmp, VAR_GLOBAL);
 	    snprintf(tmp, sizeof(tmp), "%u", myPid);
 	    Var_Set(".MAKE.PID", tmp, VAR_GLOBAL);
 	    snprintf(tmp, sizeof(tmp), "%u", getppid());
 	    Var_Set(".MAKE.PPID", tmp, VAR_GLOBAL);
+	}
 	if (makelevel > 0) {
 		char pn[1024];
 		snprintf(pn, sizeof(pn), "%s[%d]", progname, makelevel);
 		progname = bmake_strdup(pn);
+	}
 #ifdef USE_META
 	meta_init();
 #endif
 	Dir_Init();
 	/*
 	 * First snag any flags out of the MAKE environment variable.
 	 * (Note this is *not* MAKEFLAGS since /bin/make uses that and it's
 	 * in a different format).
 	 */
 #ifdef POSIX
 	p1 = explode(getenv("MAKEFLAGS"));
 	Main_ParseArgLine(p1);
 	free(p1);
 #else
 	Main_ParseArgLine(getenv("MAKE"));
 #endif
 	/*
 	 * Find where we are (now).
 	 * We take care of PWD for the automounter below...
 	 */
 	if (getcwd(curdir, MAXPATHLEN) == NULL) {
 		(void)fprintf(stderr, "%s: getcwd: %s.\n",
 		    progname, strerror(errno));
 		exit(2);
+	}
 	MainParseArgs(argc, argv);
 	if (opts.enterFlag)
 		printf("%s: Entering directory `%s'\n", progname, curdir);
 	/*
 	 * Verify that cwd is sane.
 	 */
 	if (stat(curdir, &sa) == -1) {
 	    (void)fprintf(stderr, "%s: %s: %s.\n",
 		 progname, curdir, strerror(errno));
 	    exit(2);
+	}
 #ifndef NO_PWD_OVERRIDE
 	HandlePWD(&sa);
 #endif
 	Var_Set(".CURDIR", curdir, VAR_GLOBAL);
 	InitObjdir(machine, machine_arch);
 	/*
 	 * Initialize archive, target and suffix modules in preparation for
 	 * parsing the makefile(s)
 	 */
 	Arch_Init();
 	Targ_Init();
 	Suff_Init();
 	Trace_Init(tracefile);
 	DEFAULT = NULL;
 	(void)time(&now);
 	Trace_Log(MAKESTART, NULL);
 	InitVarTargets();
-	InitDefIncPath(syspath);
+	InitDefSysIncPath(syspath);
 	/*
 	 * Read in the built-in rules first, followed by the specified
 	 * makefiles, or the default makefile and Makefile, in that order,
 	 * if no makefiles were given on the command line.
 	 */
 	if (!opts.noBuiltins)
 		ReadBuiltinRules();
 	ReadMakefiles();
 	/* In particular suppress .depend for '-r -V .OBJDIR -f /dev/null' */
 	if (!opts.noBuiltins || !opts.printVars) {
 	    /* ignore /dev/null and anything starting with "no" */
 	    (void)Var_Subst("${.MAKE.DEPENDFILE:N/dev/null:Nno*:T}",
 		VAR_CMD, VARE_WANTRES, &makeDependfile);
 	    if (makeDependfile[0] != '\0') {
 		/* TODO: handle errors */
 		doing_depend = TRUE;
 		(void)ReadMakefile(makeDependfile);
 		doing_depend = FALSE;
+	    }
+	}
 	if (enterFlagObj)
 		printf("%s: Entering directory `%s'\n", progname, objdir);
 	MakeMode(NULL);
 	Var_Append("MFLAGS", Var_Value(MAKEFLAGS, VAR_GLOBAL, &p1), VAR_GLOBAL);
 	bmake_free(p1);
 	InitMaxJobs();
 	/*
 	 * Be compatible if user did not specify -j and did not explicitly
 	 * turned compatibility on
 	 */
 	if (!opts.compatMake && !forceJobs) {
 	    opts.compatMake = TRUE;
+	}
 	if (!opts.compatMake)
 	    Job_ServerStart(maxJobTokens, jp_0, jp_1);
 	DEBUG5(JOB, "job_pipe %d %d, maxjobs %d, tokens %d, compat %d\n",
 	       jp_0, jp_1, opts.maxJobs, maxJobTokens, opts.compatMake ? 1 : 0);
 	if (!opts.printVars)
 	    Main_ExportMAKEFLAGS(TRUE);	/* initial export */
 	InitVpath();
 	/*
 	 * Now that all search paths have been read for suffixes et al, it's
 	 * time to add the default search path to their lists...
 	 */
 	Suff_DoPaths();
 	/*
 	 * Propagate attributes through :: dependency lists.
 	 */
 	Targ_Propagate();
 	/* print the initial graph, if the user requested it */
 	if (DEBUG(GRAPH1))
 		Targ_PrintGraph(1);
 	/* print the values of any variables requested by the user */
 	if (opts.printVars) {
 		doPrintVars();
 		outOfDate = FALSE;
 	} else {
 		outOfDate = runTargets();
+	}
 	CleanUp();
 	return outOfDate ? 1 : 0;
+}
 /* Open and parse the given makefile, with all its side effects.
+ *
  * Results:
  *	0 if ok. -1 if couldn't open file.
  */
 static int
 ReadMakefile(const char *fname)
+{
 	int fd;
 	char *name, *path = NULL;
 	if (!strcmp(fname, "-")) {
 		Parse_File(NULL /*stdin*/, -1);
 		Var_Set("MAKEFILE", "", VAR_INTERNAL);
 	} else {
 		/* if we've chdir'd, rebuild the path name */
 		if (strcmp(curdir, objdir) && *fname != '/') {
 			path = str_concat3(curdir, "/", fname);
 			fd = open(path, O_RDONLY);
 			if (fd != -1) {
 				fname = path;
 				goto found;
+			}
 			free(path);
 			/* If curdir failed, try objdir (ala .depend) */
 			path = str_concat3(objdir, "/", fname);
 			fd = open(path, O_RDONLY);
 			if (fd != -1) {
 				fname = path;
 				goto found;
+			}
 		} else {
 			fd = open(fname, O_RDONLY);
 			if (fd != -1)
 				goto found;
+		}
 		/* look in -I and system include directories. */
 		name = Dir_FindFile(fname, parseIncPath);
-		if (!name)
+		if (!name) {
-			name = Dir_FindFile(fname,
+			SearchPath *sysInc = Lst_IsEmpty(sysIncPath)
-				Lst_IsEmpty(sysIncPath) ? defIncPath : sysIncPath);
+					     ? defSysIncPath : sysIncPath;
 			name = Dir_FindFile(fname, sysInc);
+		}
 		if (!name || (fd = open(name, O_RDONLY)) == -1) {
 			free(name);
 			free(path);
 			return -1;
+		}
 		fname = name;
 		/*
 		 * set the MAKEFILE variable desired by System V fans -- the
 		 * placement of the setting here means it gets set to the last
 		 * makefile specified, as it is set by SysV make.
 		 */
 found:
 		if (!doing_depend)
 			Var_Set("MAKEFILE", fname, VAR_INTERNAL);
 		Parse_File(fname, fd);
+	}
 	free(path);
 	return 0;
+}
 /*-
  * Cmd_Exec --
  *	Execute the command in cmd, and return the output of that command
  *	in a string.  In the output, newlines are replaced with spaces.
+ *
  * Results:
  *	A string containing the output of the command, or the empty string.
  *	*errfmt returns a format string describing the command failure,
  *	if any, using a single %s conversion specification.
+ *
  * Side Effects:
  *	The string must be freed by the caller.
  */
 char *
 Cmd_Exec(const char *cmd, const char **errfmt)
+{
     const char	*args[4];	/* Args for invoking the shell */
     int		fds[2];		/* Pipe streams */
     int		cpid;		/* Child PID */
     int		pid;		/* PID from wait() */
     int		status;		/* command exit status */
     Buffer	buf;		/* buffer to store the result */
     ssize_t	bytes_read;
     char	*res;		/* result */
     size_t	res_len;
     char	*cp;
     int		savederr;	/* saved errno */
     *errfmt = NULL;
     if (!shellName)
 	Shell_Init();
     /*
      * Set up arguments for shell
      */
     args[0] = shellName;
     args[1] = "-c";
     args[2] = cmd;
     args[3] = NULL;
     /*
      * Open a pipe for fetching its output
      */
     if (pipe(fds) == -1) {
 	*errfmt = "Couldn't create pipe for \"%s\"";
 	goto bad;
+    }
     /*
      * Fork
      */
     switch (cpid = vFork()) {
     case 0:
 	/*
 	 * Close input side of pipe
 	 */
 	(void)close(fds[0]);
 	/*
 	 * Duplicate the output stream to the shell's output, then
 	 * shut the extra thing down. Note we don't fetch the error
 	 * stream...why not? Why?
 	 */
 	(void)dup2(fds[1], 1);
 	(void)close(fds[1]);
 	Var_ExportVars();
 	(void)execv(shellPath, UNCONST(args));
 	_exit(1);
 	/*NOTREACHED*/
     case -1:
 	*errfmt = "Couldn't exec \"%s\"";
 	goto bad;
     default:
 	/*
 	 * No need for the writing half
 	 */
 	(void)close(fds[1]);
 	savederr = 0;
 	Buf_Init(&buf, 0);
 	do {
 	    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));
 	if (bytes_read == -1)
 	    savederr = errno;
 	/*
 	 * Close the input side of the pipe.
 	 */
 	(void)close(fds[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);
 	if (savederr != 0)
 	    *errfmt = "Couldn't read shell's output for \"%s\"";
 	if (WIFSIGNALED(status))
 	    *errfmt = "\"%s\" exited on a signal";
 	else if (WEXITSTATUS(status) != 0)
 	    *errfmt = "\"%s\" returned non-zero status";
 	/* Convert newlines to spaces.  A final newline is just stripped */
 	if (res_len > 0 && res[res_len - 1] == '\n')
 	    res[res_len - 1] = '\0';
 	for (cp = res; *cp != '\0'; cp++)
 	    if (*cp == '\n')
 		*cp = ' ';
 	break;
+    }
     return res;
 bad:
     return bmake_strdup("");
+}
 /* Print a printf-style error message.
+ *
  * This error message has no consequences, in particular it does not affect
  * the exit status. */
 void
 Error(const char *fmt, ...)
+{
 	va_list ap;
 	FILE *err_file;
 	err_file = opts.debug_file;
 	if (err_file == stdout)
 		err_file = stderr;
 	(void)fflush(stdout);
 	for (;;) {
 		va_start(ap, fmt);
 		fprintf(err_file, "%s: ", progname);
 		(void)vfprintf(err_file, fmt, ap);
 		va_end(ap);
 		(void)fprintf(err_file, "\n");
 		(void)fflush(err_file);
 		if (err_file == stderr)
 			break;
 		err_file = stderr;
+	}
+}
 /* Produce a Fatal error message, then exit immediately.
+ *
  * If jobs are running, waits for them to finish. */
 void
 Fatal(const char *fmt, ...)
+{
 	va_list ap;
 	va_start(ap, fmt);
 	if (jobsRunning)
 		Job_Wait();
 	(void)fflush(stdout);
 	(void)vfprintf(stderr, fmt, ap);
 	va_end(ap);
 	(void)fprintf(stderr, "\n");
 	(void)fflush(stderr);
 	PrintOnError(NULL, NULL);
 	if (DEBUG(GRAPH2) || DEBUG(GRAPH3))
 		Targ_PrintGraph(2);
 	Trace_Log(MAKEERROR, 0);
 	exit(2);		/* Not 1 so -q can distinguish error */
+}
 /* Major exception once jobs are being created.
  * Kills all jobs, prints a message and exits. */
 void
 Punt(const char *fmt, ...)
+{
 	va_list ap;
 	va_start(ap, fmt);
 	(void)fflush(stdout);
 	(void)fprintf(stderr, "%s: ", progname);
 	(void)vfprintf(stderr, fmt, ap);
 	va_end(ap);
 	(void)fprintf(stderr, "\n");
 	(void)fflush(stderr);
 	PrintOnError(NULL, NULL);
 	DieHorribly();
+}
 /* Exit without giving a message. */
 void
 DieHorribly(void)
+{
 	if (jobsRunning)
 		Job_AbortAll();
 	if (DEBUG(GRAPH2))
 		Targ_PrintGraph(2);
 	Trace_Log(MAKEERROR, 0);
 	exit(2);		/* Not 1, so -q can distinguish error */
+}
 /* Called when aborting due to errors in child shell to signal abnormal exit.
  * The program exits.
  * Errors is the number of errors encountered in Make_Make. */
 void
 Finish(int errors)
+{
 	if (dieQuietly(NULL, -1))
 		exit(2);
 	Fatal("%d error%s", errors, errors == 1 ? "" : "s");
+}
 /*
  * eunlink --
  *	Remove a file carefully, avoiding directories.
  */
 int
 eunlink(const char *file)
+{
 	struct stat st;
 	if (lstat(file, &st) == -1)
 		return -1;
 	if (S_ISDIR(st.st_mode)) {
 		errno = EISDIR;
 		return -1;
+	}
 	return unlink(file);
+}
 static void
 write_all(int fd, const void *data, size_t n)
+{
 	const char *mem = data;
 	while (n > 0) {
 		ssize_t written = write(fd, mem, n);
 		if (written == -1 && errno == EAGAIN)
 			continue;
 		if (written == -1)
 			break;
 		mem += written;
 		n -= (size_t)written;
+	}
+}
 /*
  * execDie --
  *	Print why exec failed, avoiding stdio.
  */
 void MAKE_ATTR_DEAD
 execDie(const char *af, const char *av)
+{
 	Buffer buf;
 	Buf_Init(&buf, 0);
 	Buf_AddStr(&buf, progname);
 	Buf_AddStr(&buf, ": ");
 	Buf_AddStr(&buf, af);
 	Buf_AddStr(&buf, "(");
 	Buf_AddStr(&buf, av);
 	Buf_AddStr(&buf, ") failed (");
 	Buf_AddStr(&buf, strerror(errno));
 	Buf_AddStr(&buf, ")\n");
 	write_all(STDERR_FILENO, Buf_GetAll(&buf, NULL), Buf_Len(&buf));
 	Buf_Destroy(&buf, TRUE);
 	_exit(1);
+}
 /*
  * usage --
  *	exit with usage message
  */
 static void
 usage(void)
+{
 	char *p;
 	if ((p = strchr(progname, '[')) != NULL)
 		*p = '\0';
 	(void)fprintf(stderr,
 "usage: %s [-BeikNnqrstWwX] \n"
 "            [-C directory] [-D variable] [-d flags] [-f makefile]\n"
 "            [-I directory] [-J private] [-j max_jobs] [-m directory] [-T file]\n"
 "            [-V variable] [-v variable] [variable=value] [target ...]\n",
 	    progname);
 	exit(2);
+}
 /*
  * realpath(3) can get expensive, cache results...
  */
 static GNode *cached_realpaths = NULL;
 static GNode *
 get_cached_realpaths(void)
+{
     if (!cached_realpaths) {
 	cached_realpaths = Targ_NewGN("Realpath");
 #ifndef DEBUG_REALPATH_CACHE
 	cached_realpaths->flags = INTERNAL;
 #endif
+    }
     return cached_realpaths;
+}
 /* purge any relative paths */
 static void
 purge_cached_realpaths(void)
+{
     GNode *cache = get_cached_realpaths();
     HashEntry *he, *nhe;
     HashIter hi;
     HashIter_Init(&hi, &cache->context);
     he = HashIter_Next(&hi);
     while (he != NULL) {
 	nhe = HashIter_Next(&hi);
 	if (he->key[0] != '/') {
 	    if (DEBUG(DIR))
 		fprintf(stderr, "cached_realpath: purging %s\n", he->key);
 	    HashTable_DeleteEntry(&cache->context, he);
+	}
 	he = nhe;
+    }
+}
 char *
 cached_realpath(const char *pathname, char *resolved)
+{
     GNode *cache;
     const char *rp;
     char *cp;
     if (!pathname || !pathname[0])
 	return NULL;
     cache = get_cached_realpaths();
     if ((rp = Var_Value(pathname, cache, &cp)) != NULL) {
 	/* a hit */
 	strncpy(resolved, rp, MAXPATHLEN);
 	resolved[MAXPATHLEN - 1] = '\0';
     } else if ((rp = realpath(pathname, resolved)) != NULL) {
 	Var_Set(pathname, rp, cache);
     } /* else should we negative-cache? */
     bmake_free(cp);
     return rp ? resolved : NULL;
+}
 /*
  * Return true if we should die without noise.
  * For example our failing child was a sub-make
  * or failure happend elsewhere.
  */
 int
 dieQuietly(GNode *gn, int bf)
+{
     static int quietly = -1;
     if (quietly < 0) {
 	if (DEBUG(JOB) || getBoolean(".MAKE.DIE_QUIETLY", 1) == 0)
 	    quietly = 0;
 	else if (bf >= 0)
 	    quietly = bf;
 	else
 	    quietly = gn != NULL ? ((gn->type  & (OP_MAKE)) != 0) : 0;
+    }
     return quietly;
+}
 static void
 SetErrorVars(GNode *gn)
+{
     StringListNode *ln;
     /*
      * We can print this even if there is no .ERROR target.
      */
     Var_Set(".ERROR_TARGET", gn->name, VAR_GLOBAL);
     Var_Delete(".ERROR_CMD", VAR_GLOBAL);
     for (ln = gn->commands->first; ln != NULL; ln = ln->next) {
 	const char *cmd = ln->datum;
 	if (cmd == NULL)
 	    break;
 	Var_Append(".ERROR_CMD", cmd, VAR_GLOBAL);
+    }
+}
 void
 PrintOnError(GNode *gn, const char *s)
+{
     static GNode *en = NULL;
     const char *expr;
     char *cp;
     if (DEBUG(HASH)) {
 	Targ_Stats();
 	Var_Stats();
+    }
     /* we generally want to keep quiet if a sub-make died */
     if (dieQuietly(gn, -1))
 	return;
     if (s)
 	printf("%s", s);
     printf("\n%s: stopped in %s\n", progname, curdir);
     if (en)
 	return;				/* we've been here! */
     if (gn)
         SetErrorVars(gn);
     expr = "${MAKE_PRINT_VAR_ON_ERROR:@v@$v='${$v}'\n@}";
     (void)Var_Subst(expr, VAR_GLOBAL, VARE_WANTRES, &cp);
     /* TODO: handle errors */
     printf("%s", cp);
     free(cp);
     fflush(stdout);
     /*
      * Finally, see if there is a .ERROR target, and run it if so.
      */
     en = Targ_FindNode(".ERROR");
     if (en) {
 	en->type |= OP_SPECIAL;
 	Compat_Make(en, en);
+    }
+}
 void
 Main_ExportMAKEFLAGS(Boolean first)
+{
     static Boolean once = TRUE;
     const char *expr;
     char *s;
     if (once != first)
 	return;
     once = FALSE;
     expr = "${.MAKEFLAGS} ${.MAKEOVERRIDES:O:u:@v@$v=${$v:Q}@}";
     (void)Var_Subst(expr, VAR_CMD, VARE_WANTRES, &s);
     /* TODO: handle errors */
     if (s[0] != '\0') {
 #ifdef POSIX
 	setenv("MAKEFLAGS", s, 1);
 #else
 	setenv("MAKE", s, 1);
 #endif
+    }
+}
 char *
 getTmpdir(void)
+{
     static char *tmpdir = NULL;
     if (!tmpdir) {
 	struct stat st;
 	/*
 	 * Honor $TMPDIR but only if it is valid.
 	 * Ensure it ends with /.
 	 */
 	(void)Var_Subst("${TMPDIR:tA:U" _PATH_TMP "}/", VAR_GLOBAL,
 			VARE_WANTRES, &tmpdir);
 	/* TODO: handle errors */
 	if (stat(tmpdir, &st) < 0 || !S_ISDIR(st.st_mode)) {
 	    free(tmpdir);
 	    tmpdir = bmake_strdup(_PATH_TMP);
+	}
+    }
     return tmpdir;
+}
 /*
  * Create and open a temp file using "pattern".
  * If "fnamep" is provided set it to a copy of the filename created.
  * Otherwise unlink the file once open.
  */
 int
 mkTempFile(const char *pattern, char **fnamep)
+{
     static char *tmpdir = NULL;
     char tfile[MAXPATHLEN];
     int fd;
     if (!pattern)
 	pattern = TMPPAT;
     if (!tmpdir)
 	tmpdir = getTmpdir();
     if (pattern[0] == '/') {
 	snprintf(tfile, sizeof(tfile), "%s", pattern);
     } else {
 	snprintf(tfile, sizeof(tfile), "%s%s", tmpdir, pattern);
+    }
     if ((fd = mkstemp(tfile)) < 0)
 	Punt("Could not create temporary file %s: %s", tfile, strerror(errno));
     if (fnamep) {
 	*fnamep = bmake_strdup(tfile);
     } else {
 	unlink(tfile);			/* we just want the descriptor */
+    }
     return fd;
+}
 /*
  * Convert a string representation of a boolean.
  * Anything that looks like "No", "False", "Off", "0" etc,
  * is FALSE, otherwise TRUE.
  */
 Boolean
 s2Boolean(const char *s, Boolean bf)
+{
     if (s) {
 	switch(*s) {
 	case '\0':			/* not set - the default wins */
 	    break;
 	case '0':
 	case 'F':
 	case 'f':
 	case 'N':
 	case 'n':
 	    bf = FALSE;
 	    break;
 	case 'O':
 	case 'o':
 	    switch (s[1]) {
 	    case 'F':
 	    case 'f':
 		bf = FALSE;
 		break;
 	    default:
 		bf = TRUE;
 		break;
+	    }
 	    break;
 	default:
 	    bf = TRUE;
 	    break;
+	}
+    }
     return bf;
+}
 /*
  * Return a Boolean based on setting of a knob.
+ *
  * If the knob is not set, the supplied default is the return value.
  * If set, anything that looks or smells like "No", "False", "Off", "0" etc,
  * is FALSE, otherwise TRUE.
  */
 Boolean
 getBoolean(const char *name, Boolean fallback)
+{
     char *expr = str_concat3("${", name, ":U:tl}");
     char *value;
     Boolean res;
     (void)Var_Subst(expr, VAR_GLOBAL, VARE_WANTRES, &value);
     /* TODO: handle errors */
     res = s2Boolean(value, fallback);
     free(value);
     free(expr);
     return res;
+}

 @@ -1,3201 +1,3201 @@
-/*	$NetBSD: parse.c,v 1.408 2020/10/28 03:12:54 rillig Exp $	*/
+/*	$NetBSD: parse.c,v 1.409 2020/10/28 03:21:25 rillig Exp $	*/
 /*
  * Copyright (c) 1988, 1989, 1990, 1993
  *	The Regents of the University of California.  All rights reserved.
+ *
  * This code is derived from software contributed to Berkeley by
  * Adam de Boor.
+ *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
  * 3. Neither the name of the University nor the names of its contributors
  *    may be used to endorse or promote products derived from this software
  *    without specific prior written permission.
+ *
  * THIS SOFTWARE IS PROVIDED BY THE REGENTS AND CONTRIBUTORS ``AS IS'' AND
  * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
  * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
  * ARE DISCLAIMED.  IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE
  * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
  * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
  * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
  * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
  * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
  * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
  * SUCH DAMAGE.
  */
 /*
  * Copyright (c) 1989 by Berkeley Softworks
  * All rights reserved.
+ *
  * This code is derived from software contributed to Berkeley by
  * Adam de Boor.
+ *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
  * 3. All advertising materials mentioning features or use of this software
  *    must display the following acknowledgement:
  *	This product includes software developed by the University of
  *	California, Berkeley and its contributors.
  * 4. Neither the name of the University nor the names of its contributors
  *    may be used to endorse or promote products derived from this software
  *    without specific prior written permission.
+ *
  * THIS SOFTWARE IS PROVIDED BY THE REGENTS AND CONTRIBUTORS ``AS IS'' AND
  * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
  * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
  * ARE DISCLAIMED.  IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE
  * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
  * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
  * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
  * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
  * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
  * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
  * SUCH DAMAGE.
  */
 /*
  * Parsing of makefiles.
+ *
  * Parse_File is the main entry point and controls most of the other
  * functions in this module.
+ *
  * The directories for the .include "..." directive are kept in
  * 'parseIncPath', while those for .include <...> are kept in 'sysIncPath'.
  * The targets currently being defined are kept in 'targets'.
+ *
  * Interface:
  *	Parse_Init	Initialize the module
+ *
  *	Parse_End	Clean up the module
+ *
  *	Parse_File	Parse a top-level makefile.  Included files are
  *			handled by Parse_include_file though.
+ *
  *	Parse_IsVar	Return TRUE if the given line is a variable
  *			assignment. Used by MainParseArgs to determine if
  *			an argument is a target or a variable assignment.
  *			Used internally for pretty much the same thing.
+ *
  *	Parse_Error	Report a parse error, a warning or an informational
  *			message.
+ *
  *	Parse_MainName	Returns a list of the main target to create.
  */
 #include <sys/types.h>
 #include <sys/mman.h>
 #include <sys/stat.h>
 #include <errno.h>
 #include <stdarg.h>
 #include <stdint.h>
 #ifndef MAP_FILE
 #define MAP_FILE 0
 #endif
 #ifndef MAP_COPY
 #define MAP_COPY MAP_PRIVATE
 #endif
 #include "make.h"
 #include "dir.h"
 #include "job.h"
 #include "pathnames.h"
 /*	"@(#)parse.c	8.3 (Berkeley) 3/19/94"	*/
-MAKE_RCSID("$NetBSD: parse.c,v 1.408 2020/10/28 03:12:54 rillig Exp $");
+MAKE_RCSID("$NetBSD: parse.c,v 1.409 2020/10/28 03:21:25 rillig Exp $");
 /* types and constants */
 /*
  * Structure for a file being read ("included file")
  */
 typedef struct IFile {
     char *fname;		/* name of file */
     Boolean fromForLoop;	/* simulated .include by the .for loop */
     int lineno;			/* current line number in file */
     int first_lineno;		/* line number of start of text */
     unsigned int cond_depth;	/* 'if' nesting when file opened */
     Boolean depending;		/* state of doing_depend on EOF */
     /* The buffer from which the file's content is read. */
     char *buf_freeIt;
     char *buf_ptr;		/* next char to be read */
     char *buf_end;
     char *(*nextbuf)(void *, size_t *); /* Function to get more data */
     void *nextbuf_arg;		/* Opaque arg for nextbuf() */
     struct loadedfile *lf;	/* loadedfile object, if any */
 } IFile;
 /*
  * Tokens for target attributes
  */
 typedef enum ParseSpecial {
     SP_ATTRIBUTE,	/* Generic attribute */
     SP_BEGIN,		/* .BEGIN */
     SP_DEFAULT,		/* .DEFAULT */
     SP_DELETE_ON_ERROR,	/* .DELETE_ON_ERROR */
     SP_END,		/* .END */
     SP_ERROR,		/* .ERROR */
     SP_IGNORE,		/* .IGNORE */
     SP_INCLUDES,	/* .INCLUDES; not mentioned in the manual page */
     SP_INTERRUPT,	/* .INTERRUPT */
     SP_LIBS,		/* .LIBS; not mentioned in the manual page */
     SP_MAIN,		/* .MAIN and we don't have anything user-specified to
 			 * make */
     SP_META,		/* .META */
     SP_MFLAGS,		/* .MFLAGS or .MAKEFLAGS */
     SP_NOMETA,		/* .NOMETA */
     SP_NOMETA_CMP,	/* .NOMETA_CMP */
     SP_NOPATH,		/* .NOPATH */
     SP_NOT,		/* Not special */
     SP_NOTPARALLEL,	/* .NOTPARALLEL or .NO_PARALLEL */
     SP_NULL,		/* .NULL; not mentioned in the manual page */
     SP_OBJDIR,		/* .OBJDIR */
     SP_ORDER,		/* .ORDER */
     SP_PARALLEL,	/* .PARALLEL; not mentioned in the manual page */
     SP_PATH,		/* .PATH or .PATH.suffix */
     SP_PHONY,		/* .PHONY */
 #ifdef POSIX
     SP_POSIX,		/* .POSIX; not mentioned in the manual page */
 #endif
     SP_PRECIOUS,	/* .PRECIOUS */
     SP_SHELL,		/* .SHELL */
     SP_SILENT,		/* .SILENT */
     SP_SINGLESHELL,	/* .SINGLESHELL; not mentioned in the manual page */
     SP_STALE,		/* .STALE */
     SP_SUFFIXES,	/* .SUFFIXES */
     SP_WAIT		/* .WAIT */
 } ParseSpecial;
 typedef List SearchPathList;
 typedef ListNode SearchPathListNode;
 /* result data */
 /*
  * The main target to create. This is the first target on the first
  * dependency line in the first makefile.
  */
 static GNode *mainNode;
 /* eval state */
 /* During parsing, the targets from the currently active dependency line,
  * or NULL if the current line does not belong to a dependency line, for
  * example because it is a variable assignment.
+ *
  * See unit-tests/deptgt.mk, keyword "parse.c:targets". */
 static GNodeList *targets;
 #ifdef CLEANUP
 /* All shell commands for all targets, in no particular order and possibly
  * with duplicates.  Kept in a separate list since the commands from .USE or
  * .USEBEFORE nodes are shared with other GNodes, thereby giving up the
  * easily understandable ownership over the allocated strings. */
 static StringList *targCmds;
 #endif
 /*
  * Predecessor node for handling .ORDER. Initialized to NULL when .ORDER
  * seen, then set to each successive source on the line.
  */
 static GNode *order_pred;
 /* parser state */
 /* number of fatal errors */
 static int fatals = 0;
 /*
  * Variables for doing includes
  */
 /* The include chain of makefiles.  At the bottom is the top-level makefile
  * from the command line, and on top of that, there are the included files or
  * .for loops, up to and including the current file.
+ *
  * This data could be used to print stack traces on parse errors.  As of
  * 2020-09-14, this is not done though.  It seems quite simple to print the
  * tuples (fname:lineno:fromForLoop), from top to bottom.  This simple idea is
  * made complicated by the fact that the .for loops also use this stack for
  * storing information.
+ *
  * The lineno fields of the IFiles with fromForLoop == TRUE look confusing,
  * which is demonstrated by the test 'include-main.mk'.  They seem sorted
  * backwards since they tell the number of completely parsed lines, which for
  * a .for loop is right after the terminating .endfor.  To compensate for this
  * confusion, there is another field first_lineno pointing at the start of the
  * .for loop, 1-based for human consumption.
+ *
  * To make the stack trace intuitive, the entry below the first .for loop must
  * be ignored completely since neither its lineno nor its first_lineno is
  * useful.  Instead, the topmost of each chain of .for loop needs to be
  * printed twice, once with its first_lineno and once with its lineno.
+ *
  * As of 2020-10-28, using the above rules, the stack trace for the .info line
  * in include-subsub.mk would be:
+ *
  *	includes[5]:	include-subsub.mk:4
  *			(lineno, from an .include)
  *	includes[4]:	include-sub.mk:32
  *			(lineno, from a .for loop below an .include)
  *	includes[4]:	include-sub.mk:31
  *			(first_lineno, from a .for loop, lineno == 32)
  *	includes[3]:	include-sub.mk:30
  *			(first_lineno, from a .for loop, lineno == 33)
  *	includes[2]:	include-sub.mk:29
  *			(first_lineno, from a .for loop, lineno == 34)
  *	includes[1]:	include-sub.mk:35
  *			(not printed since it is below a .for loop)
  *	includes[0]:	include-main.mk:27
  */
 static Vector /* of IFile */ includes;
 static IFile *
 GetInclude(size_t i)
+{
     return Vector_Get(&includes, i);
+}
 /* The file that is currently being read. */
 static IFile *
 CurFile(void)
+{
     return GetInclude(includes.len - 1);
+}
-/* include paths (lists of directories) */
+/* include paths */
 SearchPath *parseIncPath;	/* dirs for "..." includes */
 SearchPath *sysIncPath;		/* dirs for <...> includes */
-SearchPath *defIncPath;		/* default for sysIncPath */
+SearchPath *defSysIncPath;		/* default for sysIncPath */
 /* parser tables */
 /*
  * The parseKeywords table is searched using binary search when deciding
  * if a target or source is special. The 'spec' field is the ParseSpecial
  * type of the keyword (SP_NOT if the keyword isn't special as a target) while
  * the 'op' field is the operator to apply to the list of targets if the
  * keyword is used as a source ("0" if the keyword isn't special as a source)
  */
 static const struct {
     const char   *name;		/* Name of keyword */
     ParseSpecial  spec;		/* Type when used as a target */
     GNodeType	  op;		/* Operator when used as a source */
 } parseKeywords[] = {
     { ".BEGIN",		SP_BEGIN,	0 },
     { ".DEFAULT",	SP_DEFAULT,	0 },
     { ".DELETE_ON_ERROR", SP_DELETE_ON_ERROR, 0 },
     { ".END",		SP_END,		0 },
     { ".ERROR",		SP_ERROR,	0 },
     { ".EXEC",		SP_ATTRIBUTE,	OP_EXEC },
     { ".IGNORE",	SP_IGNORE,	OP_IGNORE },
     { ".INCLUDES",	SP_INCLUDES,	0 },
     { ".INTERRUPT",	SP_INTERRUPT,	0 },
     { ".INVISIBLE",	SP_ATTRIBUTE,	OP_INVISIBLE },
     { ".JOIN",		SP_ATTRIBUTE,	OP_JOIN },
     { ".LIBS",		SP_LIBS,	0 },
     { ".MADE",		SP_ATTRIBUTE,	OP_MADE },
     { ".MAIN",		SP_MAIN,	0 },
     { ".MAKE",		SP_ATTRIBUTE,	OP_MAKE },
     { ".MAKEFLAGS",	SP_MFLAGS,	0 },
     { ".META",		SP_META,	OP_META },
     { ".MFLAGS",	SP_MFLAGS,	0 },
     { ".NOMETA",	SP_NOMETA,	OP_NOMETA },
     { ".NOMETA_CMP",	SP_NOMETA_CMP,	OP_NOMETA_CMP },
     { ".NOPATH",	SP_NOPATH,	OP_NOPATH },
     { ".NOTMAIN",	SP_ATTRIBUTE,	OP_NOTMAIN },
     { ".NOTPARALLEL",	SP_NOTPARALLEL,	0 },
     { ".NO_PARALLEL",	SP_NOTPARALLEL,	0 },
     { ".NULL",		SP_NULL,	0 },
     { ".OBJDIR",	SP_OBJDIR,	0 },
     { ".OPTIONAL",	SP_ATTRIBUTE,	OP_OPTIONAL },
     { ".ORDER",		SP_ORDER,	0 },
     { ".PARALLEL",	SP_PARALLEL,	0 },
     { ".PATH",		SP_PATH,	0 },
     { ".PHONY",		SP_PHONY,	OP_PHONY },
 #ifdef POSIX
     { ".POSIX",		SP_POSIX,	0 },
 #endif
     { ".PRECIOUS",	SP_PRECIOUS,	OP_PRECIOUS },
     { ".RECURSIVE",	SP_ATTRIBUTE,	OP_MAKE },
     { ".SHELL",		SP_SHELL,	0 },
     { ".SILENT",	SP_SILENT,	OP_SILENT },
     { ".SINGLESHELL",	SP_SINGLESHELL,	0 },
     { ".STALE",		SP_STALE,	0 },
     { ".SUFFIXES",	SP_SUFFIXES,	0 },
     { ".USE",		SP_ATTRIBUTE,	OP_USE },
     { ".USEBEFORE",	SP_ATTRIBUTE,	OP_USEBEFORE },
     { ".WAIT",		SP_WAIT,	0 },
 };
 /* file loader */
 struct loadedfile {
 	const char *path;		/* name, for error reports */
 	char *buf;			/* contents buffer */
 	size_t len;			/* length of contents */
 	size_t maplen;			/* length of mmap area, or 0 */
 	Boolean used;			/* XXX: have we used the data yet */
 };
 static struct loadedfile *
 loadedfile_create(const char *path)
+{
 	struct loadedfile *lf;
 	lf = bmake_malloc(sizeof(*lf));
 	lf->path = path == NULL ? "(stdin)" : path;
 	lf->buf = NULL;
 	lf->len = 0;
 	lf->maplen = 0;
 	lf->used = FALSE;
 	return lf;
+}
 static void
 loadedfile_destroy(struct loadedfile *lf)
+{
 	if (lf->buf != NULL) {
 		if (lf->maplen > 0) {
 			munmap(lf->buf, lf->maplen);
 		} else {
 			free(lf->buf);
+		}
+	}
 	free(lf);
+}
 /*
  * nextbuf() operation for loadedfile, as needed by the weird and twisted
  * logic below. Once that's cleaned up, we can get rid of lf->used...
  */
 static char *
 loadedfile_nextbuf(void *x, size_t *len)
+{
 	struct loadedfile *lf = x;
 	if (lf->used) {
 		return NULL;
+	}
 	lf->used = TRUE;
 	*len = lf->len;
 	return lf->buf;
+}
 /*
  * Try to get the size of a file.
  */
 static Boolean
 load_getsize(int fd, size_t *ret)
+{
 	struct stat st;
 	if (fstat(fd, &st) < 0) {
 		return FALSE;
+	}
 	if (!S_ISREG(st.st_mode)) {
 		return FALSE;
+	}
 	/*
 	 * st_size is an off_t, which is 64 bits signed; *ret is
 	 * size_t, which might be 32 bits unsigned or 64 bits
 	 * unsigned. Rather than being elaborate, just punt on
 	 * files that are more than 2^31 bytes. We should never
 	 * see a makefile that size in practice...
+	 *
 	 * While we're at it reject negative sizes too, just in case.
 	 */
 	if (st.st_size < 0 || st.st_size > 0x7fffffff) {
 		return FALSE;
+	}
 	*ret = (size_t)st.st_size;
 	return TRUE;
+}
 /*
  * Read in a file.
+ *
  * Until the path search logic can be moved under here instead of
  * being in the caller in another source file, we need to have the fd
  * passed in already open. Bleh.
+ *
  * If the path is NULL use stdin and (to insure against fd leaks)
  * assert that the caller passed in -1.
  */
 static struct loadedfile *
 loadfile(const char *path, int fd)
+{
 	struct loadedfile *lf;
 	static unsigned long pagesize = 0;
 	ssize_t result;
 	size_t bufpos;
 	lf = loadedfile_create(path);
 	if (path == NULL) {
 		assert(fd == -1);
 		fd = STDIN_FILENO;
 	} else {
 #if 0 /* notyet */
 		fd = open(path, O_RDONLY);
 		if (fd < 0) {
 			...
 			Error("%s: %s", path, strerror(errno));
 			exit(1);
+		}
 #endif
+	}
 	if (load_getsize(fd, &lf->len)) {
 		/* found a size, try mmap */
 		if (pagesize == 0)
 			pagesize = (unsigned long)sysconf(_SC_PAGESIZE);
 		if (pagesize == 0 || pagesize == (unsigned long)-1) {
 			pagesize = 0x1000;
+		}
 		/* round size up to a page */
 		lf->maplen = pagesize * ((lf->len + pagesize - 1)/pagesize);
 		/*
 		 * XXX hack for dealing with empty files; remove when
 		 * we're no longer limited by interfacing to the old
 		 * logic elsewhere in this file.
 		 */
 		if (lf->maplen == 0) {
 			lf->maplen = pagesize;
+		}
 		/*
 		 * FUTURE: remove PROT_WRITE when the parser no longer
 		 * needs to scribble on the input.
 		 */
 		lf->buf = mmap(NULL, lf->maplen, PROT_READ|PROT_WRITE,
 			       MAP_FILE|MAP_COPY, fd, 0);
 		if (lf->buf != MAP_FAILED) {
 			/* succeeded */
 			if (lf->len == lf->maplen && lf->buf[lf->len - 1] != '\n') {
 				char *b = bmake_malloc(lf->len + 1);
 				b[lf->len] = '\n';
 				memcpy(b, lf->buf, lf->len++);
 				munmap(lf->buf, lf->maplen);
 				lf->maplen = 0;
 				lf->buf = b;
+			}
 			goto done;
+		}
+	}
 	/* cannot mmap; load the traditional way */
 	lf->maplen = 0;
 	lf->len = 1024;
 	lf->buf = bmake_malloc(lf->len);
 	bufpos = 0;
 	while (1) {
 		assert(bufpos <= lf->len);
 		if (bufpos == lf->len) {
 			if (lf->len > SIZE_MAX/2) {
 				errno = EFBIG;
 				Error("%s: file too large", path);
 				exit(1);
+			}
 			lf->len *= 2;
 			lf->buf = bmake_realloc(lf->buf, lf->len);
+		}
 		assert(bufpos < lf->len);
 		result = read(fd, lf->buf + bufpos, lf->len - bufpos);
 		if (result < 0) {
 			Error("%s: read error: %s", path, strerror(errno));
 			exit(1);
+		}
 		if (result == 0) {
 			break;
+		}
 		bufpos += (size_t)result;
+	}
 	assert(bufpos <= lf->len);
 	lf->len = bufpos;
 	/* truncate malloc region to actual length (maybe not useful) */
 	if (lf->len > 0) {
 		/* as for mmap case, ensure trailing \n */
 		if (lf->buf[lf->len - 1] != '\n')
 			lf->len++;
 		lf->buf = bmake_realloc(lf->buf, lf->len);
 		lf->buf[lf->len - 1] = '\n';
+	}
 done:
 	if (path != NULL) {
 		close(fd);
+	}
 	return lf;
+}
 /* old code */
 /* Check if the current character is escaped on the current line. */
 static Boolean
 ParseIsEscaped(const char *line, const char *c)
+{
     Boolean active = FALSE;
     for (;;) {
 	if (line == c)
 	    return active;
 	if (*--c != '\\')
 	    return active;
 	active = !active;
+    }
+}
 /* Add the filename and lineno to the GNode so that we remember where it
  * was first defined. */
 static void
 ParseMark(GNode *gn)
+{
     IFile *curFile = CurFile();
     gn->fname = curFile->fname;
     gn->lineno = curFile->lineno;
+}
 /* Look in the table of keywords for one matching the given string.
  * Return the index of the keyword, or -1 if it isn't there. */
 static int
 ParseFindKeyword(const char *str)
+{
     int start, end, cur;
     int diff;
     start = 0;
     end = sizeof parseKeywords / sizeof parseKeywords[0] - 1;
     do {
 	cur = start + (end - start) / 2;
 	diff = strcmp(str, parseKeywords[cur].name);
 	if (diff == 0) {
 	    return cur;
 	} else if (diff < 0) {
 	    end = cur - 1;
 	} else {
 	    start = cur + 1;
+	}
     } while (start <= end);
     return -1;
+}
 static void
 PrintLocation(FILE *f, const char *filename, size_t lineno)
+{
 	char dirbuf[MAXPATHLEN+1];
 	const char *dir, *base;
 	char *dir_freeIt, *base_freeIt;
 	if (*filename == '/' || strcmp(filename, "(stdin)") == 0) {
 		(void)fprintf(f, "\"%s\" line %zu: ", filename, lineno);
 		return;
+	}
 	/* Find out which makefile is the culprit.
 	 * We try ${.PARSEDIR} and apply realpath(3) if not absolute. */
 	dir = Var_Value(".PARSEDIR", VAR_GLOBAL, &dir_freeIt);
 	if (dir == NULL)
 		dir = ".";
 	if (*dir != '/')
 		dir = realpath(dir, dirbuf);
 	base = Var_Value(".PARSEFILE", VAR_GLOBAL, &base_freeIt);
 	if (base == NULL) {
 		const char *slash = strrchr(filename, '/');
 		base = slash != NULL ? slash + 1 : filename;
+	}
 	(void)fprintf(f, "\"%s/%s\" line %zu: ", dir, base, lineno);
 	bmake_free(base_freeIt);
 	bmake_free(dir_freeIt);
+}
 /* Print a parse error message, including location information.
+ *
  * Increment "fatals" if the level is PARSE_FATAL, and continue parsing
  * until the end of the current top-level makefile, then exit (see
  * Parse_File). */
 static void
 ParseVErrorInternal(FILE *f, const char *cfname, size_t clineno,
 		    ParseErrorLevel type, const char *fmt, va_list ap)
+{
 	static Boolean fatal_warning_error_printed = FALSE;
 	(void)fprintf(f, "%s: ", progname);
 	if (cfname != NULL)
 		PrintLocation(f, cfname, clineno);
 	if (type == PARSE_WARNING)
 		(void)fprintf(f, "warning: ");
 	(void)vfprintf(f, fmt, ap);
 	(void)fprintf(f, "\n");
 	(void)fflush(f);
 	if (type == PARSE_INFO)
 		return;
 	if (type == PARSE_FATAL || opts.parseWarnFatal)
 		fatals++;
 	if (opts.parseWarnFatal && !fatal_warning_error_printed) {
 		Error("parsing warnings being treated as errors");
 		fatal_warning_error_printed = TRUE;
+	}
+}
 static void
 ParseErrorInternal(const char *cfname, size_t clineno, ParseErrorLevel type,
 		   const char *fmt, ...)
+{
 	va_list ap;
 	va_start(ap, fmt);
 	(void)fflush(stdout);
 	ParseVErrorInternal(stderr, cfname, clineno, type, fmt, ap);
 	va_end(ap);
 	if (opts.debug_file != stderr && opts.debug_file != stdout) {
 		va_start(ap, fmt);
 		ParseVErrorInternal(opts.debug_file, cfname, clineno, type,
 				    fmt, ap);
 		va_end(ap);
+	}
+}
 /* External interface to ParseErrorInternal; uses the default filename and
  * line number.
+ *
  * Fmt is given without a trailing newline. */
 void
 Parse_Error(ParseErrorLevel type, const char *fmt, ...)
+{
 	va_list ap;
 	const char *fname;
 	size_t lineno;
 	if (includes.len == 0) {
 		fname = NULL;
 		lineno = 0;
 	} else {
 		IFile *curFile = CurFile();
 		fname = curFile->fname;
 		lineno = (size_t)curFile->lineno;
+	}
 	va_start(ap, fmt);
 	(void)fflush(stdout);
 	ParseVErrorInternal(stderr, fname, lineno, type, fmt, ap);
 	va_end(ap);
 	if (opts.debug_file != stderr && opts.debug_file != stdout) {
 		va_start(ap, fmt);
 		ParseVErrorInternal(opts.debug_file, fname, lineno, type,
 				    fmt, ap);
 		va_end(ap);
+	}
+}
 /* Parse a .info .warning or .error directive.
+ *
  * The input is the line minus the ".".  We substitute variables, print the
  * message and exit(1) (for .error) or just print a warning if the directive
  * is malformed.
  */
 static Boolean
 ParseMessage(const char *directive)
+{
     const char *p = directive;
     int mtype = *p == 'i' ? PARSE_INFO :
 		*p == 'w' ? PARSE_WARNING : PARSE_FATAL;
     char *arg;
     while (ch_isalpha(*p))
 	p++;
     if (!ch_isspace(*p))
 	return FALSE;		/* missing argument */
     cpp_skip_whitespace(&p);
     (void)Var_Subst(p, VAR_CMD, VARE_WANTRES, &arg);
     /* TODO: handle errors */
     Parse_Error(mtype, "%s", arg);
     free(arg);
     if (mtype == PARSE_FATAL) {
 	PrintOnError(NULL, NULL);
 	exit(1);
+    }
     return TRUE;
+}
 /* Add the child to the parent's children.
+ *
  * Additionally, add the parent to the child's parents, but only if the
  * target is not special.  An example for such a special target is .END,
  * which does not need to be informed once the child target has been made. */
 static void
 LinkSource(GNode *pgn, GNode *cgn, Boolean isSpecial)
+{
     if ((pgn->type & OP_DOUBLEDEP) && !Lst_IsEmpty(pgn->cohorts))
 	pgn = pgn->cohorts->last->datum;
     Lst_Append(pgn->children, cgn);
     pgn->unmade++;
     /* Special targets like .END don't need any children. */
     if (!isSpecial)
 	Lst_Append(cgn->parents, pgn);
     if (DEBUG(PARSE)) {
 	debug_printf("# %s: added child %s - %s\n",
 		     __func__, pgn->name, cgn->name);
 	Targ_PrintNode(pgn, 0);
 	Targ_PrintNode(cgn, 0);
+    }
+}
 /* Add the node to each target from the current dependency group. */
 static void
 LinkToTargets(GNode *gn, Boolean isSpecial)
+{
     GNodeListNode *ln;
     for (ln = targets->first; ln != NULL; ln = ln->next)
 	LinkSource(ln->datum, gn, isSpecial);
+}
 static Boolean
 TryApplyDependencyOperator(GNode *gn, GNodeType op)
+{
     /*
      * If the node occurred on the left-hand side of a dependency and the
      * operator also defines a dependency, they must match.
      */
     if ((op & OP_OPMASK) && (gn->type & OP_OPMASK) &&
 	((op & OP_OPMASK) != (gn->type & OP_OPMASK)))
+    {
 	Parse_Error(PARSE_FATAL, "Inconsistent operator for %s", gn->name);
 	return FALSE;
+    }
     if (op == OP_DOUBLEDEP && (gn->type & OP_OPMASK) == OP_DOUBLEDEP) {
 	/*
 	 * If the node was the object of a :: operator, we need to create a
 	 * new instance of it for the children and commands on this dependency
 	 * line. The new instance is placed on the 'cohorts' list of the
 	 * initial one (note the initial one is not on its own cohorts list)
 	 * and the new instance is linked to all parents of the initial
 	 * instance.
 	 */
 	GNode *cohort;
 	/*
 	 * Propagate copied bits to the initial node.  They'll be propagated
 	 * back to the rest of the cohorts later.
 	 */
 	gn->type |= op & ~OP_OPMASK;
 	cohort = Targ_NewInternalNode(gn->name);
 	if (doing_depend)
 	    ParseMark(cohort);
 	/*
 	 * Make the cohort invisible as well to avoid duplicating it into
 	 * other variables. True, parents of this target won't tend to do
 	 * anything with their local variables, but better safe than
 	 * sorry. (I think this is pointless now, since the relevant list
 	 * traversals will no longer see this node anyway. -mycroft)
 	 */
 	cohort->type = op | OP_INVISIBLE;
 	Lst_Append(gn->cohorts, cohort);
 	cohort->centurion = gn;
 	gn->unmade_cohorts++;
 	snprintf(cohort->cohort_num, sizeof cohort->cohort_num, "#%d",
 		 (unsigned int)gn->unmade_cohorts % 1000000);
     } else {
 	/*
 	 * We don't want to nuke any previous flags (whatever they were) so we
 	 * just OR the new operator into the old
 	 */
 	gn->type |= op;
+    }
     return TRUE;
+}
 static void
 ApplyDependencyOperator(GNodeType op)
+{
     GNodeListNode *ln;
     for (ln = targets->first; ln != NULL; ln = ln->next)
 	if (!TryApplyDependencyOperator(ln->datum, op))
 	    break;
+}
 static Boolean
 ParseDoSrcKeyword(const char *src, ParseSpecial specType)
+{
     static int wait_number = 0;
     char wait_src[16];
     GNode *gn;
     if (*src == '.' && ch_isupper(src[1])) {
 	int keywd = ParseFindKeyword(src);
 	if (keywd != -1) {
 	    int op = parseKeywords[keywd].op;
 	    if (op != 0) {
 		ApplyDependencyOperator(op);
 		return TRUE;
+	    }
 	    if (parseKeywords[keywd].spec == SP_WAIT) {
 		/*
 		 * We add a .WAIT node in the dependency list.
 		 * After any dynamic dependencies (and filename globbing)
 		 * have happened, it is given a dependency on the each
 		 * previous child back to and previous .WAIT node.
 		 * The next child won't be scheduled until the .WAIT node
 		 * is built.
 		 * We give each .WAIT node a unique name (mainly for diag).
 		 */
 		snprintf(wait_src, sizeof wait_src, ".WAIT_%u", ++wait_number);
 		gn = Targ_NewInternalNode(wait_src);
 		if (doing_depend)
 		    ParseMark(gn);
 		gn->type = OP_WAIT | OP_PHONY | OP_DEPENDS | OP_NOTMAIN;
 		LinkToTargets(gn, specType != SP_NOT);
 		return TRUE;
+	    }
+	}
+    }
     return FALSE;
+}
 static void
 ParseDoSrcMain(const char *src)
+{
     /*
      * If we have noted the existence of a .MAIN, it means we need
      * to add the sources of said target to the list of things
      * to create. The string 'src' is likely to be free, so we
      * must make a new copy of it. Note that this will only be
      * invoked if the user didn't specify a target on the command
      * line. This is to allow #ifmake's to succeed, or something...
      */
     Lst_Append(opts.create, bmake_strdup(src));
     /*
      * Add the name to the .TARGETS variable as well, so the user can
      * employ that, if desired.
      */
     Var_Append(".TARGETS", src, VAR_GLOBAL);
+}
 static void
 ParseDoSrcOrder(const char *src)
+{
     GNode *gn;
     /*
      * Create proper predecessor/successor links between the previous
      * source and the current one.
      */
     gn = Targ_GetNode(src);
     if (doing_depend)
 	ParseMark(gn);
     if (order_pred != NULL) {
 	Lst_Append(order_pred->order_succ, gn);
 	Lst_Append(gn->order_pred, order_pred);
 	if (DEBUG(PARSE)) {
 	    debug_printf("# %s: added Order dependency %s - %s\n",
 			 __func__, order_pred->name, gn->name);
 	    Targ_PrintNode(order_pred, 0);
 	    Targ_PrintNode(gn, 0);
+	}
+    }
     /*
      * The current source now becomes the predecessor for the next one.
      */
     order_pred = gn;
+}
 static void
 ParseDoSrcOther(const char *src, GNodeType tOp, ParseSpecial specType)
+{
     GNode *gn;
     /*
      * If the source is not an attribute, we need to find/create
      * a node for it. After that we can apply any operator to it
      * from a special target or link it to its parents, as
      * appropriate.
+     *
      * In the case of a source that was the object of a :: operator,
      * the attribute is applied to all of its instances (as kept in
      * the 'cohorts' list of the node) or all the cohorts are linked
      * to all the targets.
      */
     /* Find/create the 'src' node and attach to all targets */
     gn = Targ_GetNode(src);
     if (doing_depend)
 	ParseMark(gn);
     if (tOp) {
 	gn->type |= tOp;
     } else {
 	LinkToTargets(gn, specType != SP_NOT);
+    }
+}
 /* Given the name of a source in a dependency line, figure out if it is an
  * attribute (such as .SILENT) and apply it to the targets if it is. Else
  * decide if there is some attribute which should be applied *to* the source
  * because of some special target (such as .PHONY) and apply it if so.
  * Otherwise, make the source a child of the targets in the list 'targets'.
+ *
  * Input:
  *	tOp		operator (if any) from special targets
  *	src		name of the source to handle
  */
 static void
 ParseDoSrc(GNodeType tOp, const char *src, ParseSpecial specType)
+{
     if (ParseDoSrcKeyword(src, specType))
 	return;
     if (specType == SP_MAIN)
 	ParseDoSrcMain(src);
     else if (specType == SP_ORDER)
 	ParseDoSrcOrder(src);
     else
 	ParseDoSrcOther(src, tOp, specType);
+}
 /* If we have yet to decide on a main target to make, in the absence of any
  * user input, we want the first target on the first dependency line that is
  * actually a real target (i.e. isn't a .USE or .EXEC rule) to be made. */
 static void
 FindMainTarget(void)
+{
     GNodeListNode *ln;
     if (mainNode != NULL)
 	return;
     for (ln = targets->first; ln != NULL; ln = ln->next) {
 	GNode *gn = ln->datum;
 	if (!(gn->type & OP_NOTARGET)) {
 	    mainNode = gn;
 	    Targ_SetMain(gn);
 	    return;
+	}
+    }
+}
 /*
  * We got to the end of the line while we were still looking at targets.
+ *
  * Ending a dependency line without an operator is a Bozo no-no.  As a
  * heuristic, this is also often triggered by undetected conflicts from
  * cvs/rcs merges.
  */
 static void
 ParseErrorNoDependency(const char *lstart)
+{
     if ((strncmp(lstart, "<<<<<<", 6) == 0) ||
 	(strncmp(lstart, "======", 6) == 0) ||
 	(strncmp(lstart, ">>>>>>", 6) == 0))
 	Parse_Error(PARSE_FATAL,
 		    "Makefile appears to contain unresolved cvs/rcs/??? merge conflicts");
     else if (lstart[0] == '.') {
 	const char *dirstart = lstart + 1;
 	const char *dirend;
 	cpp_skip_whitespace(&dirstart);
 	dirend = dirstart;
 	while (ch_isalnum(*dirend) || *dirend == '-')
 	    dirend++;
 	Parse_Error(PARSE_FATAL, "Unknown directive \"%.*s\"",
 		    (int)(dirend - dirstart), dirstart);
     } else
 	Parse_Error(PARSE_FATAL, "Need an operator");
+}
 static void
 ParseDependencyTargetWord(/*const*/ char **pp, const char *lstart)
+{
     /*const*/ char *cp = *pp;
     while (*cp != '\0') {
 	if ((ch_isspace(*cp) || *cp == '!' || *cp == ':' || *cp == '(') &&
 	    !ParseIsEscaped(lstart, cp))
 	    break;
 	if (*cp == '$') {
 	    /*
 	     * Must be a dynamic source (would have been expanded
 	     * otherwise), so call the Var module to parse the puppy
 	     * so we can safely advance beyond it...There should be
 	     * no errors in this, as they would have been discovered
 	     * in the initial Var_Subst and we wouldn't be here.
 	     */
 	    const char *nested_p = cp;
 	    const char *nested_val;
 	    void *freeIt;
 	    (void)Var_Parse(&nested_p, VAR_CMD, VARE_UNDEFERR|VARE_WANTRES,
 			    &nested_val, &freeIt);
 	    /* TODO: handle errors */
 	    free(freeIt);
 	    cp += nested_p - cp;
 	} else
 	    cp++;
+    }
     *pp = cp;
+}
 /*
  * Certain special targets have special semantics:
  *	.PATH		Have to set the dirSearchPath
  *			variable too
  *	.MAIN		Its sources are only used if
  *			nothing has been specified to
  *			create.
  *	.DEFAULT	Need to create a node to hang
  *			commands on, but we don't want
  *			it in the graph, nor do we want
  *			it to be the Main Target, so we
  *			create it, set OP_NOTMAIN and
  *			add it to the list, setting
  *			DEFAULT to the new node for
  *			later use. We claim the node is
  *			A transformation rule to make
  *			life easier later, when we'll
  *			use Make_HandleUse to actually
  *			apply the .DEFAULT commands.
  *	.PHONY		The list of targets
  *	.NOPATH		Don't search for file in the path
  *	.STALE
  *	.BEGIN
  *	.END
  *	.ERROR
  *	.DELETE_ON_ERROR
  *	.INTERRUPT	Are not to be considered the
  *			main target.
  *	.NOTPARALLEL	Make only one target at a time.
  *	.SINGLESHELL	Create a shell for each command.
  *	.ORDER		Must set initial predecessor to NULL
  */
 static void
 ParseDoDependencyTargetSpecial(ParseSpecial *inout_specType,
 			       const char *line,
 			       SearchPathList **inout_paths)
+{
     switch (*inout_specType) {
     case SP_PATH:
 	if (*inout_paths == NULL) {
 	    *inout_paths = Lst_New();
+	}
 	Lst_Append(*inout_paths, dirSearchPath);
 	break;
     case SP_MAIN:
 	if (!Lst_IsEmpty(opts.create)) {
 	    *inout_specType = SP_NOT;
+	}
 	break;
     case SP_BEGIN:
     case SP_END:
     case SP_STALE:
     case SP_ERROR:
     case SP_INTERRUPT: {
 	GNode *gn = Targ_GetNode(line);
 	if (doing_depend)
 	    ParseMark(gn);
 	gn->type |= OP_NOTMAIN|OP_SPECIAL;
 	Lst_Append(targets, gn);
 	break;
+    }
     case SP_DEFAULT: {
 	GNode *gn = Targ_NewGN(".DEFAULT");
 	gn->type |= OP_NOTMAIN|OP_TRANSFORM;
 	Lst_Append(targets, gn);
 	DEFAULT = gn;
 	break;
+    }
     case SP_DELETE_ON_ERROR:
 	deleteOnError = TRUE;
 	break;
     case SP_NOTPARALLEL:
 	opts.maxJobs = 1;
 	break;
     case SP_SINGLESHELL:
 	opts.compatMake = TRUE;
 	break;
     case SP_ORDER:
 	order_pred = NULL;
 	break;
     default:
 	break;
+    }
+}
 /*
  * .PATH<suffix> has to be handled specially.
  * Call on the suffix module to give us a path to modify.
  */
 static Boolean
 ParseDoDependencyTargetPath(const char *line, SearchPathList **inout_paths)
+{
     SearchPath *path;
     path = Suff_GetPath(&line[5]);
     if (path == NULL) {
 	Parse_Error(PARSE_FATAL,
 		    "Suffix '%s' not defined (yet)",
 		    &line[5]);
 	return FALSE;
     } else {
 	if (*inout_paths == NULL) {
 	    *inout_paths = Lst_New();
+	}
 	Lst_Append(*inout_paths, path);
+    }
     return TRUE;
+}
 /*
  * See if it's a special target and if so set specType to match it.
  */
 static Boolean
 ParseDoDependencyTarget(const char *line, ParseSpecial *inout_specType,
 			GNodeType *out_tOp, SearchPathList **inout_paths)
+{
     int keywd;
     if (!(*line == '.' && ch_isupper(line[1])))
 	return TRUE;
     /*
      * See if the target is a special target that must have it
      * or its sources handled specially.
      */
     keywd = ParseFindKeyword(line);
     if (keywd != -1) {
 	if (*inout_specType == SP_PATH && parseKeywords[keywd].spec != SP_PATH) {
 	    Parse_Error(PARSE_FATAL, "Mismatched special targets");
 	    return FALSE;
+	}
 	*inout_specType = parseKeywords[keywd].spec;
 	*out_tOp = parseKeywords[keywd].op;
 	ParseDoDependencyTargetSpecial(inout_specType, line, inout_paths);
     } else if (strncmp(line, ".PATH", 5) == 0) {
 	*inout_specType = SP_PATH;
 	if (!ParseDoDependencyTargetPath(line, inout_paths))
 	    return FALSE;
+    }
     return TRUE;
+}
 static void
 ParseDoDependencyTargetMundane(char *line, StringList *curTargs)
+{
     if (Dir_HasWildcards(line)) {
 	/*
 	 * Targets are to be sought only in the current directory,
 	 * so create an empty path for the thing. Note we need to
 	 * use Dir_Destroy in the destruction of the path as the
 	 * Dir module could have added a directory to the path...
 	 */
 	SearchPath *emptyPath = Lst_New();
 	Dir_Expand(line, emptyPath, curTargs);
 	Lst_Destroy(emptyPath, Dir_Destroy);
     } else {
 	/*
 	 * No wildcards, but we want to avoid code duplication,
 	 * so create a list with the word on it.
 	 */
 	Lst_Append(curTargs, line);
+    }
     /* Apply the targets. */
     while (!Lst_IsEmpty(curTargs)) {
 	char *targName = Lst_Dequeue(curTargs);
 	GNode *gn = Suff_IsTransform(targName)
 		    ? Suff_AddTransform(targName)
 		    : Targ_GetNode(targName);
 	if (doing_depend)
 	    ParseMark(gn);
 	Lst_Append(targets, gn);
+    }
+}
 static void
 ParseDoDependencyTargetExtraWarn(char **pp, const char *lstart)
+{
     Boolean warning = FALSE;
     char *cp = *pp;
     while (*cp && (ParseIsEscaped(lstart, cp) ||
 		   (*cp != '!' && *cp != ':'))) {
 	if (ParseIsEscaped(lstart, cp) ||
 	    (*cp != ' ' && *cp != '\t')) {
 	    warning = TRUE;
+	}
 	cp++;
+    }
     if (warning) {
 	Parse_Error(PARSE_WARNING, "Extra target ignored");
+    }
     *pp = cp;
+}
 static void
 ParseDoDependencyCheckSpec(ParseSpecial specType)
+{
     switch (specType) {
     default:
 	Parse_Error(PARSE_WARNING,
 		    "Special and mundane targets don't mix. Mundane ones ignored");
 	break;
     case SP_DEFAULT:
     case SP_STALE:
     case SP_BEGIN:
     case SP_END:
     case SP_ERROR:
     case SP_INTERRUPT:
 	/*
 	 * These four create nodes on which to hang commands, so
 	 * targets shouldn't be empty...
 	 */
     case SP_NOT:
 	/*
 	 * Nothing special here -- targets can be empty if it wants.
 	 */
 	break;
+    }
+}
 static Boolean
 ParseDoDependencyParseOp(char **pp, const char *lstart, GNodeType *out_op)
+{
     const char *cp = *pp;
     if (*cp == '!') {
 	*out_op = OP_FORCE;
 	(*pp)++;
 	return TRUE;
+    }
     if (*cp == ':') {
 	if (cp[1] == ':') {
 	    *out_op = OP_DOUBLEDEP;
 	    (*pp) += 2;
 	} else {
 	    *out_op = OP_DEPENDS;
 	    (*pp)++;
+	}
 	return TRUE;
+    }
+    {
 	const char *msg = lstart[0] == '.' ? "Unknown directive"
 					   : "Missing dependency operator";
 	Parse_Error(PARSE_FATAL, "%s", msg);
 	return FALSE;
+    }
+}
 static void
 ClearPaths(SearchPathList *paths)
+{
     if (paths != NULL) {
 	SearchPathListNode *ln;
 	for (ln = paths->first; ln != NULL; ln = ln->next)
 	    Dir_ClearPath(ln->datum);
+    }
     Dir_SetPATH();
+}
 static void
 ParseDoDependencySourcesEmpty(ParseSpecial specType, SearchPathList *paths)
+{
     switch (specType) {
     case SP_SUFFIXES:
 	Suff_ClearSuffixes();
 	break;
     case SP_PRECIOUS:
 	allPrecious = TRUE;
 	break;
     case SP_IGNORE:
 	opts.ignoreErrors = TRUE;
 	break;
     case SP_SILENT:
 	opts.beSilent = TRUE;
 	break;
     case SP_PATH:
 	ClearPaths(paths);
 	break;
 #ifdef POSIX
     case SP_POSIX:
 	Var_Set("%POSIX", "1003.2", VAR_GLOBAL);
 	break;
 #endif
     default:
 	break;
+    }
+}
 static void
 AddToPaths(const char *dir, SearchPathList *paths)
+{
     if (paths != NULL) {
 	SearchPathListNode *ln;
 	for (ln = paths->first; ln != NULL; ln = ln->next)
 	    (void)Dir_AddDir(ln->datum, dir);
+    }
+}
 /*
  * If the target was one that doesn't take files as its sources
  * but takes something like suffixes, we take each
  * space-separated word on the line as a something and deal
  * with it accordingly.
+ *
  * If the target was .SUFFIXES, we take each source as a
  * suffix and add it to the list of suffixes maintained by the
  * Suff module.
+ *
  * If the target was a .PATH, we add the source as a directory
  * to search on the search path.
+ *
  * If it was .INCLUDES, the source is taken to be the suffix of
  * files which will be #included and whose search path should
  * be present in the .INCLUDES variable.
+ *
  * If it was .LIBS, the source is taken to be the suffix of
  * files which are considered libraries and whose search path
  * should be present in the .LIBS variable.
+ *
  * If it was .NULL, the source is the suffix to use when a file
  * has no valid suffix.
+ *
  * If it was .OBJDIR, the source is a new definition for .OBJDIR,
  * and will cause make to do a new chdir to that path.
  */
 static void
 ParseDoDependencySourceSpecial(ParseSpecial specType, char *word,
 			       SearchPathList *paths)
+{
     switch (specType) {
     case SP_SUFFIXES:
 	Suff_AddSuffix(word, &mainNode);
 	break;
     case SP_PATH:
 	AddToPaths(word, paths);
 	break;
     case SP_INCLUDES:
 	Suff_AddInclude(word);
 	break;
     case SP_LIBS:
 	Suff_AddLib(word);
 	break;
     case SP_NULL:
 	Suff_SetNull(word);
 	break;
     case SP_OBJDIR:
 	Main_SetObjdir("%s", word);
 	break;
     default:
 	break;
+    }
+}
 static Boolean
 ParseDoDependencyTargets(char **inout_cp,
 			 char **inout_line,
 			 const char *lstart,
 			 ParseSpecial *inout_specType,
 			 GNodeType *inout_tOp,
 			 SearchPathList **inout_paths,
 			 StringList *curTargs)
+{
     char *cp = *inout_cp;
     char *line = *inout_line;
     char savec;
     for (;;) {
 	/*
 	 * Here LINE points to the beginning of the next word, and
 	 * LSTART points to the actual beginning of the line.
 	 */
 	/* Find the end of the next word. */
 	cp = line;
 	ParseDependencyTargetWord(&cp, lstart);
 	/*
 	 * If the word is followed by a left parenthesis, it's the
 	 * name of an object file inside an archive (ar file).
 	 */
 	if (!ParseIsEscaped(lstart, cp) && *cp == '(') {
 	    /*
 	     * Archives must be handled specially to make sure the OP_ARCHV
 	     * flag is set in their 'type' field, for one thing, and because
 	     * things like "archive(file1.o file2.o file3.o)" are permissible.
 	     * Arch_ParseArchive will set 'line' to be the first non-blank
 	     * after the archive-spec. It creates/finds nodes for the members
 	     * and places them on the given list, returning TRUE if all
 	     * went well and FALSE if there was an error in the
 	     * specification. On error, line should remain untouched.
 	     */
 	    if (!Arch_ParseArchive(&line, targets, VAR_CMD)) {
 		Parse_Error(PARSE_FATAL,
 			    "Error in archive specification: \"%s\"", line);
 		return FALSE;
 	    } else {
 		/* Done with this word; on to the next. */
 		cp = line;
 		continue;
+	    }
+	}
 	if (!*cp) {
 	    ParseErrorNoDependency(lstart);
 	    return FALSE;
+	}
 	/* Insert a null terminator. */
 	savec = *cp;
 	*cp = '\0';
 	if (!ParseDoDependencyTarget(line, inout_specType, inout_tOp,
 				     inout_paths))
 	    return FALSE;
 	/*
 	 * Have word in line. Get or create its node and stick it at
 	 * the end of the targets list
 	 */
 	if (*inout_specType == SP_NOT && *line != '\0') {
 	    ParseDoDependencyTargetMundane(line, curTargs);
 	} else if (*inout_specType == SP_PATH && *line != '.' && *line != '\0') {
 	    Parse_Error(PARSE_WARNING, "Extra target (%s) ignored", line);
+	}
 	/* Don't need the inserted null terminator any more. */
 	*cp = savec;
 	/*
 	 * If it is a special type and not .PATH, it's the only target we
 	 * allow on this line...
 	 */
 	if (*inout_specType != SP_NOT && *inout_specType != SP_PATH) {
 	    ParseDoDependencyTargetExtraWarn(&cp, lstart);
 	} else {
 	    pp_skip_whitespace(&cp);
+	}
 	line = cp;
 	if (*line == '\0')
 	    break;
 	if ((*line == '!' || *line == ':') && !ParseIsEscaped(lstart, line))
 	    break;
+    }
     *inout_cp = cp;
     *inout_line = line;
     return TRUE;
+}
 static void
 ParseDoDependencySourcesSpecial(char *start, char *end,
 				ParseSpecial specType, SearchPathList *paths)
+{
     char savec;
     while (*start) {
 	while (*end && !ch_isspace(*end))
 	    end++;
 	savec = *end;
 	*end = '\0';
 	ParseDoDependencySourceSpecial(specType, start, paths);
 	*end = savec;
 	if (savec != '\0')
 	    end++;
 	pp_skip_whitespace(&end);
 	start = end;
+    }
+}
 static Boolean
 ParseDoDependencySourcesMundane(char *start, char *end,
 				ParseSpecial specType, GNodeType tOp)
+{
     while (*start) {
 	/*
 	 * The targets take real sources, so we must beware of archive
 	 * specifications (i.e. things with left parentheses in them)
 	 * and handle them accordingly.
 	 */
 	for (; *end && !ch_isspace(*end); end++) {
 	    if (*end == '(' && end > start && end[-1] != '$') {
 		/*
 		 * Only stop for a left parenthesis if it isn't at the
 		 * start of a word (that'll be for variable changes
 		 * later) and isn't preceded by a dollar sign (a dynamic
 		 * source).
 		 */
 		break;
+	    }
+	}
 	if (*end == '(') {
 	    GNodeList *sources = Lst_New();
 	    if (!Arch_ParseArchive(&start, sources, VAR_CMD)) {
 		Parse_Error(PARSE_FATAL,
 			    "Error in source archive spec \"%s\"", start);
 		return FALSE;
+	    }
 	    while (!Lst_IsEmpty(sources)) {
 		GNode *gn = Lst_Dequeue(sources);
 		ParseDoSrc(tOp, gn->name, specType);
+	    }
 	    Lst_Free(sources);
 	    end = start;
 	} else {
 	    if (*end) {
 		*end = '\0';
 		end++;
+	    }
 	    ParseDoSrc(tOp, start, specType);
+	}
 	pp_skip_whitespace(&end);
 	start = end;
+    }
     return TRUE;
+}
 /* Parse a dependency line consisting of targets, followed by a dependency
  * operator, optionally followed by sources.
+ *
  * The nodes of the sources are linked as children to the nodes of the
  * targets. Nodes are created as necessary.
+ *
  * The operator is applied to each node in the global 'targets' list,
  * which is where the nodes found for the targets are kept, by means of
  * the ParseDoOp function.
+ *
  * The sources are parsed in much the same way as the targets, except
  * that they are expanded using the wildcarding scheme of the C-Shell,
  * and all instances of the resulting words in the list of all targets
  * are found. Each of the resulting nodes is then linked to each of the
  * targets as one of its children.
+ *
  * Certain targets and sources such as .PHONY or .PRECIOUS are handled
  * specially. These are the ones detailed by the specType variable.
+ *
  * The storing of transformation rules such as '.c.o' is also taken care of
  * here. A target is recognized as a transformation rule by calling
  * Suff_IsTransform. If it is a transformation rule, its node is gotten
  * from the suffix module via Suff_AddTransform rather than the standard
  * Targ_FindNode in the target module.
  */
 static void
 ParseDoDependency(char *line)
+{
     char *cp;			/* our current position */
     GNodeType op;		/* the operator on the line */
     SearchPathList *paths;	/* search paths to alter when parsing
 				 * a list of .PATH targets */
     int tOp;			/* operator from special target */
     StringList *curTargs;	/* target names to be found and added
 				 * to the targets list */
     char *lstart = line;
     /*
      * specType contains the SPECial TYPE of the current target. It is SP_NOT
      * if the target is unspecial. If it *is* special, however, the children
      * are linked as children of the parent but not vice versa.
      */
     ParseSpecial specType = SP_NOT;
     DEBUG1(PARSE, "ParseDoDependency(%s)\n", line);
     tOp = 0;
     paths = NULL;
     curTargs = Lst_New();
     /*
      * First, grind through the targets.
      */
     if (!ParseDoDependencyTargets(&cp, &line, lstart, &specType, &tOp, &paths,
 				  curTargs))
 	goto out;
     /*
      * Don't need the list of target names anymore...
      */
     Lst_Free(curTargs);
     curTargs = NULL;
     if (!Lst_IsEmpty(targets))
 	ParseDoDependencyCheckSpec(specType);
     /*
      * Have now parsed all the target names. Must parse the operator next.
      */
     if (!ParseDoDependencyParseOp(&cp, lstart, &op))
 	goto out;
     /*
      * Apply the operator to the target. This is how we remember which
      * operator a target was defined with. It fails if the operator
      * used isn't consistent across all references.
      */
     ApplyDependencyOperator(op);
     /*
      * Onward to the sources.
+     *
      * LINE will now point to the first source word, if any, or the
      * end of the string if not.
      */
     pp_skip_whitespace(&cp);
     line = cp;
     /*
      * Several special targets take different actions if present with no
      * sources:
      *	a .SUFFIXES line with no sources clears out all old suffixes
      *	a .PRECIOUS line makes all targets precious
      *	a .IGNORE line ignores errors for all targets
      *	a .SILENT line creates silence when making all targets
      *	a .PATH removes all directories from the search path(s).
      */
     if (!*line) {
 	ParseDoDependencySourcesEmpty(specType, paths);
     } else if (specType == SP_MFLAGS) {
 	/*
 	 * Call on functions in main.c to deal with these arguments and
 	 * set the initial character to a null-character so the loop to
 	 * get sources won't get anything
 	 */
 	Main_ParseArgLine(line);
 	*line = '\0';
     } else if (specType == SP_SHELL) {
 	if (!Job_ParseShell(line)) {
 	    Parse_Error(PARSE_FATAL, "improper shell specification");
 	    goto out;
+	}
 	*line = '\0';
     } else if (specType == SP_NOTPARALLEL || specType == SP_SINGLESHELL ||
 	       specType == SP_DELETE_ON_ERROR) {
 	*line = '\0';
+    }
     /*
      * NOW GO FOR THE SOURCES
      */
     if (specType == SP_SUFFIXES || specType == SP_PATH ||
 	specType == SP_INCLUDES || specType == SP_LIBS ||
 	specType == SP_NULL || specType == SP_OBJDIR)
+    {
 	ParseDoDependencySourcesSpecial(line, cp, specType, paths);
 	if (paths) {
 	    Lst_Free(paths);
 	    paths = NULL;
+	}
 	if (specType == SP_PATH)
 	    Dir_SetPATH();
     } else {
 	assert(paths == NULL);
 	if (!ParseDoDependencySourcesMundane(line, cp, specType, tOp))
 	    goto out;
+    }
     FindMainTarget();
 out:
     if (paths != NULL)
 	Lst_Free(paths);
     if (curTargs != NULL)
 	Lst_Free(curTargs);
+}
 typedef struct VarAssignParsed {
     const char *nameStart;	/* unexpanded */
     const char *nameEnd;	/* before operator adjustment */
     const char *eq;		/* the '=' of the assignment operator */
 } VarAssignParsed;
 /* Determine the assignment operator and adjust the end of the variable
  * name accordingly. */
 static void
 AdjustVarassignOp(const VarAssignParsed *pvar, const char *value,
 		  VarAssign *out_var)
+{
     const char *op = pvar->eq;
     const char * const name = pvar->nameStart;
     VarAssignOp type;
     if (op > name && op[-1] == '+') {
 	type = VAR_APPEND;
 	op--;
     } else if (op > name && op[-1] == '?') {
 	op--;
 	type = VAR_DEFAULT;
     } else if (op > name && op[-1] == ':') {
 	op--;
 	type = VAR_SUBST;
     } else if (op > name && op[-1] == '!') {
 	op--;
 	type = VAR_SHELL;
     } else {
 	type = VAR_NORMAL;
 #ifdef SUNSHCMD
 	while (op > name && ch_isspace(op[-1]))
 	    op--;
 	if (op >= name + 3 && op[-3] == ':' && op[-2] == 's' && op[-1] == 'h') {
 	    type = VAR_SHELL;
 	    op -= 3;
+	}
 #endif
+    }
+    {
 	const char *nameEnd = pvar->nameEnd < op ? pvar->nameEnd : op;
 	out_var->varname = bmake_strsedup(pvar->nameStart, nameEnd);
 	out_var->op = type;
 	out_var->value = value;
+    }
+}
 /* Parse a variable assignment, consisting of a single-word variable name,
  * optional whitespace, an assignment operator, optional whitespace and the
  * variable value.
+ *
  * Used for both lines in a file and command line arguments. */
 Boolean
 Parse_IsVar(const char *p, VarAssign *out_var)
+{
     VarAssignParsed pvar;
     const char *firstSpace = NULL;
     char ch;
     int level = 0;
     /* Skip to variable name */
     while (*p == ' ' || *p == '\t')
 	p++;
     /* During parsing, the '+' of the '+=' operator is initially parsed
      * as part of the variable name.  It is later corrected, as is the ':sh'
      * modifier. Of these two (nameEnd and op), the earlier one determines the
      * actual end of the variable name. */
     pvar.nameStart = p;
 #ifdef CLEANUP
     pvar.nameEnd = NULL;
     pvar.eq = NULL;
 #endif
     /* Scan for one of the assignment operators outside a variable expansion */
     while ((ch = *p++) != 0) {
 	if (ch == '(' || ch == '{') {
 	    level++;
 	    continue;
+	}
 	if (ch == ')' || ch == '}') {
 	    level--;
 	    continue;
+	}
 	if (level != 0)
 	    continue;
 	if (ch == ' ' || ch == '\t')
 	    if (firstSpace == NULL)
 		firstSpace = p - 1;
 	while (ch == ' ' || ch == '\t')
 	    ch = *p++;
 #ifdef SUNSHCMD
 	if (ch == ':' && strncmp(p, "sh", 2) == 0) {
 	    p += 2;
 	    continue;
+	}
 #endif
 	if (ch == '=') {
 	    pvar.eq = p - 1;
 	    pvar.nameEnd = firstSpace != NULL ? firstSpace : p - 1;
 	    cpp_skip_whitespace(&p);
 	    AdjustVarassignOp(&pvar, p, out_var);
 	    return TRUE;
+	}
 	if (*p == '=' && (ch == '+' || ch == ':' || ch == '?' || ch == '!')) {
 	    pvar.eq = p;
 	    pvar.nameEnd = firstSpace != NULL ? firstSpace : p;
 	    p++;
 	    cpp_skip_whitespace(&p);
 	    AdjustVarassignOp(&pvar, p, out_var);
 	    return TRUE;
+	}
 	if (firstSpace != NULL)
 	    return FALSE;
+    }
     return FALSE;
+}
 static void
 VarCheckSyntax(VarAssignOp type, const char *uvalue, GNode *ctxt)
+{
     if (DEBUG(LINT)) {
 	if (type != VAR_SUBST && strchr(uvalue, '$') != NULL) {
 	    /* Check for syntax errors such as unclosed expressions or
 	     * unknown modifiers. */
 	    char *expandedValue;
 	    (void)Var_Subst(uvalue, ctxt, VARE_NONE, &expandedValue);
 	    /* TODO: handle errors */
 	    free(expandedValue);
+	}
+    }
+}
 static void
 VarAssign_EvalSubst(const char *name, const char *uvalue, GNode *ctxt,
 		    const char **out_avalue, void **out_avalue_freeIt)
+{
     const char *avalue = uvalue;
     char *evalue;
     /*
      * Allow variables in the old value to be undefined, but leave their
      * expressions alone -- this is done by forcing oldVars to be false.
      * XXX: This can cause recursive variables, but that's not hard to do,
      * and this allows someone to do something like
+     *
      *  CFLAGS = $(.INCLUDES)
      *  CFLAGS := -I.. $(CFLAGS)
+     *
      * And not get an error.
      */
     Boolean oldOldVars = oldVars;
     oldVars = FALSE;
     /*
      * make sure that we set the variable the first time to nothing
      * so that it gets substituted!
      */
     if (!Var_Exists(name, ctxt))
 	Var_Set(name, "", ctxt);
     (void)Var_Subst(uvalue, ctxt, VARE_WANTRES|VARE_ASSIGN, &evalue);
     /* TODO: handle errors */
     oldVars = oldOldVars;
     avalue = evalue;
     Var_Set(name, avalue, ctxt);
     *out_avalue = avalue;
     *out_avalue_freeIt = evalue;
+}
 static void
 VarAssign_EvalShell(const char *name, const char *uvalue, GNode *ctxt,
 		    const char **out_avalue, void **out_avalue_freeIt)
+{
     const char *cmd, *errfmt;
     char *cmdOut;
     void *cmd_freeIt = NULL;
     cmd = uvalue;
     if (strchr(cmd, '$') != NULL) {
 	char *ecmd;
 	(void)Var_Subst(cmd, VAR_CMD, VARE_UNDEFERR|VARE_WANTRES, &ecmd);
 	/* TODO: handle errors */
 	cmd = cmd_freeIt = ecmd;
+    }
     cmdOut = Cmd_Exec(cmd, &errfmt);
     Var_Set(name, cmdOut, ctxt);
     *out_avalue = *out_avalue_freeIt = cmdOut;
     if (errfmt)
 	Parse_Error(PARSE_WARNING, errfmt, cmd);
     free(cmd_freeIt);
+}
 /* Perform a variable assignment.
+ *
  * The actual value of the variable is returned in *out_avalue and
  * *out_avalue_freeIt.  Especially for VAR_SUBST and VAR_SHELL this can differ
  * from the literal value.
+ *
  * Return whether the assignment was actually done.  The assignment is only
  * skipped if the operator is '?=' and the variable already exists. */
 static Boolean
 VarAssign_Eval(const char *name, VarAssignOp op, const char *uvalue,
 	       GNode *ctxt, const char **out_avalue, void **out_avalue_freeIt)
+{
     const char *avalue = uvalue;
     void *avalue_freeIt = NULL;
     if (op == VAR_APPEND) {
 	Var_Append(name, uvalue, ctxt);
     } else if (op == VAR_SUBST) {
         VarAssign_EvalSubst(name, uvalue, ctxt, &avalue, &avalue_freeIt);
     } else if (op == VAR_SHELL) {
         VarAssign_EvalShell(name, uvalue, ctxt, &avalue, &avalue_freeIt);
     } else {
 	if (op == VAR_DEFAULT && Var_Exists(name, ctxt)) {
 	    *out_avalue_freeIt = NULL;
 	    return FALSE;
+	}
 	/* Normal assignment -- just do it. */
 	Var_Set(name, uvalue, ctxt);
+    }
     *out_avalue = avalue;
     *out_avalue_freeIt = avalue_freeIt;
     return TRUE;
+}
 static void
 VarAssignSpecial(const char *name, const char *avalue)
+{
     if (strcmp(name, MAKEOVERRIDES) == 0)
 	Main_ExportMAKEFLAGS(FALSE);	/* re-export MAKEFLAGS */
     else if (strcmp(name, ".CURDIR") == 0) {
 	/*
 	 * Someone is being (too?) clever...
 	 * Let's pretend they know what they are doing and
 	 * re-initialize the 'cur' CachedDir.
 	 */
 	Dir_InitCur(avalue);
 	Dir_SetPATH();
     } else if (strcmp(name, MAKE_JOB_PREFIX) == 0) {
 	Job_SetPrefix();
     } else if (strcmp(name, MAKE_EXPORTED) == 0) {
 	Var_Export(avalue, FALSE);
+    }
+}
 /* Take the variable assignment in the passed line and execute it.
+ *
  * Note: There is a lexical ambiguity with assignment modifier characters
  * in variable names. This routine interprets the character before the =
  * as a modifier. Therefore, an assignment like
  *	C++=/usr/bin/CC
  * is interpreted as "C+ +=" instead of "C++ =".
+ *
  * Input:
  *	p		A line guaranteed to be a variable assignment
  *			(see Parse_IsVar).
  *	ctxt		Context in which to do the assignment
  */
 void
 Parse_DoVar(VarAssign *var, GNode *ctxt)
+{
     const char *avalue;		/* actual value (maybe expanded) */
     void *avalue_freeIt;
     VarCheckSyntax(var->op, var->value, ctxt);
     if (VarAssign_Eval(var->varname, var->op, var->value, ctxt,
 		       &avalue, &avalue_freeIt))
 	VarAssignSpecial(var->varname, avalue);
     free(avalue_freeIt);
     free(var->varname);
+}
 /*
  * ParseMaybeSubMake --
  *	Scan the command string to see if it a possible submake node
  * Input:
  *	cmd		the command to scan
  * Results:
  *	TRUE if the command is possibly a submake, FALSE if not.
  */
 static Boolean
 ParseMaybeSubMake(const char *cmd)
+{
     size_t i;
     static struct {
 	const char *name;
 	size_t len;
     } vals[] = {
 #define MKV(A)	{	A, sizeof(A) - 1	}
 	MKV("${MAKE}"),
 	MKV("${.MAKE}"),
 	MKV("$(MAKE)"),
 	MKV("$(.MAKE)"),
 	MKV("make"),
     };
     for (i = 0; i < sizeof vals / sizeof vals[0]; i++) {
 	char *ptr;
 	if ((ptr = strstr(cmd, vals[i].name)) == NULL)
 	    continue;
 	if ((ptr == cmd || !ch_isalnum(ptr[-1]))
 	    && !ch_isalnum(ptr[vals[i].len]))
 	    return TRUE;
+    }
     return FALSE;
+}
 /* Append the command to the target node.
+ *
  * The node may be marked as a submake node if the command is determined to
  * be that. */
 static void
 ParseAddCmd(GNode *gn, char *cmd)
+{
     /* Add to last (ie current) cohort for :: targets */
     if ((gn->type & OP_DOUBLEDEP) && gn->cohorts->last != NULL)
 	gn = gn->cohorts->last->datum;
     /* if target already supplied, ignore commands */
     if (!(gn->type & OP_HAS_COMMANDS)) {
 	Lst_Append(gn->commands, cmd);
 	if (ParseMaybeSubMake(cmd))
 	    gn->type |= OP_SUBMAKE;
 	ParseMark(gn);
     } else {
 #if 0
 	/* XXX: We cannot do this until we fix the tree */
 	Lst_Append(gn->commands, cmd);
 	Parse_Error(PARSE_WARNING,
 		     "overriding commands for target \"%s\"; "
 		     "previous commands defined at %s: %d ignored",
 		     gn->name, gn->fname, gn->lineno);
 #else
 	Parse_Error(PARSE_WARNING,
 		    "duplicate script for target \"%s\" ignored",
 		    gn->name);
 	ParseErrorInternal(gn->fname, (size_t)gn->lineno, PARSE_WARNING,
 			   "using previous script for \"%s\" defined here",
 			   gn->name);
 #endif
+    }
+}
 /* Add a directory to the path searched for included makefiles bracketed
  * by double-quotes. */
 void
 Parse_AddIncludeDir(const char *dir)
+{
     (void)Dir_AddDir(parseIncPath, dir);
+}
 /* Push to another file.
+ *
  * The input is the line minus the '.'. A file spec is a string enclosed in
  * <> or "". The <> file is looked for only in sysIncPath. The "" file is
  * first searched in the parsedir and then in the directories specified by
  * the -I command line options.
  */
 static void
 Parse_include_file(char *file, Boolean isSystem, Boolean depinc, int silent)
+{
     struct loadedfile *lf;
     char *fullname;		/* full pathname of file */
     char *newName;
     char *prefEnd, *incdir;
     int fd;
     int i;
     /*
      * Now we know the file's name and its search path, we attempt to
      * find the durn thing. A return of NULL indicates the file don't
      * exist.
      */
     fullname = file[0] == '/' ? bmake_strdup(file) : NULL;
     if (fullname == NULL && !isSystem) {
 	/*
 	 * Include files contained in double-quotes are first searched for
 	 * relative to the including file's location. We don't want to
 	 * cd there, of course, so we just tack on the old file's
 	 * leading path components and call Dir_FindFile to see if
 	 * we can locate the beast.
 	 */
 	incdir = bmake_strdup(CurFile()->fname);
 	prefEnd = strrchr(incdir, '/');
 	if (prefEnd != NULL) {
 	    *prefEnd = '\0';
 	    /* Now do lexical processing of leading "../" on the filename */
 	    for (i = 0; strncmp(file + i, "../", 3) == 0; i += 3) {
 		prefEnd = strrchr(incdir + 1, '/');
 		if (prefEnd == NULL || strcmp(prefEnd, "/..") == 0)
 		    break;
 		*prefEnd = '\0';
+	    }
 	    newName = str_concat3(incdir, "/", file + i);
 	    fullname = Dir_FindFile(newName, parseIncPath);
 	    if (fullname == NULL)
 		fullname = Dir_FindFile(newName, dirSearchPath);
 	    free(newName);
+	}
 	free(incdir);
 	if (fullname == NULL) {
 	    /*
 	     * Makefile wasn't found in same directory as included makefile.
 	     * Search for it first on the -I search path,
 	     * then on the .PATH search path, if not found in a -I directory.
 	     * If we have a suffix specific path we should use that.
 	     */
 	    char *suff;
 	    SearchPath *suffPath = NULL;
 	    if ((suff = strrchr(file, '.'))) {
 		suffPath = Suff_GetPath(suff);
 		if (suffPath != NULL) {
 		    fullname = Dir_FindFile(file, suffPath);
+		}
+	    }
 	    if (fullname == NULL) {
 		fullname = Dir_FindFile(file, parseIncPath);
 		if (fullname == NULL) {
 		    fullname = Dir_FindFile(file, dirSearchPath);
+		}
+	    }
+	}
+    }
     /* Looking for a system file or file still not found */
     if (fullname == NULL) {
 	/*
 	 * Look for it on the system path
 	 */
-	SearchPath *path = Lst_IsEmpty(sysIncPath) ? defIncPath : sysIncPath;
+	SearchPath *path = Lst_IsEmpty(sysIncPath) ? defSysIncPath : sysIncPath;
 	fullname = Dir_FindFile(file, path);
+    }
     if (fullname == NULL) {
 	if (!silent)
 	    Parse_Error(PARSE_FATAL, "Could not find %s", file);
 	return;
+    }
     /* Actually open the file... */
     fd = open(fullname, O_RDONLY);
     if (fd == -1) {
 	if (!silent)
 	    Parse_Error(PARSE_FATAL, "Cannot open %s", fullname);
 	free(fullname);
 	return;
+    }
     /* load it */
     lf = loadfile(fullname, fd);
     /* Start reading from this file next */
     Parse_SetInput(fullname, 0, -1, loadedfile_nextbuf, lf);
     CurFile()->lf = lf;
     if (depinc)
 	doing_depend = depinc;	/* only turn it on */
+}
 static void
 ParseDoInclude(char *line)
+{
     char endc;			/* the character which ends the file spec */
     char *cp;			/* current position in file spec */
     int silent = *line != 'i';
     char *file = line + (silent ? 8 : 7);
     /* Skip to delimiter character so we know where to look */
     while (*file == ' ' || *file == '\t')
 	file++;
     if (*file != '"' && *file != '<') {
 	Parse_Error(PARSE_FATAL,
 		    ".include filename must be delimited by '\"' or '<'");
 	return;
+    }
     /*
      * Set the search path on which to find the include file based on the
      * characters which bracket its name. Angle-brackets imply it's
      * a system Makefile while double-quotes imply it's a user makefile
      */
     if (*file == '<') {
 	endc = '>';
     } else {
 	endc = '"';
+    }
     /* Skip to matching delimiter */
     for (cp = ++file; *cp && *cp != endc; cp++)
 	continue;
     if (*cp != endc) {
 	Parse_Error(PARSE_FATAL,
 		    "Unclosed %cinclude filename. '%c' expected",
 		    '.', endc);
 	return;
+    }
     *cp = '\0';
     /*
      * Substitute for any variables in the file name before trying to
      * find the thing.
      */
     (void)Var_Subst(file, VAR_CMD, VARE_WANTRES, &file);
     /* TODO: handle errors */
     Parse_include_file(file, endc == '>', *line == 'd', silent);
     free(file);
+}
 /* Split filename into dirname + basename, then assign these to the
  * given variables. */
 static void
 SetFilenameVars(const char *filename, const char *dirvar, const char *filevar)
+{
     const char *slash, *dirname, *basename;
     void *freeIt;
     slash = strrchr(filename, '/');
     if (slash == NULL) {
 	dirname = curdir;
 	basename = filename;
 	freeIt = NULL;
     } else {
 	dirname = freeIt = bmake_strsedup(filename, slash);
 	basename = slash + 1;
+    }
     Var_Set(dirvar, dirname, VAR_GLOBAL);
     Var_Set(filevar, basename, VAR_GLOBAL);
     DEBUG5(PARSE, "%s: ${%s} = `%s' ${%s} = `%s'\n",
 	   __func__, dirvar, dirname, filevar, basename);
     free(freeIt);
+}
 /* Return the immediately including file.
+ *
  * This is made complicated since the .for loop is implemented as a special
  * kind of .include; see For_Run. */
 static const char *
 GetActuallyIncludingFile(void)
+{
     size_t i;
     const IFile *incs = GetInclude(0);
     for (i = includes.len; i >= 2; i--)
 	if (!incs[i - 1].fromForLoop)
 	    return incs[i - 2].fname;
     return NULL;
+}
 /* Set .PARSEDIR, .PARSEFILE, .INCLUDEDFROMDIR and .INCLUDEDFROMFILE. */
 static void
 ParseSetParseFile(const char *filename)
+{
     const char *including;
     SetFilenameVars(filename, ".PARSEDIR", ".PARSEFILE");
     including = GetActuallyIncludingFile();
     if (including != NULL) {
 	SetFilenameVars(including,
 			".INCLUDEDFROMDIR", ".INCLUDEDFROMFILE");
     } else {
 	Var_Delete(".INCLUDEDFROMDIR", VAR_GLOBAL);
 	Var_Delete(".INCLUDEDFROMFILE", VAR_GLOBAL);
+    }
+}
 /* Track the makefiles we read - so makefiles can set dependencies on them.
  * Avoid adding anything more than once. */
 static void
 ParseTrackInput(const char *name)
+{
     char *fp = NULL;
     const char *old = Var_Value(MAKE_MAKEFILES, VAR_GLOBAL, &fp);
     if (old) {
 	size_t name_len = strlen(name);
 	const char *ep = old + strlen(old) - name_len;
 	/* does it contain name? */
 	for (; old != NULL; old = strchr(old, ' ')) {
 	    if (*old == ' ')
 		old++;
 	    if (old >= ep)
 		break;		/* cannot contain name */
 	    if (memcmp(old, name, name_len) == 0 &&
 		(old[name_len] == '\0' || old[name_len] == ' '))
 		goto cleanup;
+	}
+    }
     Var_Append(MAKE_MAKEFILES, name, VAR_GLOBAL);
 cleanup:
     bmake_free(fp);
+}
 /* Start Parsing from the given source.
+ *
  * The given file is added to the includes stack. */
 void
 Parse_SetInput(const char *name, int line, int fd,
 	       char *(*nextbuf)(void *, size_t *), void *arg)
+{
     IFile *curFile;
     char *buf;
     size_t len;
     Boolean fromForLoop = name == NULL;
     if (fromForLoop)
 	name = CurFile()->fname;
     else
 	ParseTrackInput(name);
     if (DEBUG(PARSE))
 	debug_printf("%s: file %s, line %d, fd %d, nextbuf %s, arg %p\n",
 		     __func__, name, line, fd,
 		     nextbuf == loadedfile_nextbuf ? "loadedfile" : "other",
 		     arg);
     if (fd == -1 && nextbuf == NULL)
 	/* sanity */
 	return;
     curFile = Vector_Push(&includes);
     /*
      * Once the previous state has been saved, we can get down to reading
      * the new file. We set up the name of the file to be the absolute
      * name of the include file so error messages refer to the right
      * place.
      */
     curFile->fname = bmake_strdup(name);
     curFile->fromForLoop = fromForLoop;
     curFile->lineno = line;
     curFile->first_lineno = line;
     curFile->nextbuf = nextbuf;
     curFile->nextbuf_arg = arg;
     curFile->lf = NULL;
     curFile->depending = doing_depend;	/* restore this on EOF */
     assert(nextbuf != NULL);
     /* Get first block of input data */
     buf = curFile->nextbuf(curFile->nextbuf_arg, &len);
     if (buf == NULL) {
 	/* Was all a waste of time ... */
 	if (curFile->fname)
 	    free(curFile->fname);
 	free(curFile);
 	return;
+    }
     curFile->buf_freeIt = buf;
     curFile->buf_ptr = buf;
     curFile->buf_end = buf + len;
     curFile->cond_depth = Cond_save_depth();
     ParseSetParseFile(name);
+}
 /* Check if the directive is an include directive. */
 static Boolean
 IsInclude(const char *dir, Boolean sysv)
+{
 	if (dir[0] == 's' || dir[0] == '-' || (dir[0] == 'd' && !sysv))
 		dir++;
 	if (strncmp(dir, "include", 7) != 0)
 		return FALSE;
 	/* Space is not mandatory for BSD .include */
 	return !sysv || ch_isspace(dir[7]);
+}
 #ifdef SYSVINCLUDE
 /* Check if the line is a SYSV include directive. */
 static Boolean
 IsSysVInclude(const char *line)
+{
 	const char *p;
 	if (!IsInclude(line, TRUE))
 		return FALSE;
 	/* Avoid interpreting a dependency line as an include */
 	for (p = line; (p = strchr(p, ':')) != NULL;) {
 		if (*++p == '\0') {
 			/* end of line -> dependency */
 			return FALSE;
+		}
 		if (*p == ':' || ch_isspace(*p)) {
 			/* :: operator or ': ' -> dependency */
 			return FALSE;
+		}
+	}
 	return TRUE;
+}
 /* Push to another file.  The line points to the word "include". */
 static void
 ParseTraditionalInclude(char *line)
+{
     char *cp;			/* current position in file spec */
     int done = 0;
     int silent = line[0] != 'i';
     char *file = line + (silent ? 8 : 7);
     char *all_files;
     DEBUG2(PARSE, "%s: %s\n", __func__, file);
     pp_skip_whitespace(&file);
     /*
      * Substitute for any variables in the file name before trying to
      * find the thing.
      */
     (void)Var_Subst(file, VAR_CMD, VARE_WANTRES, &all_files);
     /* TODO: handle errors */
     if (*file == '\0') {
 	Parse_Error(PARSE_FATAL, "Filename missing from \"include\"");
 	goto out;
+    }
     for (file = all_files; !done; file = cp + 1) {
 	/* Skip to end of line or next whitespace */
 	for (cp = file; *cp && !ch_isspace(*cp); cp++)
 	    continue;
 	if (*cp)
 	    *cp = '\0';
 	else
 	    done = 1;
 	Parse_include_file(file, FALSE, FALSE, silent);
+    }
 out:
     free(all_files);
+}
 #endif
 #ifdef GMAKEEXPORT
 /* Parse "export <variable>=<value>", and actually export it. */
 static void
 ParseGmakeExport(char *line)
+{
     char *variable = line + 6;
     char *value;
     DEBUG2(PARSE, "%s: %s\n", __func__, variable);
     pp_skip_whitespace(&variable);
     for (value = variable; *value && *value != '='; value++)
 	continue;
     if (*value != '=') {
 	Parse_Error(PARSE_FATAL,
 		    "Variable/Value missing from \"export\"");
 	return;
+    }
     *value++ = '\0';		/* terminate variable */
     /*
      * Expand the value before putting it in the environment.
      */
     (void)Var_Subst(value, VAR_CMD, VARE_WANTRES, &value);
     /* TODO: handle errors */
     setenv(variable, value, 1);
     free(value);
+}
 #endif
 /* Called when EOF is reached in the current file. If we were reading an
  * include file, the includes stack is popped and things set up to go back
  * to reading the previous file at the previous location.
+ *
  * Results:
  *	TRUE to continue parsing, i.e. it had only reached the end of an
  *	included file, FALSE if the main file has been parsed completely.
  */
 static Boolean
 ParseEOF(void)
+{
     char *ptr;
     size_t len;
     IFile *curFile = CurFile();
     assert(curFile->nextbuf != NULL);
     doing_depend = curFile->depending;	/* restore this */
     /* get next input buffer, if any */
     ptr = curFile->nextbuf(curFile->nextbuf_arg, &len);
     curFile->buf_ptr = ptr;
     curFile->buf_freeIt = ptr;
     curFile->buf_end = ptr + len;
     curFile->lineno = curFile->first_lineno;
     if (ptr != NULL) {
 	/* Iterate again */
 	return TRUE;
+    }
     /* Ensure the makefile (or loop) didn't have mismatched conditionals */
     Cond_restore_depth(curFile->cond_depth);
     if (curFile->lf != NULL) {
 	loadedfile_destroy(curFile->lf);
 	curFile->lf = NULL;
+    }
     /* Dispose of curFile info */
     /* Leak curFile->fname because all the gnodes have pointers to it */
     free(curFile->buf_freeIt);
     Vector_Pop(&includes);
     if (includes.len == 0) {
 	/* We've run out of input */
 	Var_Delete(".PARSEDIR", VAR_GLOBAL);
 	Var_Delete(".PARSEFILE", VAR_GLOBAL);
 	Var_Delete(".INCLUDEDFROMDIR", VAR_GLOBAL);
 	Var_Delete(".INCLUDEDFROMFILE", VAR_GLOBAL);
 	return FALSE;
+    }
     curFile = CurFile();
     DEBUG2(PARSE, "ParseEOF: returning to file %s, line %d\n",
 	   curFile->fname, curFile->lineno);
     ParseSetParseFile(curFile->fname);
     return TRUE;
+}
 #define PARSE_RAW 1
 #define PARSE_SKIP 2
 static char *
 ParseGetLine(int flags)
+{
     IFile *cf = CurFile();
     char *ptr;
     char ch;
     char *line;
     char *line_end;
     char *escaped;
     char *comment;
     char *tp;
     /* Loop through blank lines and comment lines */
     for (;;) {
 	cf->lineno++;
 	line = cf->buf_ptr;
 	ptr = line;
 	line_end = line;
 	escaped = NULL;
 	comment = NULL;
 	for (;;) {
 	    /* XXX: can buf_end ever be null? */
 	    if (cf->buf_end != NULL && ptr == cf->buf_end) {
 		/* end of buffer */
 		ch = 0;
 		break;
+	    }
 	    ch = *ptr;
 	    if (ch == 0 || (ch == '\\' && ptr[1] == 0)) {
 		/* XXX: can buf_end ever be null? */
 		if (cf->buf_end == NULL)
 		    /* End of string (aka for loop) data */
 		    break;
 		/* see if there is more we can parse */
 		while (ptr++ < cf->buf_end) {
 		    if ((ch = *ptr) == '\n') {
 			if (ptr > line && ptr[-1] == '\\')
 			    continue;
 			Parse_Error(PARSE_WARNING,
 				    "Zero byte read from file, "
 				    "skipping rest of line.");
 			break;
+		    }
+		}
 		if (cf->nextbuf != NULL) {
 		    /*
 		     * End of this buffer; return EOF and outer logic
 		     * will get the next one. (eww)
 		     */
 		    break;
+		}
 		Parse_Error(PARSE_FATAL, "Zero byte read from file");
 		return NULL;
+	    }
 	    if (ch == '\\') {
 		/* Don't treat next character as special, remember first one */
 		if (escaped == NULL)
 		    escaped = ptr;
 		if (ptr[1] == '\n')
 		    cf->lineno++;
 		ptr += 2;
 		line_end = ptr;
 		continue;
+	    }
 	    if (ch == '#' && comment == NULL) {
 		/* Remember first '#' for comment stripping */
 		/* Unless previous char was '[', as in modifier :[#] */
 		if (!(ptr > line && ptr[-1] == '['))
 		    comment = line_end;
+	    }
 	    ptr++;
 	    if (ch == '\n')
 		break;
 	    if (!ch_isspace(ch))
 		/* We are not interested in trailing whitespace */
 		line_end = ptr;
+	}
 	/* Save next 'to be processed' location */
 	cf->buf_ptr = ptr;
 	/* Check we have a non-comment, non-blank line */
 	if (line_end == line || comment == line) {
 	    if (ch == 0)
 		/* At end of file */
 		return NULL;
 	    /* Parse another line */
 	    continue;
+	}
 	/* We now have a line of data */
 	*line_end = 0;
 	if (flags & PARSE_RAW) {
 	    /* Leave '\' (etc) in line buffer (eg 'for' lines) */
 	    return line;
+	}
 	if (flags & PARSE_SKIP) {
 	    /* Completely ignore non-directives */
 	    if (line[0] != '.')
 		continue;
 	    /* We could do more of the .else/.elif/.endif checks here */
+	}
 	break;
+    }
     /* Brutally ignore anything after a non-escaped '#' in non-commands */
     if (comment != NULL && line[0] != '\t') {
 	line_end = comment;
 	*line_end = 0;
+    }
     /* If we didn't see a '\\' then the in-situ data is fine */
     if (escaped == NULL)
 	return line;
     /* Remove escapes from '\n' and '#' */
     tp = ptr = escaped;
     escaped = line;
     for (; ; *tp++ = ch) {
 	ch = *ptr++;
 	if (ch != '\\') {
 	    if (ch == 0)
 		break;
 	    continue;
+	}
 	ch = *ptr++;
 	if (ch == 0) {
 	    /* Delete '\\' at end of buffer */
 	    tp--;
 	    break;
+	}
 	if (ch == '#' && line[0] != '\t')
 	    /* Delete '\\' from before '#' on non-command lines */
 	    continue;
 	if (ch != '\n') {
 	    /* Leave '\\' in buffer for later */
 	    *tp++ = '\\';
 	    /* Make sure we don't delete an escaped ' ' from the line end */
 	    escaped = tp + 1;
 	    continue;
+	}
 	/* Escaped '\n' replace following whitespace with a single ' ' */
 	while (ptr[0] == ' ' || ptr[0] == '\t')
 	    ptr++;
 	ch = ' ';
+    }
     /* Delete any trailing spaces - eg from empty continuations */
     while (tp > escaped && ch_isspace(tp[-1]))
 	tp--;
     *tp = 0;
     return line;
+}
 /* Read an entire line from the input file. Called only by Parse_File.
+ *
  * Results:
  *	A line without its newline.
+ *
  * Side Effects:
  *	Only those associated with reading a character
  */
 static char *
 ParseReadLine(void)
+{
     char *line;			/* Result */
     int lineno;			/* Saved line # */
     int rval;
     for (;;) {
 	line = ParseGetLine(0);
 	if (line == NULL)
 	    return NULL;
 	if (line[0] != '.')
 	    return line;
 	/*
 	 * The line might be a conditional. Ask the conditional module
 	 * about it and act accordingly
 	 */
 	switch (Cond_EvalLine(line)) {
 	case COND_SKIP:
 	    /* Skip to next conditional that evaluates to COND_PARSE.  */
 	    do {
 		line = ParseGetLine(PARSE_SKIP);
 	    } while (line && Cond_EvalLine(line) != COND_PARSE);
 	    if (line == NULL)
 		break;
 	    continue;
 	case COND_PARSE:
 	    continue;
 	case COND_INVALID:    /* Not a conditional line */
 	    /* Check for .for loops */
 	    rval = For_Eval(line);
 	    if (rval == 0)
 		/* Not a .for line */
 		break;
 	    if (rval < 0)
 		/* Syntax error - error printed, ignore line */
 		continue;
 	    /* Start of a .for loop */
 	    lineno = CurFile()->lineno;
 	    /* Accumulate loop lines until matching .endfor */
 	    do {
 		line = ParseGetLine(PARSE_RAW);
 		if (line == NULL) {
 		    Parse_Error(PARSE_FATAL,
 				"Unexpected end of file in for loop.");
 		    break;
+		}
 	    } while (For_Accum(line));
 	    /* Stash each iteration as a new 'input file' */
 	    For_Run(lineno);
 	    /* Read next line from for-loop buffer */
 	    continue;
+	}
 	return line;
+    }
+}
 static void
 FinishDependencyGroup(void)
+{
     if (targets != NULL) {
 	GNodeListNode *ln;
 	for (ln = targets->first; ln != NULL; ln = ln->next) {
 	    GNode *gn = ln->datum;
 	    Suff_EndTransform(gn);
 	    /* Mark the target as already having commands if it does, to
 	     * keep from having shell commands on multiple dependency lines. */
 	    if (!Lst_IsEmpty(gn->commands))
 		gn->type |= OP_HAS_COMMANDS;
+	}
 	Lst_Free(targets);
 	targets = NULL;
+    }
+}
 /* Add the command to each target from the current dependency spec. */
 static void
 ParseLine_ShellCommand(const char *p)
+{
     cpp_skip_whitespace(&p);
     if (*p == '\0')
 	return;			/* skip empty commands */
     if (targets == NULL) {
 	Parse_Error(PARSE_FATAL, "Unassociated shell command \"%s\"", p);
 	return;
+    }
+    {
 	char *cmd = bmake_strdup(p);
 	GNodeListNode *ln;
 	for (ln = targets->first; ln != NULL; ln = ln->next) {
 	    GNode *gn = ln->datum;
 	    ParseAddCmd(gn, cmd);
+	}
 #ifdef CLEANUP
 	Lst_Append(targCmds, cmd);
 #endif
+    }
+}
 static Boolean
 ParseDirective(char *line)
+{
     char *cp;
     if (*line == '.') {
 	/*
 	 * Lines that begin with the special character may be
 	 * include or undef directives.
 	 * On the other hand they can be suffix rules (.c.o: ...)
 	 * or just dependencies for filenames that start '.'.
 	 */
 	cp = line + 1;
 	pp_skip_whitespace(&cp);
 	if (IsInclude(cp, FALSE)) {
 	    ParseDoInclude(cp);
 	    return TRUE;
+	}
 	if (strncmp(cp, "undef", 5) == 0) {
 	    const char *varname;
 	    cp += 5;
 	    pp_skip_whitespace(&cp);
 	    varname = cp;
 	    for (; !ch_isspace(*cp) && *cp != '\0'; cp++)
 		continue;
 	    *cp = '\0';
 	    Var_Delete(varname, VAR_GLOBAL);
 	    /* TODO: undefine all variables, not only the first */
 	    /* TODO: use Str_Words, like everywhere else */
 	    return TRUE;
 	} else if (strncmp(cp, "export", 6) == 0) {
 	    cp += 6;
 	    pp_skip_whitespace(&cp);
 	    Var_Export(cp, TRUE);
 	    return TRUE;
 	} else if (strncmp(cp, "unexport", 8) == 0) {
 	    Var_UnExport(cp);
 	    return TRUE;
 	} else if (strncmp(cp, "info", 4) == 0 ||
 		   strncmp(cp, "error", 5) == 0 ||
 		   strncmp(cp, "warning", 7) == 0) {
 	    if (ParseMessage(cp))
 		return TRUE;
+	}
+    }
     return FALSE;
+}
 static Boolean
 ParseVarassign(const char *line)
+{
     VarAssign var;
     if (Parse_IsVar(line, &var)) {
 	FinishDependencyGroup();
 	Parse_DoVar(&var, VAR_GLOBAL);
 	return TRUE;
+    }
     return FALSE;
+}
 static char *
 FindSemicolon(char *p)
+{
     int level = 0;
     for (; *p != '\0'; p++) {
 	if (*p == '\\' && p[1] != '\0') {
 	    p++;
 	    continue;
+	}
 	if (*p == '$' && (p[1] == '(' || p[1] == '{')) {
 	    level++;
 	    continue;
+	}
 	if (level > 0 && (*p == ')' || *p == '}')) {
 	    level--;
 	    continue;
+	}
 	if (level == 0 && *p == ';') {
 	    break;
+	}
+    }
     return p;
+}
 /* dependency	-> target... op [source...]
  * op		-> ':' | '::' | '!' */
 static void
 ParseDependency(char *line)
+{
     VarEvalFlags eflags;
     char *expanded_line;
     const char *shellcmd = NULL;
     /*
      * For some reason - probably to make the parser impossible -
      * a ';' can be used to separate commands from dependencies.
      * Attempt to avoid ';' inside substitution patterns.
      */
+    {
         char *semicolon = FindSemicolon(line);
 	if (*semicolon != '\0') {
 	    /* Terminate the dependency list at the ';' */
 	    *semicolon = '\0';
 	    shellcmd = semicolon + 1;
+	}
+    }
     /*
      * We now know it's a dependency line so it needs to have all
      * variables expanded before being parsed.
+     *
      * XXX: Ideally the dependency line would first be split into
      * its left-hand side, dependency operator and right-hand side,
      * and then each side would be expanded on its own.  This would
      * allow for the left-hand side to allow only defined variables
      * and to allow variables on the right-hand side to be undefined
      * as well.
+     *
      * Parsing the line first would also prevent that targets
      * generated from variable expressions are interpreted as the
      * dependency operator, such as in "target${:U:} middle: source",
      * in which the middle is interpreted as a source, not a target.
      */
     /* In lint mode, allow undefined variables to appear in
      * dependency lines.
+     *
      * Ideally, only the right-hand side would allow undefined
      * variables since it is common to have no dependencies.
      * Having undefined variables on the left-hand side is more
      * unusual though.  Since both sides are expanded in a single
      * pass, there is not much choice what to do here.
+     *
      * In normal mode, it does not matter whether undefined
      * variables are allowed or not since as of 2020-09-14,
      * Var_Parse does not print any parse errors in such a case.
      * It simply returns the special empty string var_Error,
      * which cannot be detected in the result of Var_Subst. */
     eflags = DEBUG(LINT) ? VARE_WANTRES : VARE_UNDEFERR | VARE_WANTRES;
     (void)Var_Subst(line, VAR_CMD, eflags, &expanded_line);
     /* TODO: handle errors */
     /* Need a fresh list for the target nodes */
     if (targets != NULL)
 	Lst_Free(targets);
     targets = Lst_New();
     ParseDoDependency(expanded_line);
     free(expanded_line);
     if (shellcmd != NULL)
 	ParseLine_ShellCommand(shellcmd);
+}
 static void
 ParseLine(char *line)
+{
     if (ParseDirective(line))
 	return;
     if (*line == '\t') {
 	ParseLine_ShellCommand(line + 1);
 	return;
+    }
 #ifdef SYSVINCLUDE
     if (IsSysVInclude(line)) {
 	/*
 	 * It's an S3/S5-style "include".
 	 */
 	ParseTraditionalInclude(line);
 	return;
+    }
 #endif
 #ifdef GMAKEEXPORT
     if (strncmp(line, "export", 6) == 0 && ch_isspace(line[6]) &&
 	strchr(line, ':') == NULL) {
 	/*
 	 * It's a Gmake "export".
 	 */
 	ParseGmakeExport(line);
 	return;
+    }
 #endif
     if (ParseVarassign(line))
 	return;
     FinishDependencyGroup();
     ParseDependency(line);
+}
 /* Parse a top-level makefile into its component parts, incorporating them
  * into the global dependency graph.
+ *
  * Input:
  *	name		The name of the file being read
  *	fd		The open file to parse; will be closed at the end
  */
 void
 Parse_File(const char *name, int fd)
+{
     char *line;			/* the line we're working on */
     struct loadedfile *lf;
     lf = loadfile(name, fd);
     assert(targets == NULL);
     fatals = 0;
     if (name == NULL)
 	name = "(stdin)";
     Parse_SetInput(name, 0, -1, loadedfile_nextbuf, lf);
     CurFile()->lf = lf;
     do {
 	while ((line = ParseReadLine()) != NULL) {
 	    DEBUG2(PARSE, "ParseReadLine (%d): '%s'\n",
 		   CurFile()->lineno, line);
 	    ParseLine(line);
+	}
 	/*
 	 * Reached EOF, but it may be just EOF of an include file...
 	 */
     } while (ParseEOF());
     FinishDependencyGroup();
     if (fatals) {
 	(void)fflush(stdout);
 	(void)fprintf(stderr,
 		      "%s: Fatal errors encountered -- cannot continue",
 		      progname);
 	PrintOnError(NULL, NULL);
 	exit(1);
+    }
+}
 /* Initialize the parsing module. */
 void
 Parse_Init(void)
+{
     mainNode = NULL;
     parseIncPath = Lst_New();
     sysIncPath = Lst_New();
-    defIncPath = Lst_New();
+    defSysIncPath = Lst_New();
     Vector_Init(&includes, sizeof(IFile));
 #ifdef CLEANUP
     targCmds = Lst_New();
 #endif
+}
 /* Clean up the parsing module. */
 void
 Parse_End(void)
+{
 #ifdef CLEANUP
     Lst_Destroy(targCmds, free);
     assert(targets == NULL);
-    Lst_Destroy(defIncPath, Dir_Destroy);
+    Lst_Destroy(defSysIncPath, Dir_Destroy);
     Lst_Destroy(sysIncPath, Dir_Destroy);
     Lst_Destroy(parseIncPath, Dir_Destroy);
     assert(includes.len == 0);
     Vector_Done(&includes);
 #endif
+}
 /*-
  *-----------------------------------------------------------------------
  * Parse_MainName --
  *	Return a Lst of the main target to create for main()'s sake. If
  *	no such target exists, we Punt with an obnoxious error message.
+ *
  * Results:
  *	A Lst of the single node to create.
+ *
  * Side Effects:
  *	None.
+ *
  *-----------------------------------------------------------------------
  */
 GNodeList *
 Parse_MainName(void)
+{
     GNodeList *mainList;
     mainList = Lst_New();
     if (mainNode == NULL) {
 	Punt("no target to make.");
 	/*NOTREACHED*/
     } else if (mainNode->type & OP_DOUBLEDEP) {
 	Lst_Append(mainList, mainNode);
 	Lst_AppendAll(mainList, mainNode->cohorts);
     } else
 	Lst_Append(mainList, mainNode);
     Var_Append(".TARGETS", mainNode->name, VAR_GLOBAL);
     return mainList;
+}

 @@ -1,711 +1,711 @@
-/*	$NetBSD: make.h,v 1.174 2020/10/27 07:03:55 rillig Exp $	*/
+/*	$NetBSD: make.h,v 1.175 2020/10/28 03:21:25 rillig Exp $	*/
 /*
  * Copyright (c) 1988, 1989, 1990, 1993
  *	The Regents of the University of California.  All rights reserved.
+ *
  * This code is derived from software contributed to Berkeley by
  * Adam de Boor.
+ *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
  * 3. Neither the name of the University nor the names of its contributors
  *    may be used to endorse or promote products derived from this software
  *    without specific prior written permission.
+ *
  * THIS SOFTWARE IS PROVIDED BY THE REGENTS AND CONTRIBUTORS ``AS IS'' AND
  * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
  * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
  * ARE DISCLAIMED.  IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE
  * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
  * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
  * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
  * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
  * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
  * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
  * SUCH DAMAGE.
+ *
  *	from: @(#)make.h	8.3 (Berkeley) 6/13/95
  */
 /*
  * Copyright (c) 1989 by Berkeley Softworks
  * All rights reserved.
+ *
  * This code is derived from software contributed to Berkeley by
  * Adam de Boor.
+ *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
  * 3. All advertising materials mentioning features or use of this software
  *    must display the following acknowledgement:
  *	This product includes software developed by the University of
  *	California, Berkeley and its contributors.
  * 4. Neither the name of the University nor the names of its contributors
  *    may be used to endorse or promote products derived from this software
  *    without specific prior written permission.
+ *
  * THIS SOFTWARE IS PROVIDED BY THE REGENTS AND CONTRIBUTORS ``AS IS'' AND
  * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
  * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
  * ARE DISCLAIMED.  IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE
  * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
  * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
  * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
  * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
  * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
  * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
  * SUCH DAMAGE.
+ *
  *	from: @(#)make.h	8.3 (Berkeley) 6/13/95
  */
 /*-
  * make.h --
  *	The global definitions for pmake
  */
 #ifndef MAKE_MAKE_H
 #define MAKE_MAKE_H
 #include <sys/types.h>
 #include <sys/param.h>
 #include <sys/stat.h>
 #include <assert.h>
 #include <ctype.h>
 #include <fcntl.h>
 #include <stdarg.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 #include <unistd.h>
 #ifdef BSD4_4
 # include <sys/cdefs.h>
 #endif
 #ifndef FD_CLOEXEC
 #define FD_CLOEXEC 1
 #endif
 #if defined(__GNUC__)
 #define	MAKE_GNUC_PREREQ(x, y)						\
 	((__GNUC__ == (x) && __GNUC_MINOR__ >= (y)) ||			\
 	 (__GNUC__ > (x)))
 #else /* defined(__GNUC__) */
 #define	MAKE_GNUC_PREREQ(x, y)	0
 #endif /* defined(__GNUC__) */
 #if MAKE_GNUC_PREREQ(2, 7)
 #define	MAKE_ATTR_UNUSED	__attribute__((__unused__))
 #else
 #define	MAKE_ATTR_UNUSED	/* delete */
 #endif
 #if MAKE_GNUC_PREREQ(2, 5)
 #define	MAKE_ATTR_DEAD		__attribute__((__noreturn__))
 #elif defined(__GNUC__)
 #define	MAKE_ATTR_DEAD		__volatile
 #else
 #define	MAKE_ATTR_DEAD		/* delete */
 #endif
 #if MAKE_GNUC_PREREQ(2, 7)
 #define MAKE_ATTR_PRINTFLIKE(fmtarg, firstvararg)	\
 	    __attribute__((__format__ (__printf__, fmtarg, firstvararg)))
 #else
 #define MAKE_ATTR_PRINTFLIKE(fmtarg, firstvararg)	/* delete */
 #endif
 /*
  * A boolean type is defined as an integer, not an enum, for historic reasons.
  * The only allowed values are the constants TRUE and FALSE (1 and 0).
  */
 #ifdef USE_DOUBLE_BOOLEAN
 /* During development, to find type mismatches in function declarations. */
 typedef double Boolean;
 #define TRUE 1.0
 #define FALSE 0.0
 #elif defined(USE_UCHAR_BOOLEAN)
 /* During development, to find code that depends on the exact value of TRUE or
  * that stores other values in Boolean variables. */
 typedef unsigned char Boolean;
 #define TRUE ((unsigned char)0xFF)
 #define FALSE ((unsigned char)0x00)
 #elif defined(USE_CHAR_BOOLEAN)
 /* During development, to find code that uses a boolean as array index, via
  * -Wchar-subscripts. */
 typedef char Boolean;
 #define TRUE ((char)-1)
 #define FALSE ((char)0x00)
 #elif defined(USE_ENUM_BOOLEAN)
 typedef enum Boolean { FALSE, TRUE } Boolean;
 #else
 typedef int Boolean;
 #ifndef TRUE
 #define TRUE	1
 #endif
 #ifndef FALSE
 #define FALSE	0
 #endif
 #endif
 #include "lst.h"
 #include "enum.h"
 #include "hash.h"
 #include "config.h"
 #include "buf.h"
 #include "make_malloc.h"
 typedef enum  {
     UNMADE,			/* Not examined yet */
     DEFERRED,			/* Examined once (building child) */
     REQUESTED,			/* on toBeMade list */
     BEINGMADE,			/* Target is already being made.
 				 * Indicates a cycle in the graph. */
     MADE,			/* Was out-of-date and has been made */
     UPTODATE,			/* Was already up-to-date */
     ERROR,			/* An error occurred while it was being
 				 * made (used only in compat mode) */
     ABORTED			/* The target was aborted due to an error
 				 * making an inferior (compat). */
 } GNodeMade;
 /* The OP_ constants are used when parsing a dependency line as a way of
  * communicating to other parts of the program the way in which a target
  * should be made.
+ *
  * Some of the OP_ constants can be combined, others cannot. */
 typedef enum GNodeType {
     /* The dependency operator ':' is the most common one.  The commands of
      * this node are executed if any child is out-of-date. */
     OP_DEPENDS		= 1 << 0,
     /* The dependency operator '!' always executes its commands, even if
      * its children are up-to-date. */
     OP_FORCE		= 1 << 1,
     /* The dependency operator '::' behaves like ':', except that it allows
      * multiple dependency groups to be defined.  Each of these groups is
      * executed on its own, independently from the others. */
     OP_DOUBLEDEP	= 1 << 2,
     /* Matches the dependency operators ':', '!' and '::'. */
     OP_OPMASK		= OP_DEPENDS|OP_FORCE|OP_DOUBLEDEP,
     /* Don't care if the target doesn't exist and can't be created */
     OP_OPTIONAL		= 1 << 3,
     /* Use associated commands for parents */
     OP_USE		= 1 << 4,
     /* Target is never out of date, but always execute commands anyway.
      * Its time doesn't matter, so it has none...sort of */
     OP_EXEC		= 1 << 5,
     /* Ignore non-zero exit status from shell commands when creating the node */
     OP_IGNORE		= 1 << 6,
     /* Don't remove the target when interrupted */
     OP_PRECIOUS		= 1 << 7,
     /* Don't echo commands when executed */
     OP_SILENT		= 1 << 8,
     /* Target is a recursive make so its commands should always be executed
      * when it is out of date, regardless of the state of the -n or -t flags */
     OP_MAKE		= 1 << 9,
     /* Target is out-of-date only if any of its children was out-of-date */
     OP_JOIN		= 1 << 10,
     /* Assume the children of the node have been already made */
     OP_MADE		= 1 << 11,
     /* Special .BEGIN, .END, .INTERRUPT */
     OP_SPECIAL		= 1 << 12,
     /* Like .USE, only prepend commands */
     OP_USEBEFORE	= 1 << 13,
     /* The node is invisible to its parents. I.e. it doesn't show up in the
      * parents' local variables. */
     OP_INVISIBLE	= 1 << 14,
     /* The node is exempt from normal 'main target' processing in parse.c */
     OP_NOTMAIN		= 1 << 15,
     /* Not a file target; run always */
     OP_PHONY		= 1 << 16,
     /* Don't search for file in the path */
     OP_NOPATH		= 1 << 17,
     /* .WAIT phony node */
     OP_WAIT		= 1 << 18,
     /* .NOMETA do not create a .meta file */
     OP_NOMETA		= 1 << 19,
     /* .META we _do_ want a .meta file */
     OP_META		= 1 << 20,
     /* Do not compare commands in .meta file */
     OP_NOMETA_CMP	= 1 << 21,
     /* Possibly a submake node */
     OP_SUBMAKE		= 1 << 22,
     /* Attributes applied by PMake */
     /* The node is a transformation rule */
     OP_TRANSFORM	= 1 << 31,
     /* Target is a member of an archive */
     /* XXX: How does this differ from OP_ARCHV? */
     OP_MEMBER		= 1 << 30,
     /* The node is a library,
      * its name has the form "-l<libname>" */
     OP_LIB		= 1 << 29,
     /* The node is an archive member,
      * its name has the form "archive(member)" */
     /* XXX: How does this differ from OP_MEMBER? */
     OP_ARCHV		= 1 << 28,
     /* Target has all the commands it should. Used when parsing to catch
      * multiple command groups for a target.  Only applies to the dependency
      * operators ':' and '!', but not to '::'. */
     OP_HAS_COMMANDS	= 1 << 27,
     /* The special command "..." has been seen. All further commands from
      * this node will be saved on the .END node instead, to be executed at
      * the very end. */
     OP_SAVE_CMDS	= 1 << 26,
     /* Already processed by Suff_FindDeps */
     OP_DEPS_FOUND	= 1 << 25,
     /* Node found while expanding .ALLSRC */
     OP_MARK		= 1 << 24,
     OP_NOTARGET		= OP_NOTMAIN | OP_USE | OP_EXEC | OP_TRANSFORM
 } GNodeType;
 typedef enum GNodeFlags {
     REMAKE	= 0x0001,	/* this target needs to be (re)made */
     CHILDMADE	= 0x0002,	/* children of this target were made */
     FORCE	= 0x0004,	/* children don't exist, and we pretend made */
     DONE_WAIT	= 0x0008,	/* Set by Make_ProcessWait() */
     DONE_ORDER	= 0x0010,	/* Build requested by .ORDER processing */
     FROM_DEPEND	= 0x0020,	/* Node created from .depend */
     DONE_ALLSRC	= 0x0040,	/* We do it once only */
     CYCLE	= 0x1000,	/* Used by MakePrintStatus */
     DONECYCLE	= 0x2000,	/* Used by MakePrintStatus */
     INTERNAL	= 0x4000	/* Internal use only */
 } GNodeFlags;
 typedef struct List StringList;
 typedef struct ListNode StringListNode;
 typedef struct List GNodeList;
 typedef struct ListNode GNodeListNode;
 typedef struct List /* of CachedDir */ SearchPath;
 /* A graph node represents a target that can possibly be made, including its
  * relation to other targets and a lot of other details. */
 typedef struct GNode {
     /* The target's name, such as "clean" or "make.c" */
     char *name;
     /* The unexpanded name of a .USE node */
     char *uname;
     /* The full pathname of the file belonging to the target.
      * XXX: What about .PHONY targets? These don't have an associated path. */
     char *path;
     /* The type of operator used to define the sources (see the OP flags below).
      * XXX: This looks like a wild mixture of type and flags. */
     GNodeType type;
     GNodeFlags flags;
     /* The state of processing on this node */
     GNodeMade made;
     int unmade;			/* The number of unmade children */
     /* The modification time; 0 means the node does not have a corresponding
      * file; see Make_OODate. */
     time_t mtime;
     struct GNode *youngestChild;
     /* The GNodes for which this node is an implied source. May be empty.
      * For example, when there is an inference rule for .c.o, the node for
      * file.c has the node for file.o in this list. */
     GNodeList *implicitParents;
     /* Other nodes of the same name, for the '::' operator. */
     GNodeList *cohorts;
     /* The nodes that depend on this one, or in other words, the nodes for
      * which this is a source. */
     GNodeList *parents;
     /* The nodes on which this one depends. */
     GNodeList *children;
     /* .ORDER nodes we need made. The nodes that must be made (if they're
      * made) before this node can be made, but that do not enter into the
      * datedness of this node. */
     GNodeList *order_pred;
     /* .ORDER nodes who need us. The nodes that must be made (if they're made
      * at all) after this node is made, but that do not depend on this node,
      * in the normal sense. */
     GNodeList *order_succ;
     /* The "#n" suffix for this cohort, or "" for other nodes */
     char cohort_num[8];
     /* The number of unmade instances on the cohorts list */
     int unmade_cohorts;
     /* Pointer to the first instance of a '::' node; only set when on a
      * cohorts list */
     struct GNode *centurion;
     /* Last time (sequence number) we tried to make this node */
     unsigned int checked_seqno;
     /* The "local" variables that are specific to this target and this target
      * only, such as $@, $<, $?.
+     *
      * Also used for the global variable scopes VAR_GLOBAL, VAR_CMD,
      * VAR_INTERNAL, which contain variables with arbitrary names. */
     HashTable context;
     /* The commands to be given to a shell to create this target. */
     StringList *commands;
     /* Suffix for the node (determined by Suff_FindDeps and opaque to everyone
      * but the Suff module) */
     struct Suff *suffix;
     /* filename where the GNode got defined */
     const char *fname;
     /* line number where the GNode got defined */
     int lineno;
 } GNode;
 /*
  * Error levels for parsing. PARSE_FATAL means the process cannot continue
  * once the top-level makefile has been parsed. PARSE_WARNING and PARSE_INFO
  * mean it can.
  */
 typedef enum ParseErrorLevel {
     PARSE_FATAL = 1,
     PARSE_WARNING,
     PARSE_INFO
 } ParseErrorLevel;
 /*
  * Values returned by Cond_EvalLine and Cond_EvalCondition.
  */
 typedef enum CondEvalResult {
     COND_PARSE,			/* Parse the next lines */
     COND_SKIP,			/* Skip the next lines */
     COND_INVALID		/* Not a conditional statement */
 } CondEvalResult;
 /*
  * Definitions for the "local" variables. Used only for clarity.
  */
 #define TARGET		"@"	/* Target of dependency */
 #define OODATE		"?"	/* All out-of-date sources */
 #define ALLSRC		">"	/* All sources */
 #define IMPSRC		"<"	/* Source implied by transformation */
 #define PREFIX		"*"	/* Common prefix */
 #define ARCHIVE		"!"	/* Archive in "archive(member)" syntax */
 #define MEMBER		"%"	/* Member in "archive(member)" syntax */
 #define FTARGET		"@F"	/* file part of TARGET */
 #define DTARGET		"@D"	/* directory part of TARGET */
 #define FIMPSRC		"<F"	/* file part of IMPSRC */
 #define DIMPSRC		"<D"	/* directory part of IMPSRC */
 #define FPREFIX		"*F"	/* file part of PREFIX */
 #define DPREFIX		"*D"	/* directory part of PREFIX */
 /*
  * Global Variables
  */
 extern SearchPath *dirSearchPath;
 				/* The list of directories to search when
 				 * looking for targets */
 extern Boolean  allPrecious;	/* True if every target is precious */
 extern Boolean  deleteOnError;	/* True if failed targets should be deleted */
 extern Boolean	doing_depend;	/* TRUE if processing .depend */
 extern GNode    *DEFAULT;	/* .DEFAULT rule */
 extern GNode	*VAR_INTERNAL;	/* Variables defined internally by make
 				 * which should not override those set by
 				 * makefiles.
 				 */
 extern GNode    *VAR_GLOBAL;	/* Variables defined in a global context, e.g
 				 * in the Makefile itself */
 extern GNode    *VAR_CMD;	/* Variables defined on the command line */
 extern char	var_Error[];	/* Value returned by Var_Parse when an error
 				 * is encountered. It actually points to
 				 * an empty string, so naive callers needn't
 				 * worry about it. */
 extern time_t	now;		/* The time at the start of this whole
 				 * process */
 extern Boolean	oldVars;	/* Do old-style variable substitution */
 extern SearchPath *sysIncPath;	/* The system include path. */
-extern SearchPath *defIncPath;	/* The default include path. */
+extern SearchPath *defSysIncPath; /* The default system include path. */
 extern char	curdir[];	/* Startup directory */
 extern char	*progname;	/* The program name */
 extern char	*makeDependfile; /* .depend */
 extern char	**savedEnv;	 /* if we replaced environ this will be non-NULL */
 extern int	makelevel;
 /*
  * We cannot vfork() in a child of vfork().
  * Most systems do not enforce this but some do.
  */
 #define vFork() ((getpid() == myPid) ? vfork() : fork())
 extern pid_t	myPid;
 #define	MAKEFLAGS	".MAKEFLAGS"
 #define	MAKEOVERRIDES	".MAKEOVERRIDES"
 #define	MAKE_JOB_PREFIX	".MAKE.JOB.PREFIX" /* prefix for job target output */
 #define	MAKE_EXPORTED	".MAKE.EXPORTED"   /* variables we export */
 #define	MAKE_MAKEFILES	".MAKE.MAKEFILES"  /* all makefiles already loaded */
 #define	MAKE_LEVEL	".MAKE.LEVEL"	   /* recursion level */
 #define MAKEFILE_PREFERENCE ".MAKE.MAKEFILE_PREFERENCE"
 #define MAKE_DEPENDFILE	".MAKE.DEPENDFILE" /* .depend */
 #define MAKE_MODE	".MAKE.MODE"
 #ifndef MAKE_LEVEL_ENV
 # define MAKE_LEVEL_ENV	"MAKELEVEL"
 #endif
 typedef enum DebugFlags {
     DEBUG_ARCH		= 1 << 0,
     DEBUG_COND		= 1 << 1,
     DEBUG_DIR		= 1 << 2,
     DEBUG_GRAPH1	= 1 << 3,
     DEBUG_GRAPH2	= 1 << 4,
     DEBUG_JOB		= 1 << 5,
     DEBUG_MAKE		= 1 << 6,
     DEBUG_SUFF		= 1 << 7,
     DEBUG_TARG		= 1 << 8,
     DEBUG_VAR		= 1 << 9,
     DEBUG_FOR		= 1 << 10,
     DEBUG_SHELL		= 1 << 11,
     DEBUG_ERROR		= 1 << 12,
     DEBUG_LOUD		= 1 << 13,
     DEBUG_META		= 1 << 14,
     DEBUG_HASH		= 1 << 15,
     DEBUG_GRAPH3	= 1 << 16,
     DEBUG_SCRIPT	= 1 << 17,
     DEBUG_PARSE		= 1 << 18,
     DEBUG_CWD		= 1 << 19,
     DEBUG_LINT		= 1 << 20
 } DebugFlags;
 #define CONCAT(a,b)	a##b
 #define	DEBUG(module)	(opts.debug & CONCAT(DEBUG_,module))
 void debug_printf(const char *, ...) MAKE_ATTR_PRINTFLIKE(1, 2);
 #define DEBUG0(module, text) \
     if (!DEBUG(module)) (void)0; \
     else debug_printf("%s", text)
 #define DEBUG1(module, fmt, arg1) \
     if (!DEBUG(module)) (void)0; \
     else debug_printf(fmt, arg1)
 #define DEBUG2(module, fmt, arg1, arg2) \
     if (!DEBUG(module)) (void)0; \
     else debug_printf(fmt, arg1, arg2)
 #define DEBUG3(module, fmt, arg1, arg2, arg3) \
     if (!DEBUG(module)) (void)0; \
     else debug_printf(fmt, arg1, arg2, arg3)
 #define DEBUG4(module, fmt, arg1, arg2, arg3, arg4) \
     if (!DEBUG(module)) (void)0; \
     else debug_printf(fmt, arg1, arg2, arg3, arg4)
 #define DEBUG5(module, fmt, arg1, arg2, arg3, arg4, arg5) \
     if (!DEBUG(module)) (void)0; \
     else debug_printf(fmt, arg1, arg2, arg3, arg4, arg5)
 typedef enum PrintVarsMode {
     COMPAT_VARS = 1,
     EXPAND_VARS
 } PrintVarsMode;
 /* Command line options */
 typedef struct CmdOpts {
     /* -B: whether we are make compatible */
     Boolean compatMake;
     /* -d: debug control: There is one bit per module.  It is up to the
      * module what debug information to print. */
     DebugFlags debug;
     /* -df: debug output is written here - default stderr */
     FILE *debug_file;
     /* -dV: for the -V option, print unexpanded variable values */
     Boolean debugVflag;
     /* -e: check environment variables before global variables */
     Boolean checkEnvFirst;
     /* -f: the makefiles to read */
     StringList *makefiles;
     /* -i: if true, ignore all errors from shell commands */
     Boolean ignoreErrors;
     /* -j: the maximum number of jobs that can run in parallel;
      * this is coordinated with the submakes */
     int maxJobs;
     /* -k: if true, continue on unaffected portions of the graph when an
      * error occurs in one portion */
     Boolean keepgoing;
     /* -N: execute no commands from the targets */
     Boolean noRecursiveExecute;
     /* -n: execute almost no commands from the targets */
     Boolean noExecute;
     /* -q: if true, we aren't supposed to really make anything, just see if
      * the targets are out-of-date */
     Boolean queryFlag;
     /* -r: raw mode, without loading the builtin rules. */
     Boolean noBuiltins;
     /* -s: don't echo the shell commands before executing them */
     Boolean beSilent;
     /* -t: touch the targets if they are out-of-date, but don't actually
      * make them */
     Boolean touchFlag;
     /* -[Vv]: print expanded or unexpanded selected variables */
     PrintVarsMode printVars;
     /* -[Vv]: the variables to print */
     StringList *variables;
     /* -W: if true, makefile parsing warnings are treated as errors */
     Boolean parseWarnFatal;
     /* -w: print Entering and Leaving for submakes */
     Boolean enterFlag;
     /* -X: if true, do not export variables set on the command line to the
      * environment. */
     Boolean varNoExportEnv;
     /* The target names specified on the command line.
      * Used to resolve .if make(...) statements. */
     StringList *create;
 } CmdOpts;
 extern CmdOpts opts;
 #include "nonints.h"
 void Make_TimeStamp(GNode *, GNode *);
 Boolean Make_OODate(GNode *);
 void Make_ExpandUse(GNodeList *);
 time_t Make_Recheck(GNode *);
 void Make_HandleUse(GNode *, GNode *);
 void Make_Update(GNode *);
 void Make_DoAllVar(GNode *);
 Boolean Make_Run(GNodeList *);
 int dieQuietly(GNode *, int);
 void PrintOnError(GNode *, const char *);
 void Main_ExportMAKEFLAGS(Boolean);
 Boolean Main_SetObjdir(const char *, ...) MAKE_ATTR_PRINTFLIKE(1, 2);
 int mkTempFile(const char *, char **);
 int str2Lst_Append(StringList *, char *, const char *);
 void GNode_FprintDetails(FILE *, const char *, const GNode *, const char *);
 Boolean NoExecute(GNode *gn);
 /* See if the node was seen on the left-hand side of a dependency operator. */
 static MAKE_ATTR_UNUSED Boolean
 GNode_IsTarget(const GNode *gn)
+{
     return (gn->type & OP_OPMASK) != 0;
+}
 static MAKE_ATTR_UNUSED const char *
 GNode_Path(const GNode *gn)
+{
     return gn->path != NULL ? gn->path : gn->name;
+}
 #ifdef __GNUC__
 #define UNCONST(ptr)	({		\
     union __unconst {			\
 	const void *__cp;		\
 	void *__p;			\
     } __d;				\
     __d.__cp = ptr, __d.__p; })
 #else
 #define UNCONST(ptr)	(void *)(ptr)
 #endif
 /* At least GNU/Hurd systems lack hardcoded MAXPATHLEN/PATH_MAX */
 #include <limits.h>
 #ifndef MAXPATHLEN
 #define MAXPATHLEN	4096
 #endif
 #ifndef PATH_MAX
 #define PATH_MAX	MAXPATHLEN
 #endif
 #if defined(SYSV)
 #define KILLPG(pid, sig)	kill(-(pid), (sig))
 #else
 #define KILLPG(pid, sig)	killpg((pid), (sig))
 #endif
 static inline MAKE_ATTR_UNUSED Boolean ch_isalnum(char ch)
 { return isalnum((unsigned char)ch) != 0; }
 static inline MAKE_ATTR_UNUSED Boolean ch_isalpha(char ch)
 { return isalpha((unsigned char)ch) != 0; }
 static inline MAKE_ATTR_UNUSED Boolean ch_isdigit(char ch)
 { return isdigit((unsigned char)ch) != 0; }
 static inline MAKE_ATTR_UNUSED Boolean ch_isspace(char ch)
 { return isspace((unsigned char)ch) != 0; }
 static inline MAKE_ATTR_UNUSED Boolean ch_isupper(char ch)
 { return isupper((unsigned char)ch) != 0; }
 static inline MAKE_ATTR_UNUSED char ch_tolower(char ch)
 { return (char)tolower((unsigned char)ch); }
 static inline MAKE_ATTR_UNUSED char ch_toupper(char ch)
 { return (char)toupper((unsigned char)ch); }
 static inline MAKE_ATTR_UNUSED void
 cpp_skip_whitespace(const char **pp)
+{
     while (ch_isspace(**pp))
 	(*pp)++;
+}
 static inline MAKE_ATTR_UNUSED void
 pp_skip_whitespace(char **pp)
+{
     while (ch_isspace(**pp))
 	(*pp)++;
+}
 #ifdef MAKE_NATIVE
 #  include <sys/cdefs.h>
 #  ifndef lint
 #    define MAKE_RCSID(id) __RCSID(id)
 #  endif
 #else
 #  define MAKE_RCSID(id) static volatile char rcsid[] = id
 #endif
 #endif /* MAKE_MAKE_H */