[src/trunk]: src/tests/usr.bin/xlint/lint1 tests/lint: clean up accept.sh, do...

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

details:   https://anonhg.NetBSD.org/src/rev/93af12842b6e
branches:  trunk
changeset: 1022818:93af12842b6e
user:      rillig <rillig%NetBSD.org@localhost>
date:      Sun Aug 08 13:19:51 2021 +0000

description:
tests/lint: clean up accept.sh, document test for removed assertion

diffstat:

 tests/usr.bin/xlint/lint1/accept.sh                    |  6 ++++--
 tests/usr.bin/xlint/lint1/c99_bool_strict_suppressed.c |  7 ++++---
 2 files changed, 8 insertions(+), 5 deletions(-)

diffs (58 lines):

diff -r 7c0a919d58ef -r 93af12842b6e tests/usr.bin/xlint/lint1/accept.sh
--- a/tests/usr.bin/xlint/lint1/accept.sh       Sun Aug 08 12:35:43 2021 +0000
+++ b/tests/usr.bin/xlint/lint1/accept.sh       Sun Aug 08 13:19:51 2021 +0000
@@ -1,5 +1,5 @@
 #! /bin/sh
-# $NetBSD: accept.sh,v 1.4 2021/08/05 06:34:43 rillig Exp $
+# $NetBSD: accept.sh,v 1.5 2021/08/08 13:19:51 rillig Exp $
 #
 # Copyright (c) 2021 The NetBSD Foundation, Inc.
 # All rights reserved.
@@ -36,6 +36,7 @@
 . './t_integration.sh'
 
 for pattern in "$@"; do
+       # shellcheck disable=SC2231
        for test in *$pattern*.c; do
                base=${test%.*}
                cfile="$base.c"
@@ -45,7 +46,7 @@
 
                configure_test_case "$cfile"
                # shellcheck disable=SC2154
-               if [ $skip = yes ]; then
+               if [ "$skip" = yes ]; then
                        continue
                fi
 
@@ -93,4 +94,5 @@
        done
 done
 
+# shellcheck disable=SC2035
 lua '../check-expect.lua' *.c
diff -r 7c0a919d58ef -r 93af12842b6e tests/usr.bin/xlint/lint1/c99_bool_strict_suppressed.c
--- a/tests/usr.bin/xlint/lint1/c99_bool_strict_suppressed.c    Sun Aug 08 12:35:43 2021 +0000
+++ b/tests/usr.bin/xlint/lint1/c99_bool_strict_suppressed.c    Sun Aug 08 13:19:51 2021 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: c99_bool_strict_suppressed.c,v 1.3 2021/07/05 19:02:14 rillig Exp $    */
+/*     $NetBSD: c99_bool_strict_suppressed.c,v 1.4 2021/08/08 13:19:51 rillig Exp $    */
 # 3 "c99_bool_strict_suppressed.c"
 
 /*
@@ -9,12 +9,13 @@
  * This can result in assertion failures later.  One such assertion has been
  * there since at least 1995, at the beginning of expr(), ensuring that the
  * expression is either non-null or an error message has been _printed_.
- * In 1995 it was not possible to suppress error messages, which means that
+ * In 1995, it was not possible to suppress error messages, which means that
  * the number of printed errors equaled the number of occurred errors.
  *
  * In err.c 1.12 from 2000-07-06, the option -X was added, allowing to
  * suppress individual error messages.  That commit did not mention any
- * interaction with the assertion in expr().
+ * interaction with the assertion in expr().  The assertion was removed in
+ * tree.c 1.305 from 2021-07-04.
  */
 
 /* lint1-extra-flags: -T -X 107,330,331,332,333 */