diff options
| author | rillig <rillig@NetBSD.org> | 2023-07-07 06:03:31 +0000 |
|---|---|---|
| committer | rillig <rillig@NetBSD.org> | 2023-07-07 06:03:31 +0000 |
| commit | f260bdb92f361d3b4e3ab9d23e5f0eb4faafd698 (patch) | |
| tree | ab7115a0d2c19d8f3c540e8afc852a7403b759f7 /usr.bin/xlint/lint1/decl.c | |
| parent | eed2e1f5f354b83b1e2723c0412ffdf22f00d4e8 (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.c | 6 |
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; /* |
