]> granicus.if.org Git - curl/commit
checksrc: handle zero scoped ignore commands
authorDaniel Gustafsson <daniel@yesql.se>
Fri, 5 Oct 2018 11:33:21 +0000 (13:33 +0200)
committerDaniel Gustafsson <daniel@yesql.se>
Fri, 5 Oct 2018 11:33:21 +0000 (13:33 +0200)
commit4301d14b90101d61575841e85edab18d384d0154
treeb5e3aab87032c0ccd96ec84feeaa2806cb4f5ad6
parentb5d182d037364c5fff250f0ad48066c05115dc7f
checksrc: handle zero scoped ignore commands

If a !checksrc! disable command specified to ignore zero errors, it was
still added to the ignore block even though nothing was ignored. While
there were no blocks ignored that shouldn't be ignored, the processing
ended with with a warning:

<filename>:<line>:<col>: warning: Unused ignore: LONGLINE (UNUSEDIGNORE)
 /* !checksrc! disable LONGLINE 0 */
                    ^
Fix by instead treating a zero ignore as a a badcommand and throw a
warning for that one.

Closes #3096
Reviewed-by: Daniel Stenberg <daniel@haxx.se>
lib/checksrc.pl