[issue8914] Run clang's static analyzer

Brett Cannon added the comment: 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. -- ___ Python tracker _

[issue8914] Run clang's static analyzer

Petri Lehtinen added the comment: 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? -- nosy: +petri.lehtinen ___ Python tracker

[issue8914] Run clang's static analyzer

Brett Cannon added the comment: applied in r88506 -- resolution: -> fixed stage: -> committed/rejected status: open -> closed ___ Python tracker ___ ___

[issue8914] Run clang's static analyzer

Brett Cannon added the comment: 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). -- Added file: http://bugs.python.org/file206

[issue8914] Run clang's static analyzer

Brett Cannon added the comment: This is a new set of results with the analyzer_fixes.diff file applied using the latest analyzer. -- Added file: http://bugs.python.org/file20667/checker_254_2011-02-03.tar.xz ___ Python tracker

[issue8914] Run clang's static analyzer

Brett Cannon added the comment: 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. -- Added file: http://bugs.python.org/file20666/analyzer_fixes.diff

[issue8914] Run clang's static analyzer

Brett Cannon added the comment: New patch which covers dead assignments and increments. -- Added file: http://bugs.python.org/file20663/clang_analyzer.diff ___ Python tracker ___

[issue8914] Run clang's static analyzer

Changes by Brett Cannon : Removed file: http://bugs.python.org/file20660/clang_analyzer.diff ___ Python tracker ___ ___ Python-bugs-list mailin

[issue8914] Run clang's static analyzer

Raymond Hettinger added the comment: Am looking forward to the rest. This will be a nice cleanup. -- ___ Python tracker ___ ___ Python

[issue8914] Run clang's static analyzer

Brett Cannon added the comment: 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 b

[issue8914] Run clang's static analyzer

Raymond Hettinger added the comment: Nice work. I'm surprised there wasn't more dead code. -- nosy: +rhettinger ___ Python tracker ___ ___

[issue8914] Run clang's static analyzer

Brett Cannon added the comment: Here is a patch that covers all of the relevant dead assignments that were detected. -- keywords: +patch Added file: http://bugs.python.org/file20660/clang_analyzer.diff ___ Python tracker

[issue8914] Run clang's static analyzer

Antoine Pitrou added the comment: For the record, you don't have to use pax, recent GNU tar handles xz fine (use the -J option). -- nosy: +pitrou ___ Python tracker ___

[issue8914] Run clang's static analyzer

Brett Cannon added the comment: 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. -- Added file: http://bugs.python.org/fi

[issue8914] Run clang's static analyzer

Brett Cannon added the comment: 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 . -- _

[issue8914] Run clang's static analyzer

Brett Cannon added the comment: 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

[issue8914] Run clang's static analyzer

Changes by Jesús Cea Avión : -- nosy: +jcea ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.o

[issue8914] Run clang's static analyzer

New submission from Brett Cannon : 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. -- components: Extension Modules, Interpreter Core messages: 107180 nosy: brett.cann