[flashrom] [PATCH] LLVM/clang code review
Stefan Reinauer
stepan at coresystems.de
Tue Sep 15 23:30:26 CEST 2009
Carl-Daniel Hailfinger wrote:
> Hi,
>
> On 15.09.2009 19:49, Stefan Reinauer wrote:
>
>> this patch fixes all 27 flashrom source code issues reported by
>> LLVM/clang's scan-build (r79326, new build on the way).
>>
>>
>
> It does fix the warnings, but it papers over some of the bugs found by
> scan-build. Given that the real bugfixes would have to revert parts of
> the patch, please hold off committing. I'll post a detailed review later.
>
Looking forward to that. I didn't intend to paper over anything. I
surely didn't intend a redesign of that code either though ;-)
Stefan
More information about the flashrom
mailing list