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 unused JOB_IGNDOTS



details:   https://anonhg.NetBSD.org/src/rev/e06eda84359d
branches:  trunk
changeset: 945819:e06eda84359d
user:      rillig <rillig%NetBSD.org@localhost>
date:      Sun Nov 08 01:16:04 2020 +0000

description:
make(1): remove unused JOB_IGNDOTS

It had been used in 1993 at least, don't know when it was finally
removed.

diffstat:

 usr.bin/make/job.c |  20 ++++++++------------
 usr.bin/make/job.h |   4 +---
 2 files changed, 9 insertions(+), 15 deletions(-)

diffs (73 lines):

diff -r 30e2f2c0e54b -r e06eda84359d usr.bin/make/job.c
--- a/usr.bin/make/job.c        Sun Nov 08 01:12:46 2020 +0000
+++ b/usr.bin/make/job.c        Sun Nov 08 01:16:04 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: job.c,v 1.313 2020/11/08 01:07:00 rillig Exp $ */
+/*     $NetBSD: job.c,v 1.314 2020/11/08 01:16:04 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.313 2020/11/08 01:07:00 rillig Exp $");
+MAKE_RCSID("$NetBSD: job.c,v 1.314 2020/11/08 01:16:04 rillig Exp $");
 
 /* A shell defines how the commands are run.  All commands for a target are
  * written into a single file, which is then given to the shell to execute
@@ -900,12 +900,11 @@
 
        if (strcmp(cmd, "...") == 0) {
            job->node->type |= OP_SAVE_CMDS;
-           if (!(job->flags & JOB_IGNDOTS)) {
-               job->tailCmds = ln->next;
-               break;
-           }
-       } else
-           JobPrintCommand(job, ln->datum);
+           job->tailCmds = ln->next;
+           break;
+       }
+
+       JobPrintCommand(job, ln->datum);
     }
 }
 
@@ -1473,7 +1472,6 @@
  * Input:
  *     gn              target to create
  *     flags           flags for the job to override normal ones.
- *                     e.g. JOB_SPECIAL or JOB_IGNDOTS
  *     previous        The previous Job structure for this node, if any.
  *
  * Results:
@@ -1485,9 +1483,7 @@
  *     A new Job node is created and added to the list of running
  *     jobs. PMake is forked and a child shell created.
  *
- * NB: I'm fairly sure that this code is never called with JOB_SPECIAL set
- *     JOB_IGNDOTS is never set (dsl)
- *     Also the return value is ignored by everyone.
+ * NB: The return value is ignored by everyone.
  *-----------------------------------------------------------------------
  */
 static JobStartResult
diff -r 30e2f2c0e54b -r e06eda84359d usr.bin/make/job.h
--- a/usr.bin/make/job.h        Sun Nov 08 01:12:46 2020 +0000
+++ b/usr.bin/make/job.h        Sun Nov 08 01:16:04 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: job.h,v 1.60 2020/11/07 21:24:33 rillig Exp $  */
+/*     $NetBSD: job.h,v 1.61 2020/11/08 01:16:04 rillig Exp $  */
 
 /*
  * Copyright (c) 1988, 1989, 1990 The Regents of the University of California.
@@ -134,8 +134,6 @@
     /* Target is a special one. i.e. run it locally
      * if we can't export it and maxLocal is 0 */
     JOB_SPECIAL        = 1 << 2,
-    /* Ignore "..." lines when processing commands */
-    JOB_IGNDOTS        = 1 << 3,
     /* we've sent 'set -x' */
     JOB_TRACED = 1 << 10
 } JobFlags;



Home | Main Index | Thread Index | Old Index