[Python-Dev] About "Coverity Study Ranks LAMP Code Quality"
Tim Peters
tim.peters at gmail.com
Wed Mar 15 08:43:37 CET 2006
[Neal Norwitz]
> ...
> The public report says 15, but the current developer report shows 12.
> I'm not sure why there is a discrepancy. All 12 are in ctypes which
> was recently imported.
I'm having a really hard time making sense of the UI on this. When I
looked at the Python project just now (I can log in, so guess that's
what you called the "developer report" above), I see 13 "error" rows,
and none of them referencing ctypes. OTOH, maybe you'd count this as
zero rows, since there are none left with BUG or UNINSPECTED status.
I dug into one of them, a claim by the tool that after marshal.c's:
int one = 1;
int is_little_endian = (int)*(char*)&one;
we have:
Event const: After this line, the value of "is_little_endian" is equal to 1
but of course that's not so on a big-endian box, and it goes on to
claim that there's dead code because of this.
I'm not much inclined to look at more of these -- I probably waited so
long that all we have left are false positives? If not, and somebody
wants me to look at one, point it out specifically ;-)
More information about the Python-Dev
mailing list