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 redundant VARARGS comments



details:   https://anonhg.NetBSD.org/src/rev/ef17fa3fb91d
branches:  trunk
changeset: 938664:ef17fa3fb91d
user:      rillig <rillig%NetBSD.org@localhost>
date:      Sun Sep 13 06:05:56 2020 +0000

description:
make(1): remove redundant VARARGS comments

These had been necessary at a time when the functions were declared with
"unknown parameter types".

On 1994-03-05, conditional support for function prototypes was added,
for those compilers that already supported this feature.

On 2002-06-15, the prototypes were made mandatory, and since then the
VARARGS comments had been completely redundant, or worse, simply wrong
(for ParseVErrorInternal).

diffstat:

 usr.bin/make/main.c  |  9 +++------
 usr.bin/make/parse.c |  9 +++------
 2 files changed, 6 insertions(+), 12 deletions(-)

diffs (101 lines):

diff -r a8b8817ed26d -r ef17fa3fb91d usr.bin/make/main.c
--- a/usr.bin/make/main.c       Sun Sep 13 06:04:53 2020 +0000
+++ b/usr.bin/make/main.c       Sun Sep 13 06:05:56 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: main.c,v 1.334 2020/09/13 05:56:32 rillig Exp $        */
+/*     $NetBSD: main.c,v 1.335 2020/09/13 06:05:56 rillig Exp $        */
 
 /*
  * Copyright (c) 1988, 1989, 1990, 1993
@@ -69,7 +69,7 @@
  */
 
 #ifndef MAKE_NATIVE
-static char rcsid[] = "$NetBSD: main.c,v 1.334 2020/09/13 05:56:32 rillig Exp $";
+static char rcsid[] = "$NetBSD: main.c,v 1.335 2020/09/13 06:05:56 rillig Exp $";
 #else
 #include <sys/cdefs.h>
 #ifndef lint
@@ -81,7 +81,7 @@
 #if 0
 static char sccsid[] = "@(#)main.c     8.3 (Berkeley) 3/19/94";
 #else
-__RCSID("$NetBSD: main.c,v 1.334 2020/09/13 05:56:32 rillig Exp $");
+__RCSID("$NetBSD: main.c,v 1.335 2020/09/13 06:05:56 rillig Exp $");
 #endif
 #endif /* not lint */
 #endif
@@ -1703,7 +1703,6 @@
  * Side Effects:
  *     The message is printed.
  */
-/* VARARGS */
 void
 Error(const char *fmt, ...)
 {
@@ -1730,7 +1729,6 @@
 /* Produce a Fatal error message, then exit immediately.
  *
  * If jobs are running, waits for them to finish. */
-/* VARARGS */
 void
 Fatal(const char *fmt, ...)
 {
@@ -1765,7 +1763,6 @@
  * Side Effects:
  *     All children are killed indiscriminately and the program Lib_Exits
  */
-/* VARARGS */
 void
 Punt(const char *fmt, ...)
 {
diff -r a8b8817ed26d -r ef17fa3fb91d usr.bin/make/parse.c
--- a/usr.bin/make/parse.c      Sun Sep 13 06:04:53 2020 +0000
+++ b/usr.bin/make/parse.c      Sun Sep 13 06:05:56 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: parse.c,v 1.295 2020/09/12 19:41:20 rillig Exp $       */
+/*     $NetBSD: parse.c,v 1.296 2020/09/13 06:05:56 rillig Exp $       */
 
 /*
  * Copyright (c) 1988, 1989, 1990, 1993
@@ -69,14 +69,14 @@
  */
 
 #ifndef MAKE_NATIVE
-static char rcsid[] = "$NetBSD: parse.c,v 1.295 2020/09/12 19:41:20 rillig Exp $";
+static char rcsid[] = "$NetBSD: parse.c,v 1.296 2020/09/13 06:05:56 rillig Exp $";
 #else
 #include <sys/cdefs.h>
 #ifndef lint
 #if 0
 static char sccsid[] = "@(#)parse.c    8.3 (Berkeley) 3/19/94";
 #else
-__RCSID("$NetBSD: parse.c,v 1.295 2020/09/12 19:41:20 rillig Exp $");
+__RCSID("$NetBSD: parse.c,v 1.296 2020/09/13 06:05:56 rillig Exp $");
 #endif
 #endif /* not lint */
 #endif
@@ -659,7 +659,6 @@
  * Side Effects:
  *     "fatals" is incremented if the level is PARSE_FATAL.
  */
-/* VARARGS */
 static void
 ParseVErrorInternal(FILE *f, const char *cfname, size_t clineno, int type,
     const char *fmt, va_list ap)
@@ -695,7 +694,6 @@
  * Side Effects:
  *     None
  */
-/* VARARGS */
 static void
 ParseErrorInternal(const char *cfname, size_t clineno, int type,
     const char *fmt, ...)
@@ -718,7 +716,6 @@
  * line number.
  *
  * Fmt is given without a trailing newline. */
-/* VARARGS */
 void
 Parse_Error(int type, const char *fmt, ...)
 {



Home | Main Index | Thread Index | Old Index