Hi everyone, Since mongrel2 is mostly written in C, I thought it would be interesting to give a shot at the LLVM-CLang static analyzer that often provides interesting information to tune the code even more. so I did a pass on the most recent fossil trunk (checkout d8a677f2b5). I don't have must time right now to suggest fixes or to filter-out false-positives (such as for auto-generated sources or 'goto error' branches), but here is the rough HTML report attached. Feel free to have a look and give a hand in fixing those if necessary. Done with LLVM/CLang v2.9. Don't hesitate to ask for more details. Best regards, Lionel
On Tue, Apr 19, 2011 at 05:45:39PM +0200, Lionel Orry wrote: > Hi everyone, > > Since mongrel2 is mostly written in C, I thought it would be > interesting to give a shot at the LLVM-CLang static analyzer that > often provides interesting information to tune the code even more. so > I did a pass on the most recent fossil trunk (checkout d8a677f2b5). Cool, I'll take a look at this. Next time, can you toss the .bz2 on a web server and give a link instead? Kind of unkind to people to send them moby mail on a mailing list. :-) -- Zed A. Shaw http://zedshaw.com/
On Wed, Apr 20, 2011 at 12:21 AM, Zed A. Shaw <zedshaw@zedshaw.com> wrote: > On Tue, Apr 19, 2011 at 05:45:39PM +0200, Lionel Orry wrote: >> Hi everyone, >> >> Since mongrel2 is mostly written in C, I thought it would be >> interesting to give a shot at the LLVM-CLang static analyzer that >> often provides interesting information to tune the code even more. so >> I did a pass on the most recent fossil trunk (checkout d8a677f2b5). > > Cool, I'll take a look at this. Next time, can you toss the .bz2 on a > web server and give a link instead? Kind of unkind to people to send > them moby mail on a mailing list. :-) Ouch, sorry, I did not realise the archive was nerly reaching 1Mb large... I'll try to drop it elsewhere next time, for sure. Sorry for that. > > -- > Zed A. Shaw > http://zedshaw.com/ > Lionel