openocd-development-boun...@lists.berlios.de wrote:
> Cool!

I agree, it looks cool.

> I'd like to see patches having to pass clang static analyzer
> unscathed before they're ready for review :-)

I had a quick look at that output, interesting.
Some looked like real bugs, but others it seems clang was getting
it wrong.. like if (ptr == NULL) checks which it thought failed,
but then it complained about a null ptr dereference.

It seemed to get if (!ptr) right though, which I think is preferred
linux kernel style.

--
Jon Povey
jon.po...@racelogic.co.uk

Racelogic is a limited company registered in England. Registered number 2743719 
.
Registered Office Unit 10, Swan Business Centre, Osier Way, Buckingham, Bucks, 
MK18 1TB .

The information contained in this electronic mail transmission is intended by 
Racelogic Ltd for the use of the named individual or entity to which it is 
directed and may contain information that is confidential or privileged. If you 
have received this electronic mail transmission in error, please delete it from 
your system without copying or forwarding it, and notify the sender of the 
error by reply email so that the sender's address records can be corrected. The 
views expressed by the sender of this communication do not necessarily 
represent those of Racelogic Ltd. Please note that Racelogic reserves the right 
to monitor e-mail communications passing through its network


_______________________________________________
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

Reply via email to