Le 10/11/2011 09:00, Lex Trotman a écrit :
Hi All,
With Matthew's help I've run the clang static analyser over the Geany codebase.
Results are here: https://github.com/elextr/geany_stuff/raw/master/sc.xz this is a directory full of html, as usual start at index.html.
A total of 86 errors were detected.
Some of these are false positives where functions like error() don't return (but are not so marked) so code after error() won't run on the condition that caused the error. This just means marking the error function as never returns.
Others require malformed data, eg filetype with comment single, comment close but no comment open. Doesn't really need to be fixed, except to prevent a crash in case it runs into such malformed data.
Don't be too quick jumping in and "fixing" many of these since in some cases they are rather subtle, for example the fix I committed yesterday for an error detected in build.c wasn't the first obvious one.
Please enjoy! and consider 86 as not bad, especially as most are in the horror tagmanager.
I fixed a few ones in c73ee49 [1] and 048e6a6 [2]. Thanks for the analysis :)
Cheers, Colomban
[1] https://github.com/geany/geany/commit/c73ee497588296122d579bdbf789a547eb4177... [2] https://github.com/geany/geany/commit/048e6a6c3c3309e7ee1660cdec144771eb98bd...