[Reconnoiter-devel] LLVM/clang static analysis results

Piotr Sikora piotr.sikora at frickle.com
Tue May 25 10:20:27 EDT 2010


Hi Theo,
thanks for fast response!

> I looked at some of the logic errors and the look like a mix of false 
> positives and actual bugs.
>
> However, I would also say that removing these warnings is to the benefit 
> of the project as it makes future runs of clang much more enlightening.

Yes, I agree.

> I see no reason why a large patch wouldn't be accepts.

I'm really glad to hear that! There is a lot of open-source projects that 
would have problems with such patches.

> I would request that it be broken into three pieces by bug group:
> (1) removing dead stores
> (2) for code changes silencing false-positive logic errors
> (3) fixing actual logic errors

Sure, no problem.

Best regards,
Piotr Sikora < piotr.sikora at frickle.com >
 



More information about the Reconnoiter-devel mailing list