commit:     4982bec7b9623415fec4dc9d4d7a9696b47a4479
Author:     Sam James <sam <AT> gentoo <DOT> org>
AuthorDate: Wed Nov  9 02:22:06 2022 +0000
Commit:     Sam James <sam <AT> gentoo <DOT> org>
CommitDate: Wed Nov  9 02:25:45 2022 +0000
URL:        https://gitweb.gentoo.org/proj/portage.git/commit/?id=4982bec7

install-qa-check.d/90gcc-warnings: formatting tweaks

Group similar warnings together.

Signed-off-by: Sam James <sam <AT> gentoo.org>

 bin/install-qa-check.d/90gcc-warnings | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/bin/install-qa-check.d/90gcc-warnings 
b/bin/install-qa-check.d/90gcc-warnings
index 7a859eb88..162e97459 100644
--- a/bin/install-qa-check.d/90gcc-warnings
+++ b/bin/install-qa-check.d/90gcc-warnings
@@ -19,18 +19,21 @@ gcc_warn_check() {
                        # only will and does, no might :)
                        'warning: .*will.*\[-Wstrict-aliasing\]'
                        'warning: .*does.*\[-Wstrict-aliasing\]'
+
                        # implicit declaration of function ā€˜...ā€™
                        'warning: .*\[-Wimplicit-function-declaration\]'
                        # with -Wall, goes in pair with 
-Wimplicit-function-declaration
                        # but without -Wall, we need to assert for it alone
                        'warning: .*incompatible implicit declaration of 
built-in function'
                        'warning: .*\[-Wbuiltin-declaration-mismatch\]'
+
                        # 'is used uninitialized in this function' and some more
                        'warning: .*\[-Wuninitialized\]'
                        # comparisons like ā€˜X<=Y<=Zā€™ do not have their 
mathematical meaning
                        'warning: .*mathematical meaning*\[-Wparentheses\]'
                        # null argument where non-null required
                        'warning: .*\[-Wnonnull\]'
+
                        # array subscript is above/below/outside array bounds 
(FORTIFY_SOURCE)
                        'warning: .*\[-Warray-bounds\]'
                        # attempt to free a non-heap object
@@ -43,6 +46,7 @@ gcc_warn_check() {
                        'warning: .*\[-Wfortify-source\]'
                        'warning: .*assuming pointer wraparound does not occur'
                        'warning: .*escape sequence out of range'
+
                        # left-hand operand of comma expression has no effect
                        'warning: .*left.*comma.*\[-Wunused-value\]'
                        # converting to non-pointer type ... from NULL and likes
@@ -58,11 +62,13 @@ gcc_warn_check() {
                        'warning: .*\[-Wint-conversion\]'
                        # warning: cast to ... from integer of different size 
(or smaller size)
                        'warning: .*\[-Wint-to-pointer-cast\]'
+
                        # outdated?
                        'warning: .*too few arguments for format'
                        # format ... expects a matching ... argument
                        # (iow, too few arguments for format in new wording :))
                        'warning: .*matching.*\[-Wformat=\]'
+
                        # function returns address of local variable
                        'warning: .*\[-Wreturn-local-addr\]'
                        # missing return at end of function, or non-void return 
in a void function

Reply via email to