[src/trunk]: src/usr.bin/make make(1): replace pointer comparisons with enum

[rillig <rillig%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/98c5aec704db
branches:  trunk
changeset: 946712:98c5aec704db
user:      rillig <rillig%NetBSD.org@localhost>
date:      Sun Dec 06 16:24:30 2020 +0000

description:
make(1): replace pointer comparisons with enum

Keeping track of what it means if varname.str == str is not as
expressive as declaring what exactly to unexport.

diffstat:

 usr.bin/make/var.c |  33 ++++++++++++++++++++-------------
 1 files changed, 20 insertions(+), 13 deletions(-)

diffs (94 lines):

diff -r aae8c55ec5c7 -r 98c5aec704db usr.bin/make/var.c
--- a/usr.bin/make/var.c        Sun Dec 06 16:06:11 2020 +0000
+++ b/usr.bin/make/var.c        Sun Dec 06 16:24:30 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: var.c,v 1.712 2020/12/06 15:40:46 rillig Exp $ */
+/*     $NetBSD: var.c,v 1.713 2020/12/06 16:24:30 rillig Exp $ */
 
 /*
  * Copyright (c) 1988, 1989, 1990, 1993
@@ -130,7 +130,7 @@
 #include "metachar.h"
 
 /*     "@(#)var.c      8.3 (Berkeley) 3/19/94" */
-MAKE_RCSID("$NetBSD: var.c,v 1.712 2020/12/06 15:40:46 rillig Exp $");
+MAKE_RCSID("$NetBSD: var.c,v 1.713 2020/12/06 16:24:30 rillig Exp $");
 
 /* A string that may need to be freed after use. */
 typedef struct FStr {
@@ -233,6 +233,12 @@
        VAR_EXPORTED_ALL
 } VarExportedMode;
 
+typedef enum UnexportWhat {
+       UNEXPORT_NAMED,
+       UNEXPORT_ALL,
+       UNEXPORT_ENV
+} UnexportWhat;
+
 /* Flags for pattern matching in the :S and :C modifiers */
 typedef enum VarPatternFlags {
        VARP_NONE               = 0,
@@ -745,7 +751,7 @@
 }
 
 static void
-UnexportVar(const char *varname, Boolean unexport_env, Boolean adjust)
+UnexportVar(const char *varname, UnexportWhat what)
 {
        Var *v = VarFind(varname, VAR_GLOBAL, FALSE);
        if (v == NULL) {
@@ -754,13 +760,13 @@
        }
 
        DEBUG1(VAR, "Unexporting \"%s\"\n", varname);
-       if (!unexport_env && (v->flags & VAR_EXPORTED) &&
-           !(v->flags & VAR_REEXPORT))
+       if (what != UNEXPORT_ENV &&
+           (v->flags & VAR_EXPORTED) && !(v->flags & VAR_REEXPORT))
                unsetenv(v->name.str);
        v->flags &= ~(unsigned)(VAR_EXPORTED | VAR_REEXPORT);
 
-       /* If we are unexporting a list, remove each one from .MAKE.EXPORTED. */
-       if (adjust) {
+       if (what == UNEXPORT_NAMED) {
+               /* Remove the variable names from .MAKE.EXPORTED. */
                /* XXX: v->name is injected without escaping it */
                char *expr = str_concat3("${" MAKE_EXPORTED ":N",
                    v->name.str, "}");
@@ -781,20 +787,21 @@
 void
 Var_UnExport(const char *str)
 {
+       UnexportWhat what;
        FStr varnames = FSTR_INIT;
-       Boolean unexport_env;
 
        str += strlen("unexport");
-       unexport_env = strncmp(str, "-env", 4) == 0;
-       if (unexport_env) {
+       if (strncmp(str, "-env", 4) == 0) {
                UnexportEnv();
+               what = UNEXPORT_ENV;
        } else {
                cpp_skip_whitespace(&str);
-               if (str[0] != '\0')
+               what = str[0] != '\0' ? UNEXPORT_NAMED : UNEXPORT_ALL;
+               if (what == UNEXPORT_NAMED)
                        FStr_Assign(&varnames, str, NULL);
        }
 
-       if (varnames.str == NULL) {
+       if (what != UNEXPORT_NAMED) {
                char *expanded;
                /* Using .MAKE.EXPORTED */
                (void)Var_Subst("${" MAKE_EXPORTED ":O:u}", VAR_GLOBAL,
@@ -809,7 +816,7 @@
                Words words = Str_Words(varnames.str, FALSE);
                for (i = 0; i < words.len; i++) {
                        const char *varname = words.words[i];
-                       UnexportVar(varname, unexport_env, varnames.str == str);
+                       UnexportVar(varname, what);
                }
                Words_Free(words);
                if (varnames.str != str)