[flac-dev] [PATCH] Fix cppcheck warnings

Erik de Castro Lopo mle+la at mega-nerd.com
Thu Jan 19 09:42:40 UTC 2017


Rosen Penev wrote:

> fedora comes with 1.75 so I used that. I also used some additional
> flags: cppcheck --enable=all --inconclusive --std=posix --force

Thanks! I had wanted to use verison 1.76.1 from Debian in CI, but never
enabled it because it was giving me a false positive even with relatively
standard command line options.

I've now refactored the code so that there is no false positive and
I'm playing around with `--enable=warning` which enables the unsigned/print
warning you patch fixes and a couple of other interesting things.

Cheers,
Erik
-- 
----------------------------------------------------------------------
Erik de Castro Lopo
http://www.mega-nerd.com/


More information about the flac-dev mailing list