Source-Changes-HG archive

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

[src/trunk]: src/usr.bin/make make(1): remove const from function parameters



details:   https://anonhg.NetBSD.org/src/rev/0d8a981e58c0
branches:  trunk
changeset: 947130:0d8a981e58c0
user:      rillig <rillig%NetBSD.org@localhost>
date:      Sat Dec 12 00:33:25 2020 +0000

description:
make(1): remove const from function parameters

These have been left-overs from refactoring, when these pieces were
extracted to separate functions.

diffstat:

 usr.bin/make/compat.c |   6 +++---
 usr.bin/make/cond.c   |   6 +++---
 usr.bin/make/for.c    |   6 +++---
 usr.bin/make/job.c    |  11 +++++------
 usr.bin/make/var.c    |  35 ++++++++++++++++-------------------
 5 files changed, 30 insertions(+), 34 deletions(-)

diffs (197 lines):

diff -r a94659a50f92 -r 0d8a981e58c0 usr.bin/make/compat.c
--- a/usr.bin/make/compat.c     Sat Dec 12 00:05:05 2020 +0000
+++ b/usr.bin/make/compat.c     Sat Dec 12 00:33:25 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: compat.c,v 1.206 2020/12/12 00:05:05 rillig Exp $      */
+/*     $NetBSD: compat.c,v 1.207 2020/12/12 00:33:25 rillig Exp $      */
 
 /*
  * Copyright (c) 1988, 1989, 1990 The Regents of the University of California.
@@ -96,7 +96,7 @@
 #include "pathnames.h"
 
 /*     "@(#)compat.c   8.2 (Berkeley) 3/19/94" */
-MAKE_RCSID("$NetBSD: compat.c,v 1.206 2020/12/12 00:05:05 rillig Exp $");
+MAKE_RCSID("$NetBSD: compat.c,v 1.207 2020/12/12 00:33:25 rillig Exp $");
 
 static GNode *curTarg = NULL;
 static pid_t compatChild;
@@ -468,7 +468,7 @@
 }
 
 static Boolean
-MakeUnmade(GNode *const gn, GNode *const pgn)
+MakeUnmade(GNode *gn, GNode *pgn)
 {
 
        assert(gn->made == UNMADE);
diff -r a94659a50f92 -r 0d8a981e58c0 usr.bin/make/cond.c
--- a/usr.bin/make/cond.c       Sat Dec 12 00:05:05 2020 +0000
+++ b/usr.bin/make/cond.c       Sat Dec 12 00:33:25 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: cond.c,v 1.221 2020/12/05 18:15:40 rillig Exp $        */
+/*     $NetBSD: cond.c,v 1.222 2020/12/12 00:33:25 rillig Exp $        */
 
 /*
  * Copyright (c) 1988, 1989, 1990 The Regents of the University of California.
@@ -94,7 +94,7 @@
 #include "dir.h"
 
 /*     "@(#)cond.c     8.2 (Berkeley) 1/2/94"  */
-MAKE_RCSID("$NetBSD: cond.c,v 1.221 2020/12/05 18:15:40 rillig Exp $");
+MAKE_RCSID("$NetBSD: cond.c,v 1.222 2020/12/12 00:33:25 rillig Exp $");
 
 /*
  * The parsing of conditional expressions is based on this grammar:
@@ -1117,7 +1117,7 @@
  *                     or because the condition could not be evaluated
  */
 CondEvalResult
-Cond_EvalLine(const char *const line)
+Cond_EvalLine(const char *line)
 {
        typedef enum IfState {
 
diff -r a94659a50f92 -r 0d8a981e58c0 usr.bin/make/for.c
--- a/usr.bin/make/for.c        Sat Dec 12 00:05:05 2020 +0000
+++ b/usr.bin/make/for.c        Sat Dec 12 00:33:25 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: for.c,v 1.115 2020/11/07 21:04:43 rillig Exp $ */
+/*     $NetBSD: for.c,v 1.116 2020/12/12 00:33:25 rillig Exp $ */
 
 /*
  * Copyright (c) 1992, The Regents of the University of California.
@@ -60,7 +60,7 @@
 #include "make.h"
 
 /*     "@(#)for.c      8.1 (Berkeley) 6/6/93"  */
-MAKE_RCSID("$NetBSD: for.c,v 1.115 2020/11/07 21:04:43 rillig Exp $");
+MAKE_RCSID("$NetBSD: for.c,v 1.116 2020/12/12 00:33:25 rillig Exp $");
 
 static int forLevel = 0;       /* Nesting level */
 
@@ -375,7 +375,7 @@
 /* While expanding the body of a .for loop, replace single-character
  * variable expressions like $i with their ${:U...} expansion. */
 static void
-SubstVarShort(For *f, char const ch, const char **pp, const char **inout_mark)
+SubstVarShort(For *f, char ch, const char **pp, const char **inout_mark)
 {
     const char *p = *pp;
     size_t i;
diff -r a94659a50f92 -r 0d8a981e58c0 usr.bin/make/job.c
--- a/usr.bin/make/job.c        Sat Dec 12 00:05:05 2020 +0000
+++ b/usr.bin/make/job.c        Sat Dec 12 00:33:25 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: job.c,v 1.369 2020/12/12 00:05:05 rillig Exp $ */
+/*     $NetBSD: job.c,v 1.370 2020/12/12 00:33:25 rillig Exp $ */
 
 /*
  * Copyright (c) 1988, 1989, 1990 The Regents of the University of California.
@@ -143,7 +143,7 @@
 #include "trace.h"
 
 /*     "@(#)job.c      8.2 (Berkeley) 3/19/94" */
-MAKE_RCSID("$NetBSD: job.c,v 1.369 2020/12/12 00:05:05 rillig Exp $");
+MAKE_RCSID("$NetBSD: job.c,v 1.370 2020/12/12 00:33:25 rillig Exp $");
 
 /*
  * A shell defines how the commands are run.  All commands for a target are
@@ -805,9 +805,8 @@
 }
 
 static void
-JobPrintSpecials(Job *const job, const char *const escCmd,
-                Boolean const run, CommandFlags *const inout_cmdFlags,
-                const char **const inout_cmdTemplate)
+JobPrintSpecials(Job *job, const char *escCmd, Boolean run,
+                CommandFlags *inout_cmdFlags, const char **inout_cmdTemplate)
 {
        if (!run)
                inout_cmdFlags->ignerr = FALSE;
@@ -836,7 +835,7 @@
  * after all other targets have been made.
  */
 static void
-JobPrintCommand(Job *job, const char * const ucmd)
+JobPrintCommand(Job *job, const char *ucmd)
 {
        Boolean run;
 
diff -r a94659a50f92 -r 0d8a981e58c0 usr.bin/make/var.c
--- a/usr.bin/make/var.c        Sat Dec 12 00:05:05 2020 +0000
+++ b/usr.bin/make/var.c        Sat Dec 12 00:33:25 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: var.c,v 1.719 2020/12/07 01:50:19 rillig Exp $ */
+/*     $NetBSD: var.c,v 1.720 2020/12/12 00:33:25 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.719 2020/12/07 01:50:19 rillig Exp $");
+MAKE_RCSID("$NetBSD: var.c,v 1.720 2020/12/12 00:33:25 rillig Exp $");
 
 /* A string that may need to be freed after use. */
 typedef struct FStr {
@@ -3269,7 +3269,7 @@
 #endif
 
 static void
-LogBeforeApply(const ApplyModifiersState *st, const char *mod, const char endc)
+LogBeforeApply(const ApplyModifiersState *st, const char *mod, char endc)
 {
        char eflags_str[VarEvalFlags_ToStringSize];
        char vflags_str[VarFlags_ToStringSize];
@@ -3388,11 +3388,8 @@
 /* While expanding a variable expression, expand and apply indirect
  * modifiers such as in ${VAR:${M_indirect}}. */
 static ApplyModifiersIndirectResult
-ApplyModifiersIndirect(
-    ApplyModifiersState *const st,
-    const char **const inout_p,
-    void **const inout_freeIt
-)
+ApplyModifiersIndirect(ApplyModifiersState *st, const char **inout_p,
+                      void **inout_freeIt)
 {
        const char *p = *inout_p;
        const char *mods;
@@ -3454,15 +3451,15 @@
 /* Apply any modifiers (such as :Mpattern or :@var@loop@ or :Q or ::=value). */
 static char *
 ApplyModifiers(
-    const char **const pp,     /* the parsing position, updated upon return */
-    char *const val,           /* the current value of the expression */
-    char const startc,         /* '(' or '{', or '\0' for indirect modifiers */
-    char const endc,           /* ')' or '}', or '\0' for indirect modifiers */
-    Var *const v,
-    VarExprFlags *const exprFlags,
-    GNode *const ctxt,         /* for looking up and modifying variables */
-    VarEvalFlags const eflags,
-    void **const inout_freeIt  /* free this after using the return value */
+    const char **pp,           /* the parsing position, updated upon return */
+    char *val,                 /* the current value of the expression */
+    char startc,               /* '(' or '{', or '\0' for indirect modifiers */
+    char endc,                 /* ')' or '}', or '\0' for indirect modifiers */
+    Var *v,
+    VarExprFlags *exprFlags,
+    GNode *ctxt,               /* for looking up and modifying variables */
+    VarEvalFlags eflags,
+    void **inout_freeIt                /* free this after using the return value */
 )
 {
        ApplyModifiersState st = {
@@ -4120,8 +4117,8 @@
 }
 
 static void
-VarSubstNested(const char **const pp, Buffer *const buf, GNode *const ctxt,
-              VarEvalFlags const eflags, Boolean *inout_errorReported)
+VarSubstNested(const char **pp, Buffer *buf, GNode *ctxt,
+              VarEvalFlags eflags, Boolean *inout_errorReported)
 {
        const char *p = *pp;
        const char *nested_p = p;



Home | Main Index | Thread Index | Old Index