diff options
| author | Bruce Smith <bruce@nanorex.com> | 2007-11-08 00:13:10 +0000 |
|---|---|---|
| committer | Bruce Smith <bruce@nanorex.com> | 2007-11-08 00:13:10 +0000 |
| commit | 65a257af5aafed4859e3d51172ef27e002ef5fd0 (patch) | |
| tree | 00d29d332aa2d0a254bc1d7fefd8435c5e1403ed | |
| parent | 11f094451282af7b92fabf0f59c87f96820107db (diff) | |
| download | nanoengineer-65a257af5aafed4859e3d51172ef27e002ef5fd0.tar.gz nanoengineer-65a257af5aafed4859e3d51172ef27e002ef5fd0.zip | |
add summary report for E0602 (Undefined variable)
| -rw-r--r-- | cad/src/tools/SEMBot/Pylint.rcfile | 7 | ||||
| -rwxr-xr-x | cad/src/tools/SEMBot/index.php | 7 | ||||
| -rwxr-xr-x | cad/src/tools/SEMBot/runPylint.sh | 15 |
3 files changed, 21 insertions, 8 deletions
diff --git a/cad/src/tools/SEMBot/Pylint.rcfile b/cad/src/tools/SEMBot/Pylint.rcfile index 60416e98c..af4e66053 100644 --- a/cad/src/tools/SEMBot/Pylint.rcfile +++ b/cad/src/tools/SEMBot/Pylint.rcfile @@ -65,10 +65,9 @@ disable-msg=C0103, W0107, W0603, W0142 # explanations of some enabled but potentially confusing messages: # E1111 = Assigning to function call which doesn't return -- this warns about var = func(...) # when func() doesn't return anything or always returns None (I think). - -# consider adding summary reports for the following, for all files: [bruce 071107] -# W0311 - Bad indentation -# ? - Undefined variable (I forget the exact wording -- happens when you forget an import) +# +# see also runPylint.sh, which generates summary reports of certain messages +# over all source files, e.g. W0611 and E0602. [REPORTS] diff --git a/cad/src/tools/SEMBot/index.php b/cad/src/tools/SEMBot/index.php index 1dab06aa5..a2d28b9bd 100755 --- a/cad/src/tools/SEMBot/index.php +++ b/cad/src/tools/SEMBot/index.php @@ -115,7 +115,7 @@ Welcome to the Nanorex Software-Engineering Mechanisms Robot (SEMBot). <tr> <td align="right">Pylint: </td> <td><span class="summary-name"><?php include 'Pylint.result'; ?> out of 10.0</span></td> - <td><a href="SVN-D/cad/src/pylint_global.0.html">Detail</a> (Filtered: <a href="W0611.txt">W0611</a>*)</td></tr> + <td><a href="SVN-D/cad/src/pylint_global.0.html">Detail</a> (Filtered: <a href="W0611.txt">W0611</a>* <a href="E0602.txt">E0602</a>* )</td></tr> </table></td> </tr> @@ -132,7 +132,10 @@ Welcome to the Nanorex Software-Engineering Mechanisms Robot (SEMBot). <td class="summary"> Each convention, refactor, warning, and error message code is following by a brief description, but if that is insufficient, you can get a slightly more informative description for it here: <a href="http://www.logilab.org/card/wikiid/pylintfeatures">Pylint man page</a> <p> - * W0611 - Unused import warning. Emitted when an imported module or variable is not used.</td> + * W0611 - Unused import warning. Emitted when an imported module or variable is not used. + <p> + * E0602 - Undefined variable error. Emitted when a non-builtin symbol is used, but no definition or import of it can be found. + </td> </tr> </table> <pre> diff --git a/cad/src/tools/SEMBot/runPylint.sh b/cad/src/tools/SEMBot/runPylint.sh index 23cf64e5c..9e5bd5cea 100755 --- a/cad/src/tools/SEMBot/runPylint.sh +++ b/cad/src/tools/SEMBot/runPylint.sh @@ -50,7 +50,18 @@ rm pylint.?.result tmp.txt popd -# Generate a filtered report (W0611 - Unused import %s Used when an imported -# module or variable is not used. +# Generate filtered summary reports: +# (their output files have links in index.php) + +# W0611 - Unused import %s. Emitted when an imported module or variable is not used. grep W0611 SVN-D/cad/src/pylint_* > W0611.txt +# E0602 - Undefined variable %s. Emitted when a non-builtin symbol is used, but no +# definition or import of it can be found. +grep E0602 SVN-D/cad/src/pylint_* > E0602.txt + +# consider adding summary reports for the following: +# W0311 - Bad indentation +# ? - tabs in indentation + +# end |