Module Name: src Committed By: rillig Date: Sun Jan 2 02:16:12 UTC 2022
Modified Files: src/usr.bin/make: parse.c Log Message: make: remove redundant IFile.fromForLoop No functional change. To generate a diff of this commit: cvs rdiff -u -r1.616 -r1.617 src/usr.bin/make/parse.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/usr.bin/make/parse.c diff -u src/usr.bin/make/parse.c:1.616 src/usr.bin/make/parse.c:1.617 --- src/usr.bin/make/parse.c:1.616 Sun Jan 2 01:54:43 2022 +++ src/usr.bin/make/parse.c Sun Jan 2 02:16:12 2022 @@ -1,4 +1,4 @@ -/* $NetBSD: parse.c,v 1.616 2022/01/02 01:54:43 rillig Exp $ */ +/* $NetBSD: parse.c,v 1.617 2022/01/02 02:16:12 rillig Exp $ */ /* * Copyright (c) 1988, 1989, 1990, 1993 @@ -110,7 +110,7 @@ #include "pathnames.h" /* "@(#)parse.c 8.3 (Berkeley) 3/19/94" */ -MAKE_RCSID("$NetBSD: parse.c,v 1.616 2022/01/02 01:54:43 rillig Exp $"); +MAKE_RCSID("$NetBSD: parse.c,v 1.617 2022/01/02 02:16:12 rillig Exp $"); /* types and constants */ @@ -119,8 +119,6 @@ MAKE_RCSID("$NetBSD: parse.c,v 1.616 202 */ typedef struct IFile { FStr name; /* absolute or relative to the cwd */ - /* TODO: merge with forLoop */ - bool 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 */ @@ -385,20 +383,15 @@ PrintStackTrace(void) for (i = n; i-- > 0;) { const IFile *entry = entries + i; const char *fname = entry->name.str; - bool printLineno; char dirbuf[MAXPATHLEN + 1]; if (fname[0] != '/' && strcmp(fname, "(stdin)") != 0) fname = realpath(fname, dirbuf); - printLineno = !entry->fromForLoop; - if (i + 1 < n && entries[i + 1].fromForLoop == printLineno) - printLineno = entry->fromForLoop; - - if (printLineno) + if (entries[i + 1 < n ? i + 1 : i].forLoop == NULL) debug_printf("\tin .include from %s:%d\n", fname, entry->lineno); - if (entry->fromForLoop) + if (entry->forLoop != NULL) debug_printf("\tin .for loop from %s:%d\n", fname, entry->first_lineno); } @@ -2082,7 +2075,7 @@ GetActuallyIncludingFile(void) const IFile *incs = GetInclude(0); for (i = includes.len; i >= 2; i--) - if (!incs[i - 1].fromForLoop) + if (incs[i - 1].forLoop == NULL) return incs[i - 2].name.str; return NULL; } @@ -2168,20 +2161,17 @@ Parse_PushInput(const char *name, int li struct ForLoop *forLoop) { IFile *curFile; - bool fromForLoop = name == NULL; - if (fromForLoop) + if (forLoop != NULL) name = CurFile()->name.str; else TrackInput(name); DEBUG3(PARSE, "Parse_PushInput: %s %s, line %d\n", - !fromForLoop ? "file" : ".for loop in", - name, lineno); + forLoop != NULL ? ".for loop in": "file", name, lineno); curFile = Vector_Push(&includes); curFile->name = FStr_InitOwn(bmake_strdup(name)); - curFile->fromForLoop = fromForLoop; curFile->lineno = lineno; curFile->first_lineno = lineno; curFile->buf = buf;