Module Name: src
Committed By: rillig
Date: Sun Dec 10 14:59:48 UTC 2023
Modified Files:
src/tests/usr.bin/xlint/lint1: d_c99_bool_strict.c
src/usr.bin/xlint/lint1: decl.c init.c
src/usr.bin/xlint/xlint: xlint.c
Log Message:
lint: clean up comments
No functional change.
To generate a diff of this commit:
cvs rdiff -u -r1.44 -r1.45 src/tests/usr.bin/xlint/lint1/d_c99_bool_strict.c
cvs rdiff -u -r1.383 -r1.384 src/usr.bin/xlint/lint1/decl.c
cvs rdiff -u -r1.252 -r1.253 src/usr.bin/xlint/lint1/init.c
cvs rdiff -u -r1.120 -r1.121 src/usr.bin/xlint/xlint/xlint.c
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
Modified files:
Index: src/tests/usr.bin/xlint/lint1/d_c99_bool_strict.c
diff -u src/tests/usr.bin/xlint/lint1/d_c99_bool_strict.c:1.44 src/tests/usr.bin/xlint/lint1/d_c99_bool_strict.c:1.45
--- src/tests/usr.bin/xlint/lint1/d_c99_bool_strict.c:1.44 Wed Aug 2 18:51:25 2023
+++ src/tests/usr.bin/xlint/lint1/d_c99_bool_strict.c Sun Dec 10 14:59:47 2023
@@ -1,4 +1,4 @@
-/* $NetBSD: d_c99_bool_strict.c,v 1.44 2023/08/02 18:51:25 rillig Exp $ */
+/* $NetBSD: d_c99_bool_strict.c,v 1.45 2023/12/10 14:59:47 rillig Exp $ */
# 3 "d_c99_bool_strict.c"
/*
@@ -160,7 +160,7 @@ enum strict_bool_constant_expressions {
/*
* Without strict bool mode, these two variants of an expression can
* occur when a preprocessor macro is either defined to 1 or left
- * empty (since C99), as in lint1/ops.def.
+ * empty (since C99).
*
* In strict bool mode, the resulting expression can be compared
* against 0 to achieve the same effect (so +0 != 0 or 1 + 0 != 0).
Index: src/usr.bin/xlint/lint1/decl.c
diff -u src/usr.bin/xlint/lint1/decl.c:1.383 src/usr.bin/xlint/lint1/decl.c:1.384
--- src/usr.bin/xlint/lint1/decl.c:1.383 Sun Dec 3 18:17:41 2023
+++ src/usr.bin/xlint/lint1/decl.c Sun Dec 10 14:59:47 2023
@@ -1,4 +1,4 @@
-/* $NetBSD: decl.c,v 1.383 2023/12/03 18:17:41 rillig Exp $ */
+/* $NetBSD: decl.c,v 1.384 2023/12/10 14:59:47 rillig Exp $ */
/*
* Copyright (c) 1996 Christopher G. Demetriou. All Rights Reserved.
@@ -38,7 +38,7 @@
#include <sys/cdefs.h>
#if defined(__RCSID)
-__RCSID("$NetBSD: decl.c,v 1.383 2023/12/03 18:17:41 rillig Exp $");
+__RCSID("$NetBSD: decl.c,v 1.384 2023/12/10 14:59:47 rillig Exp $");
#endif
#include <sys/param.h>
@@ -97,11 +97,8 @@ initdecl(void)
}
/*
- * Returns a shared type structure for arithmetic types and void.
- *
- * It's important to duplicate this structure using block_dup_type or
- * expr_dup_type if it is to be modified (adding qualifiers or anything
- * else).
+ * Returns a shared type structure for arithmetic types and void. The returned
+ * type must not be modified; use block_dup_type or expr_dup_type if necessary.
*/
type_t *
gettyp(tspec_t t)
@@ -296,7 +293,6 @@ typedef_error(type_t *td, tspec_t t)
return td;
invalid:
- /* Anything else is not accepted. */
dcs->d_invalid_type_combination = true;
return td;
}
Index: src/usr.bin/xlint/lint1/init.c
diff -u src/usr.bin/xlint/lint1/init.c:1.252 src/usr.bin/xlint/lint1/init.c:1.253
--- src/usr.bin/xlint/lint1/init.c:1.252 Sun Dec 3 13:12:40 2023
+++ src/usr.bin/xlint/lint1/init.c Sun Dec 10 14:59:47 2023
@@ -1,4 +1,4 @@
-/* $NetBSD: init.c,v 1.252 2023/12/03 13:12:40 rillig Exp $ */
+/* $NetBSD: init.c,v 1.253 2023/12/10 14:59:47 rillig Exp $ */
/*
* Copyright (c) 1994, 1995 Jochen Pohl
@@ -38,7 +38,7 @@
#include <sys/cdefs.h>
#if defined(__RCSID)
-__RCSID("$NetBSD: init.c,v 1.252 2023/12/03 13:12:40 rillig Exp $");
+__RCSID("$NetBSD: init.c,v 1.253 2023/12/10 14:59:47 rillig Exp $");
#endif
#include <stdlib.h>
@@ -100,7 +100,7 @@ typedef struct designator {
/*
* The path from the "current object" of a brace level to the sub-object that
- * is initialized by an expression. Examples for designations are '.member'
+ * is initialized by an expression. Examples of designations are '.member'
* or '.member[123].member.member[1][1]'.
*
* C99 6.7.8p6, 6.7.8p7
@@ -115,7 +115,7 @@ typedef struct designation {
* Everything that happens between a '{' and the corresponding '}', as part
* of an initialization.
*
- * Each brace level has a "current object". For the outermost brace level,
+ * Each brace level has a "current object". For the outermost brace level,
* it is the same as the object to be initialized. Each nested '{' begins a
* nested brace level, for the sub-object pointed to by the designator of the
* outer brace level.
Index: src/usr.bin/xlint/xlint/xlint.c
diff -u src/usr.bin/xlint/xlint/xlint.c:1.120 src/usr.bin/xlint/xlint/xlint.c:1.121
--- src/usr.bin/xlint/xlint/xlint.c:1.120 Sun Dec 3 18:17:41 2023
+++ src/usr.bin/xlint/xlint/xlint.c Sun Dec 10 14:59:47 2023
@@ -1,4 +1,4 @@
-/* $NetBSD: xlint.c,v 1.120 2023/12/03 18:17:41 rillig Exp $ */
+/* $NetBSD: xlint.c,v 1.121 2023/12/10 14:59:47 rillig Exp $ */
/*
* Copyright (c) 1996 Christopher G. Demetriou. All Rights Reserved.
@@ -38,7 +38,7 @@
#include <sys/cdefs.h>
#if defined(__RCSID)
-__RCSID("$NetBSD: xlint.c,v 1.120 2023/12/03 18:17:41 rillig Exp $");
+__RCSID("$NetBSD: xlint.c,v 1.121 2023/12/10 14:59:47 rillig Exp $");
#endif
#include <sys/param.h>
@@ -376,7 +376,7 @@ run_cpp(const char *name)
list_add(&args, name);
list_add_ref(&args, NULL);
- /* we reuse the same tmp file for cpp output, so rewind and truncate */
+ /* Rewind after a possible previous run of cpp and lint1. */
if (lseek(cpp.outfd, 0, SEEK_SET) != 0) {
warn("lseek");
terminate(-1);
