summaryrefslogtreecommitdiff
path: root/usr.bin/xlint/lint1/decl.c
diff options
context:
space:
mode:
authorrillig <rillig@NetBSD.org>2023-07-07 06:03:31 +0000
committerrillig <rillig@NetBSD.org>2023-07-07 06:03:31 +0000
commitf260bdb92f361d3b4e3ab9d23e5f0eb4faafd698 (patch)
treeab7115a0d2c19d8f3c540e8afc852a7403b759f7 /usr.bin/xlint/lint1/decl.c
parenteed2e1f5f354b83b1e2723c0412ffdf22f00d4e8 (diff)
lint: only skip 'unused' warnings after errors, not other warnings
Previously, in -w mode, any warning suppressed further 'unused' warnings, even though there was no need to do that. This can be seen in the test gcc_attribute_var.c, where only the last unused variable from a function was marked as unused, the others slipped through. Fixed by counting the errors and the warnings separately and only combining them if actually desired.
Diffstat (limited to 'usr.bin/xlint/lint1/decl.c')
-rw-r--r--usr.bin/xlint/lint1/decl.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/usr.bin/xlint/lint1/decl.c b/usr.bin/xlint/lint1/decl.c
index 55fbb025402..03ae41acc9c 100644
--- a/usr.bin/xlint/lint1/decl.c
+++ b/usr.bin/xlint/lint1/decl.c
@@ -1,4 +1,4 @@
-/* $NetBSD: decl.c,v 1.341 2023/07/03 10:23:12 rillig Exp $ */
+/* $NetBSD: decl.c,v 1.342 2023/07/07 06:03:31 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.341 2023/07/03 10:23:12 rillig Exp $");
+__RCSID("$NetBSD: decl.c,v 1.342 2023/07/07 06:03:31 rillig Exp $");
#endif
#include <sys/param.h>
@@ -2935,7 +2935,7 @@ check_variable_usage(bool novar, sym_t *sym)
return;
/* errors in expressions easily cause lots of these warnings */
- if (nerr != 0)
+ if (seen_error)
return;
/*