homepage

Just like I did for Python 2.7.
Should use this issue to keep track of what I have already processed and what I had to skip because OS X doesn't have the right files.

Analyzed by downloading the latest Clang static analyzer from http://clang-analyzer.llvm.org/ and running ``scan-build ./configure --with-pydebug; scan-build make -s -j2``. The attached tar.xz (yes I went with a cutting edge archival format; get pax on UNIX to read it) contains the HTML report.

Well, there is no upload because the compressed file is 70 MB. But you can download the file from https://docs.google.com/leaf?id=0B7CvitGf6YffMGFlYTg2N2UtMjY3Yi00ZTg1LWI2NWUtNmRiMmEwYzZjMTQ1&sort=name&layout=list&num=50 .

Turns out I was using pax wrong. =) Thought it automatically compressed new files; turns out it doesn't. Using the zx compressor it shrank the 70 MB file down, so ignore the Google Docs upload and just grab this.

For the record, you don't have to use pax, recent GNU tar handles xz fine (use the -J option).

Here is a patch that covers all of the relevant dead assignments that were detected.

Nice work. I'm surprised there wasn't more dead code.

This was just dead assignments. I have not tackled Idempotent operations, dead increments, dead initializations, dead nested assignments, possible deref of NULL, deref of unassigned pointer, division by zero, undefined/garbage results, or undefined alloc of 0 bytes.
IOW I tackled 36 out of 164 reported issues. =)

Am looking forward to the rest.
This will be a nice cleanup.

New patch which covers dead assignments and increments.

Here is a new patch which includes setting the noreturn attribute on Py_FatalError() and Py_Exit() in order to make the null pointer deref analysis results more tractable.

This is a new set of results with the analyzer_fixes.diff file applied using the latest analyzer.

OK, done going through the results. Attached is a patch with all of the fixes (only one actual bug which I file an issue for separately; everything else is removing a little bit of dead code here and there).

applied in r88506 

Regarding issue 12595 (function redeclaration warning when including Python.h), could a small part (or optionally all) of this patch be backported to 3.2?

I don't feel comfortable changing what is defined in a header file in a point release, so I am not going to backport the fix.