diff options
author | Daniel Gustafsson <daniel@yesql.se> | 2018-10-05 13:29:37 +0200 |
---|---|---|
committer | Daniel Gustafsson <daniel@yesql.se> | 2018-10-05 13:29:37 +0200 |
commit | b5d182d037364c5fff250f0ad48066c05115dc7f (patch) | |
tree | 1dfd03a9d87ae2ef1bfefd44a0c1f0d341ac51c2 /docs/GOVERNANCE.md | |
parent | 4c35f24ef463a2124395a8a6eb2c23e3c568e067 (diff) |
checksrc: enable strict mode and warnings
Enable strict and warnings mode for checksrc to ensure we aren't missing
anything due to bugs in the checking code. This uncovered a few things
which are all fixed in this commit:
* several variables were used uninitialized
* several variables were not defined in the correct scope
* the whitelist filehandle was read even if the file didn't exist
* the enable_warn() call when a disable counter had expired was passing
incorrect variables, but since the checkwarn() call is unlikely to hit
(the counter is only decremented to zero on actual ignores) it didn't
manifest a problem.
Closes #3090
Reviewed-by: Daniel Stenberg <daniel@haxx.se>
Reviewed-by: Marcel Raad <Marcel.Raad@teamviewer.com>
Diffstat (limited to 'docs/GOVERNANCE.md')
0 files changed, 0 insertions, 0 deletions