Index: third_party/pylint/lint.py |
diff --git a/third_party/pylint/lint.py b/third_party/pylint/lint.py |
new file mode 100644 |
index 0000000000000000000000000000000000000000..48ebdf266a4fe540aed3625d3801a2e63eae60e3 |
--- /dev/null |
+++ b/third_party/pylint/lint.py |
@@ -0,0 +1,934 @@ |
+# Copyright (c) 2003-2010 Sylvain Thenault (thenault@gmail.com). |
+# Copyright (c) 2003-2010 LOGILAB S.A. (Paris, FRANCE). |
+# http://www.logilab.fr/ -- mailto:contact@logilab.fr |
+# |
+# This program is free software; you can redistribute it and/or modify it under |
+# the terms of the GNU General Public License as published by the Free Software |
+# Foundation; either version 2 of the License, or (at your option) any later |
+# version. |
+# |
+# This program is distributed in the hope that it will be useful, but WITHOUT |
+# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS |
+# FOR A PARTICULAR PURPOSE. See the GNU General Public License for more details |
+# |
+# You should have received a copy of the GNU General Public License along with |
+# this program; if not, write to the Free Software Foundation, Inc., |
+# 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. |
+""" %prog [options] module_or_package |
+ |
+ Check that a module satisfy a coding standard (and more !). |
+ |
+ %prog --help |
+ |
+ Display this help message and exit. |
+ |
+ %prog --help-msg <msg-id>[,<msg-id>] |
+ |
+ Display help messages about given message identifiers and exit. |
+""" |
+ |
+# import this first to avoid builtin namespace pollution |
+from pylint.checkers import utils |
+ |
+import sys |
+import os |
+import re |
+import tokenize |
+from warnings import warn |
+ |
+from logilab.common.configuration import UnsupportedAction, OptionsManagerMixIn |
+from logilab.common.optik_ext import check_csv |
+from logilab.common.modutils import load_module_from_name |
+from logilab.common.interface import implements |
+from logilab.common.textutils import splitstrip |
+from logilab.common.ureports import Table, Text, Section |
+from logilab.common.__pkginfo__ import version as common_version |
+ |
+from logilab.astng import MANAGER, nodes, ASTNGBuildingException |
+from logilab.astng.__pkginfo__ import version as astng_version |
+ |
+from pylint.utils import PyLintASTWalker, UnknownMessage, MessagesHandlerMixIn,\ |
+ ReportsHandlerMixIn, MSG_TYPES, expand_modules |
+from pylint.interfaces import ILinter, IRawChecker, IASTNGChecker |
+from pylint.checkers import BaseRawChecker, EmptyReport, \ |
+ table_lines_from_stats |
+from pylint.reporters.text import TextReporter, ParseableTextReporter, \ |
+ VSTextReporter, ColorizedTextReporter |
+from pylint.reporters.html import HTMLReporter |
+from pylint import config |
+ |
+from pylint.__pkginfo__ import version |
+ |
+ |
+OPTION_RGX = re.compile('\s*#*\s*pylint:(.*)') |
+REPORTER_OPT_MAP = {'text': TextReporter, |
+ 'parseable': ParseableTextReporter, |
+ 'msvs': VSTextReporter, |
+ 'colorized': ColorizedTextReporter, |
+ 'html': HTMLReporter,} |
+ |
+ |
+# Python Linter class ######################################################### |
+ |
+MSGS = { |
+ 'F0001': ('%s', |
+ 'Used when an error occurred preventing the analysis of a \ |
+ module (unable to find it for instance).'), |
+ 'F0002': ('%s: %s', |
+ 'Used when an unexpected error occurred while building the ASTNG \ |
+ representation. This is usually accompanied by a traceback. \ |
+ Please report such errors !'), |
+ 'F0003': ('ignored builtin module %s', |
+ 'Used to indicate that the user asked to analyze a builtin module\ |
+ which has been skipped.'), |
+ 'F0004': ('unexpected inferred value %s', |
+ 'Used to indicate that some value of an unexpected type has been \ |
+ inferred.'), |
+ 'F0010': ('error while code parsing: %s', |
+ 'Used when an exception occured while building the ASTNG \ |
+ representation which could be handled by astng.'), |
+ |
+ |
+ 'I0001': ('Unable to run raw checkers on built-in module %s', |
+ 'Used to inform that a built-in module has not been checked \ |
+ using the raw checkers.'), |
+ |
+ 'I0010': ('Unable to consider inline option %r', |
+ 'Used when an inline option is either badly formatted or can\'t \ |
+ be used inside modules.'), |
+ |
+ 'I0011': ('Locally disabling %s', |
+ 'Used when an inline option disables a message or a messages \ |
+ category.'), |
+ 'I0012': ('Locally enabling %s', |
+ 'Used when an inline option enables a message or a messages \ |
+ category.'), |
+ 'I0013': ('Ignoring entire file', |
+ 'Used to inform that the file will not be checked'), |
+ |
+ |
+ 'E0001': ('%s', |
+ 'Used when a syntax error is raised for a module.'), |
+ |
+ 'E0011': ('Unrecognized file option %r', |
+ 'Used when an unknown inline option is encountered.'), |
+ 'E0012': ('Bad option value %r', |
+ 'Used when a bad value for an inline option is encountered.'), |
+ } |
+ |
+ |
+class PyLinter(OptionsManagerMixIn, MessagesHandlerMixIn, ReportsHandlerMixIn, |
+ BaseRawChecker): |
+ """lint Python modules using external checkers. |
+ |
+ This is the main checker controlling the other ones and the reports |
+ generation. It is itself both a raw checker and an astng checker in order |
+ to: |
+ * handle message activation / deactivation at the module level |
+ * handle some basic but necessary stats'data (number of classes, methods...) |
+ |
+ IDE plugins developpers: you may have to call |
+ `logilab.astng.builder.MANAGER.astng_cache.clear()` accross run if you want |
+ to ensure the latest code version is actually checked. |
+ """ |
+ |
+ __implements__ = (ILinter, IRawChecker) |
+ |
+ name = 'master' |
+ priority = 0 |
+ level = 0 |
+ msgs = MSGS |
+ may_be_disabled = False |
+ |
+ @staticmethod |
+ def make_options(): |
+ return (('ignore', |
+ {'type' : 'csv', 'metavar' : '<file>[,<file>...]', |
+ 'dest' : 'black_list', 'default' : ('CVS',), |
+ 'help' : 'Add files or directories to the blacklist. \ |
+They should be base names, not paths.'}), |
+ ('persistent', |
+ {'default': True, 'type' : 'yn', 'metavar' : '<y_or_n>', |
+ 'level': 1, |
+ 'help' : 'Pickle collected data for later comparisons.'}), |
+ |
+ ('load-plugins', |
+ {'type' : 'csv', 'metavar' : '<modules>', 'default' : (), |
+ 'level': 1, |
+ 'help' : 'List of plugins (as comma separated values of \ |
+python modules names) to load, usually to register additional checkers.'}), |
+ |
+ ('output-format', |
+ {'default': 'text', 'type': 'choice', 'metavar' : '<format>', |
+ 'choices': REPORTER_OPT_MAP.keys(), |
+ 'short': 'f', |
+ 'group': 'Reports', |
+ 'help' : 'Set the output format. Available formats are text,\ |
+ parseable, colorized, msvs (visual studio) and html'}), |
+ |
+ ('include-ids', |
+ {'type' : 'yn', 'metavar' : '<y_or_n>', 'default' : 0, |
+ 'short': 'i', |
+ 'group': 'Reports', |
+ 'help' : 'Include message\'s id in output'}), |
+ |
+ ('files-output', |
+ {'default': 0, 'type' : 'yn', 'metavar' : '<y_or_n>', |
+ 'group': 'Reports', 'level': 1, |
+ 'help' : 'Put messages in a separate file for each module / \ |
+package specified on the command line instead of printing them on stdout. \ |
+Reports (if any) will be written in a file name "pylint_global.[txt|html]".'}), |
+ |
+ ('reports', |
+ {'default': 1, 'type' : 'yn', 'metavar' : '<y_or_n>', |
+ 'short': 'r', |
+ 'group': 'Reports', |
+ 'help' : 'Tells whether to display a full report or only the\ |
+ messages'}), |
+ |
+ ('evaluation', |
+ {'type' : 'string', 'metavar' : '<python_expression>', |
+ 'group': 'Reports', 'level': 1, |
+ 'default': '10.0 - ((float(5 * error + warning + refactor + \ |
+convention) / statement) * 10)', |
+ 'help' : 'Python expression which should return a note less \ |
+than 10 (10 is the highest note). You have access to the variables errors \ |
+warning, statement which respectively contain the number of errors / warnings\ |
+ messages and the total number of statements analyzed. This is used by the \ |
+ global evaluation report (RP0004).'}), |
+ |
+ ('comment', |
+ {'default': 0, 'type' : 'yn', 'metavar' : '<y_or_n>', |
+ 'group': 'Reports', 'level': 1, |
+ 'help' : 'Add a comment according to your evaluation note. \ |
+This is used by the global evaluation report (RP0004).'}), |
+ |
+ ('enable', |
+ {'type' : 'csv', 'metavar': '<msg ids>', |
+ 'short': 'e', |
+ 'group': 'Messages control', |
+ 'help' : 'Enable the message, report, category or checker with the ' |
+ 'given id(s). You can either give multiple identifier ' |
+ 'separated by comma (,) or put this option multiple time.'}), |
+ |
+ ('disable', |
+ {'type' : 'csv', 'metavar': '<msg ids>', |
+ 'short': 'd', |
+ 'group': 'Messages control', |
+ 'help' : 'Disable the message, report, category or checker ' |
+ 'with the given id(s). You can either give multiple identifier' |
+ ' separated by comma (,) or put this option multiple time ' |
+ '(only on the command line, not in the configuration file ' |
+ 'where it should appear only once).'}), |
+ ) |
+ |
+ option_groups = ( |
+ ('Messages control', 'Options controling analysis messages'), |
+ ('Reports', 'Options related to output formating and reporting'), |
+ ) |
+ |
+ def __init__(self, options=(), reporter=None, option_groups=(), |
+ pylintrc=None): |
+ # some stuff has to be done before ancestors initialization... |
+ # |
+ # checkers / reporter / astng manager |
+ self.reporter = None |
+ self._checkers = {} |
+ self._ignore_file = False |
+ # visit variables |
+ self.base_name = None |
+ self.base_file = None |
+ self.current_name = None |
+ self.current_file = None |
+ self.stats = None |
+ # init options |
+ self.options = options + PyLinter.make_options() |
+ self.option_groups = option_groups + PyLinter.option_groups |
+ self._options_methods = { |
+ 'enable': self.enable, |
+ 'disable': self.disable} |
+ self._bw_options_methods = {'disable-msg': self.disable, |
+ 'enable-msg': self.enable} |
+ full_version = '%%prog %s, \nastng %s, common %s\nPython %s' % ( |
+ version, astng_version, common_version, sys.version) |
+ OptionsManagerMixIn.__init__(self, usage=__doc__, |
+ version=full_version, |
+ config_file=pylintrc or config.PYLINTRC) |
+ MessagesHandlerMixIn.__init__(self) |
+ ReportsHandlerMixIn.__init__(self) |
+ BaseRawChecker.__init__(self) |
+ # provided reports |
+ self.reports = (('RP0001', 'Messages by category', |
+ report_total_messages_stats), |
+ ('RP0002', '% errors / warnings by module', |
+ report_messages_by_module_stats), |
+ ('RP0003', 'Messages', |
+ report_messages_stats), |
+ ('RP0004', 'Global evaluation', |
+ self.report_evaluation), |
+ ) |
+ self.register_checker(self) |
+ self._dynamic_plugins = [] |
+ self.load_provider_defaults() |
+ self.set_reporter(reporter or TextReporter(sys.stdout)) |
+ |
+ def load_default_plugins(self): |
+ from pylint import checkers |
+ checkers.initialize(self) |
+ |
+ def load_plugin_modules(self, modnames): |
+ """take a list of module names which are pylint plugins and load |
+ and register them |
+ """ |
+ for modname in modnames: |
+ if modname in self._dynamic_plugins: |
+ continue |
+ self._dynamic_plugins.append(modname) |
+ module = load_module_from_name(modname) |
+ module.register(self) |
+ |
+ def set_reporter(self, reporter): |
+ """set the reporter used to display messages and reports""" |
+ self.reporter = reporter |
+ reporter.linter = self |
+ |
+ def set_option(self, optname, value, action=None, optdict=None): |
+ """overridden from configuration.OptionsProviderMixin to handle some |
+ special options |
+ """ |
+ if optname in self._options_methods or optname in self._bw_options_methods: |
+ if value: |
+ try: |
+ meth = self._options_methods[optname] |
+ except KeyError: |
+ meth = self._bw_options_methods[optname] |
+ warn('%s is deprecated, replace it by %s' % ( |
+ optname, optname.split('-')[0]), DeprecationWarning) |
+ value = check_csv(None, optname, value) |
+ if isinstance(value, (list, tuple)): |
+ for _id in value : |
+ meth(_id) |
+ else : |
+ meth(value) |
+ elif optname == 'output-format': |
+ self.set_reporter(REPORTER_OPT_MAP[value.lower()]()) |
+ try: |
+ BaseRawChecker.set_option(self, optname, value, action, optdict) |
+ except UnsupportedAction: |
+ print >> sys.stderr, 'option %s can\'t be read from config file' % \ |
+ optname |
+ |
+ # checkers manipulation methods ############################################ |
+ |
+ def register_checker(self, checker): |
+ """register a new checker |
+ |
+ checker is an object implementing IRawChecker or / and IASTNGChecker |
+ """ |
+ assert checker.priority <= 0, 'checker priority can\'t be >= 0' |
+ self._checkers.setdefault(checker.name, []).append(checker) |
+ for r_id, r_title, r_cb in checker.reports: |
+ self.register_report(r_id, r_title, r_cb, checker) |
+ self.register_options_provider(checker) |
+ if hasattr(checker, 'msgs'): |
+ self.register_messages(checker) |
+ checker.load_defaults() |
+ |
+ def disable_noerror_messages(self): |
+ for msgcat, msgids in self._msgs_by_category.iteritems(): |
+ if msgcat == 'E': |
+ for msgid in msgids: |
+ self.enable(msgid) |
+ else: |
+ for msgid in msgids: |
+ self.disable(msgid) |
+ |
+ def disable_reporters(self): |
+ """disable all reporters""" |
+ for reporters in self._reports.values(): |
+ for report_id, _title, _cb in reporters: |
+ self.disable_report(report_id) |
+ |
+ def error_mode(self): |
+ """error mode: enable only errors; no reports, no persistent""" |
+ self.disable_noerror_messages() |
+ self.disable('miscellaneous') |
+ self.set_option('reports', False) |
+ self.set_option('persistent', False) |
+ |
+ # block level option handling ############################################# |
+ # |
+ # see func_block_disable_msg.py test case for expected behaviour |
+ |
+ def process_tokens(self, tokens): |
+ """process tokens from the current module to search for module/block |
+ level options |
+ """ |
+ comment = tokenize.COMMENT |
+ newline = tokenize.NEWLINE |
+ for (tok_type, _, start, _, line) in tokens: |
+ if tok_type not in (comment, newline): |
+ continue |
+ match = OPTION_RGX.search(line) |
+ if match is None: |
+ continue |
+ if match.group(1).strip() == "disable-all": |
+ self.add_message('I0013', line=start[0]) |
+ self._ignore_file = True |
+ return |
+ try: |
+ opt, value = match.group(1).split('=', 1) |
+ except ValueError: |
+ self.add_message('I0010', args=match.group(1).strip(), |
+ line=start[0]) |
+ continue |
+ opt = opt.strip() |
+ if opt in self._options_methods or opt in self._bw_options_methods: |
+ try: |
+ meth = self._options_methods[opt] |
+ except KeyError: |
+ meth = self._bw_options_methods[opt] |
+ warn('%s is deprecated, replace it by %s (%s, line %s)' % ( |
+ opt, opt.split('-')[0], self.current_file, line), |
+ DeprecationWarning) |
+ for msgid in splitstrip(value): |
+ try: |
+ meth(msgid, 'module', start[0]) |
+ except UnknownMessage: |
+ self.add_message('E0012', args=msgid, line=start[0]) |
+ else: |
+ self.add_message('E0011', args=opt, line=start[0]) |
+ |
+ def collect_block_lines(self, node, msg_state): |
+ """walk ast to collect block level options line numbers""" |
+ # recurse on children (depth first) |
+ for child in node.get_children(): |
+ self.collect_block_lines(child, msg_state) |
+ first = node.fromlineno |
+ last = node.tolineno |
+ # first child line number used to distinguish between disable |
+ # which are the first child of scoped node with those defined later. |
+ # For instance in the code below: |
+ # |
+ # 1. def meth8(self): |
+ # 2. """test late disabling""" |
+ # 3. # pylint: disable=E1102 |
+ # 4. print self.blip |
+ # 5. # pylint: disable=E1101 |
+ # 6. print self.bla |
+ # |
+ # E1102 should be disabled from line 1 to 6 while E1101 from line 5 to 6 |
+ # |
+ # this is necessary to disable locally messages applying to class / |
+ # function using their fromlineno |
+ if isinstance(node, (nodes.Module, nodes.Class, nodes.Function)) and node.body: |
+ firstchildlineno = node.body[0].fromlineno |
+ else: |
+ firstchildlineno = last |
+ for msgid, lines in msg_state.iteritems(): |
+ for lineno, state in lines.items(): |
+ if first <= lineno <= last: |
+ if lineno > firstchildlineno: |
+ state = True |
+ # set state for all lines for this block |
+ first, last = node.block_range(lineno) |
+ for line in xrange(first, last+1): |
+ # do not override existing entries |
+ if not line in self._module_msgs_state.get(msgid, ()): |
+ if line in lines: # state change in the same block |
+ state = lines[line] |
+ try: |
+ self._module_msgs_state[msgid][line] = state |
+ except KeyError: |
+ self._module_msgs_state[msgid] = {line: state} |
+ del lines[lineno] |
+ |
+ |
+ # code checking methods ################################################### |
+ |
+ def get_checkers(self): |
+ """return all available checkers as a list""" |
+ return [self] + [c for checkers in self._checkers.values() |
+ for c in checkers if c is not self] |
+ |
+ def prepare_checkers(self): |
+ """return checkers needed for activated messages and reports""" |
+ if not self.config.reports: |
+ self.disable_reporters() |
+ # get needed checkers |
+ neededcheckers = [self] |
+ for checker in self.get_checkers()[1:]: |
+ messages = set(msg for msg in checker.msgs |
+ if self.is_message_enabled(msg)) |
+ if (messages or |
+ any(self.report_is_enabled(r[0]) for r in checker.reports)): |
+ neededcheckers.append(checker) |
+ checker.active_msgs = messages |
+ return neededcheckers |
+ |
+ def check(self, files_or_modules): |
+ """main checking entry: check a list of files or modules from their |
+ name. |
+ """ |
+ self.reporter.include_ids = self.config.include_ids |
+ if not isinstance(files_or_modules, (list, tuple)): |
+ files_or_modules = (files_or_modules,) |
+ walker = PyLintASTWalker(self) |
+ checkers = self.prepare_checkers() |
+ rawcheckers = [c for c in checkers if implements(c, IRawChecker) |
+ and c is not self] |
+ # notify global begin |
+ for checker in checkers: |
+ checker.open() |
+ if implements(checker, IASTNGChecker): |
+ walker.add_checker(checker) |
+ # build ast and check modules or packages |
+ for descr in self.expand_files(files_or_modules): |
+ modname, filepath = descr['name'], descr['path'] |
+ self.set_current_module(modname, filepath) |
+ # get the module representation |
+ astng = self.get_astng(filepath, modname) |
+ if astng is None: |
+ continue |
+ self.base_name = descr['basename'] |
+ self.base_file = descr['basepath'] |
+ if self.config.files_output: |
+ reportfile = 'pylint_%s.%s' % (modname, self.reporter.extension) |
+ self.reporter.set_output(open(reportfile, 'w')) |
+ self._ignore_file = False |
+ # fix the current file (if the source file was not available or |
+ # if it's actually a c extension) |
+ self.current_file = astng.file |
+ self.check_astng_module(astng, walker, rawcheckers) |
+ # notify global end |
+ self.set_current_module('') |
+ self.stats['statement'] = walker.nbstatements |
+ checkers.reverse() |
+ for checker in checkers: |
+ checker.close() |
+ |
+ def expand_files(self, modules): |
+ """get modules and errors from a list of modules and handle errors |
+ """ |
+ result, errors = expand_modules(modules, self.config.black_list) |
+ for error in errors: |
+ message = modname = error["mod"] |
+ key = error["key"] |
+ self.set_current_module(modname) |
+ if key == "F0001": |
+ message = str(error["ex"]).replace(os.getcwd() + os.sep, '') |
+ self.add_message(key, args=message) |
+ return result |
+ |
+ def set_current_module(self, modname, filepath=None): |
+ """set the name of the currently analyzed module and |
+ init statistics for it |
+ """ |
+ if not modname and filepath is None: |
+ return |
+ self.current_name = modname |
+ self.current_file = filepath or modname |
+ self.stats['by_module'][modname] = {} |
+ self.stats['by_module'][modname]['statement'] = 0 |
+ for msg_cat in MSG_TYPES.values(): |
+ self.stats['by_module'][modname][msg_cat] = 0 |
+ # XXX hack, to be correct we need to keep module_msgs_state |
+ # for every analyzed module (the problem stands with localized |
+ # messages which are only detected in the .close step) |
+ if modname: |
+ self._module_msgs_state = {} |
+ self._module_msg_cats_state = {} |
+ |
+ def get_astng(self, filepath, modname): |
+ """return a astng representation for a module""" |
+ try: |
+ return MANAGER.astng_from_file(filepath, modname, source=True) |
+ except SyntaxError, ex: |
+ self.add_message('E0001', line=ex.lineno, args=ex.msg) |
+ except ASTNGBuildingException, ex: |
+ self.add_message('F0010', args=ex) |
+ except Exception, ex: |
+ import traceback |
+ traceback.print_exc() |
+ self.add_message('F0002', args=(ex.__class__, ex)) |
+ |
+ def check_astng_module(self, astng, walker, rawcheckers): |
+ """check a module from its astng representation, real work""" |
+ # call raw checkers if possible |
+ if not astng.pure_python: |
+ self.add_message('I0001', args=astng.name) |
+ else: |
+ #assert astng.file.endswith('.py') |
+ # invoke IRawChecker interface on self to fetch module/block |
+ # level options |
+ self.process_module(astng) |
+ if self._ignore_file: |
+ return False |
+ # walk ast to collect line numbers |
+ orig_state = self._module_msgs_state.copy() |
+ self._module_msgs_state = {} |
+ self.collect_block_lines(astng, orig_state) |
+ for checker in rawcheckers: |
+ checker.process_module(astng) |
+ # generate events to astng checkers |
+ walker.walk(astng) |
+ return True |
+ |
+ # IASTNGChecker interface ################################################# |
+ |
+ def open(self): |
+ """initialize counters""" |
+ self.stats = { 'by_module' : {}, |
+ 'by_msg' : {}, |
+ } |
+ for msg_cat in MSG_TYPES.values(): |
+ self.stats[msg_cat] = 0 |
+ |
+ def close(self): |
+ """close the whole package /module, it's time to make reports ! |
+ |
+ if persistent run, pickle results for later comparison |
+ """ |
+ if self.base_name is not None: |
+ # load old results if any |
+ old_stats = config.load_results(self.base_name) |
+ if self.config.reports: |
+ self.make_reports(self.stats, old_stats) |
+ elif self.config.output_format == 'html': |
+ self.reporter.display_results(Section()) |
+ # save results if persistent run |
+ if self.config.persistent: |
+ config.save_results(self.stats, self.base_name) |
+ |
+ # specific reports ######################################################## |
+ |
+ def report_evaluation(self, sect, stats, old_stats): |
+ """make the global evaluation report""" |
+ # check with at least check 1 statements (usually 0 when there is a |
+ # syntax error preventing pylint from further processing) |
+ if stats['statement'] == 0: |
+ raise EmptyReport() |
+ # get a global note for the code |
+ evaluation = self.config.evaluation |
+ try: |
+ note = eval(evaluation, {}, self.stats) |
+ except Exception, ex: |
+ msg = 'An exception occurred while rating: %s' % ex |
+ else: |
+ stats['global_note'] = note |
+ msg = 'Your code has been rated at %.2f/10' % note |
+ if 'global_note' in old_stats: |
+ msg += ' (previous run: %.2f/10)' % old_stats['global_note'] |
+ if self.config.comment: |
+ msg = '%s\n%s' % (msg, config.get_note_message(note)) |
+ sect.append(Text(msg)) |
+ |
+# some reporting functions #################################################### |
+ |
+def report_total_messages_stats(sect, stats, old_stats): |
+ """make total errors / warnings report""" |
+ lines = ['type', 'number', 'previous', 'difference'] |
+ lines += table_lines_from_stats(stats, old_stats, |
+ ('convention', 'refactor', |
+ 'warning', 'error')) |
+ sect.append(Table(children=lines, cols=4, rheaders=1)) |
+ |
+def report_messages_stats(sect, stats, _): |
+ """make messages type report""" |
+ if not stats['by_msg']: |
+ # don't print this report when we didn't detected any errors |
+ raise EmptyReport() |
+ in_order = sorted([(value, msg_id) |
+ for msg_id, value in stats['by_msg'].items() |
+ if not msg_id.startswith('I')]) |
+ in_order.reverse() |
+ lines = ('message id', 'occurrences') |
+ for value, msg_id in in_order: |
+ lines += (msg_id, str(value)) |
+ sect.append(Table(children=lines, cols=2, rheaders=1)) |
+ |
+def report_messages_by_module_stats(sect, stats, _): |
+ """make errors / warnings by modules report""" |
+ if len(stats['by_module']) == 1: |
+ # don't print this report when we are analysing a single module |
+ raise EmptyReport() |
+ by_mod = {} |
+ for m_type in ('fatal', 'error', 'warning', 'refactor', 'convention'): |
+ total = stats[m_type] |
+ for module in stats['by_module'].keys(): |
+ mod_total = stats['by_module'][module][m_type] |
+ if total == 0: |
+ percent = 0 |
+ else: |
+ percent = float((mod_total)*100) / total |
+ by_mod.setdefault(module, {})[m_type] = percent |
+ sorted_result = [] |
+ for module, mod_info in by_mod.items(): |
+ sorted_result.append((mod_info['error'], |
+ mod_info['warning'], |
+ mod_info['refactor'], |
+ mod_info['convention'], |
+ module)) |
+ sorted_result.sort() |
+ sorted_result.reverse() |
+ lines = ['module', 'error', 'warning', 'refactor', 'convention'] |
+ for line in sorted_result: |
+ if line[0] == 0 and line[1] == 0: |
+ break |
+ lines.append(line[-1]) |
+ for val in line[:-1]: |
+ lines.append('%.2f' % val) |
+ if len(lines) == 5: |
+ raise EmptyReport() |
+ sect.append(Table(children=lines, cols=5, rheaders=1)) |
+ |
+ |
+# utilities ################################################################### |
+ |
+# this may help to import modules using gettext |
+ |
+try: |
+ __builtins__._ = str |
+except AttributeError: |
+ __builtins__['_'] = str |
+ |
+ |
+class ArgumentPreprocessingError(Exception): |
+ """Raised if an error occurs during argument preprocessing.""" |
+ |
+ |
+def preprocess_options(args, search_for): |
+ """look for some options (keys of <search_for>) which have to be processed |
+ before others |
+ |
+ values of <search_for> are callback functions to call when the option is |
+ found |
+ """ |
+ i = 0 |
+ while i < len(args): |
+ arg = args[i] |
+ if arg.startswith('--'): |
+ try: |
+ option, val = arg[2:].split('=', 1) |
+ except ValueError: |
+ option, val = arg[2:], None |
+ try: |
+ cb, takearg = search_for[option] |
+ del args[i] |
+ if takearg and val is None: |
+ if i >= len(args) or args[i].startswith('-'): |
+ raise ArgumentPreprocessingError(arg) |
+ val = args[i] |
+ del args[i] |
+ cb(option, val) |
+ except KeyError: |
+ i += 1 |
+ else: |
+ i += 1 |
+ |
+class Run: |
+ """helper class to use as main for pylint : |
+ |
+ run(*sys.argv[1:]) |
+ """ |
+ LinterClass = PyLinter |
+ option_groups = ( |
+ ('Commands', 'Options which are actually commands. Options in this \ |
+group are mutually exclusive.'), |
+ ) |
+ |
+ def __init__(self, args, reporter=None, exit=True): |
+ self._rcfile = None |
+ self._plugins = [] |
+ try: |
+ preprocess_options(args, { |
+ # option: (callback, takearg) |
+ 'rcfile': (self.cb_set_rcfile, True), |
+ 'load-plugins': (self.cb_add_plugins, True), |
+ }) |
+ except ArgumentPreprocessingError, e: |
+ print >> sys.stderr, 'Argument %s expects a value.' % (e.args[0],) |
+ sys.exit(32) |
+ |
+ self.linter = linter = self.LinterClass(( |
+ ('rcfile', |
+ {'action' : 'callback', 'callback' : lambda *args: 1, |
+ 'type': 'string', 'metavar': '<file>', |
+ 'help' : 'Specify a configuration file.'}), |
+ |
+ ('init-hook', |
+ {'action' : 'callback', 'type' : 'string', 'metavar': '<code>', |
+ 'callback' : cb_init_hook, 'level': 1, |
+ 'help' : 'Python code to execute, usually for sys.path \ |
+manipulation such as pygtk.require().'}), |
+ |
+ ('help-msg', |
+ {'action' : 'callback', 'type' : 'string', 'metavar': '<msg-id>', |
+ 'callback' : self.cb_help_message, |
+ 'group': 'Commands', |
+ 'help' : '''Display a help message for the given message id and \ |
+exit. The value may be a comma separated list of message ids.'''}), |
+ |
+ ('list-msgs', |
+ {'action' : 'callback', 'metavar': '<msg-id>', |
+ 'callback' : self.cb_list_messages, |
+ 'group': 'Commands', 'level': 1, |
+ 'help' : "Generate pylint's messages."}), |
+ |
+ ('full-documentation', |
+ {'action' : 'callback', 'metavar': '<msg-id>', |
+ 'callback' : self.cb_full_documentation, |
+ 'group': 'Commands', 'level': 1, |
+ 'help' : "Generate pylint's full documentation."}), |
+ |
+ ('generate-rcfile', |
+ {'action' : 'callback', 'callback' : self.cb_generate_config, |
+ 'group': 'Commands', |
+ 'help' : '''Generate a sample configuration file according to \ |
+the current configuration. You can put other options before this one to get \ |
+them in the generated configuration.'''}), |
+ |
+ ('generate-man', |
+ {'action' : 'callback', 'callback' : self.cb_generate_manpage, |
+ 'group': 'Commands', |
+ 'help' : "Generate pylint's man page.",'hide': True}), |
+ |
+ ('errors-only', |
+ {'action' : 'callback', 'callback' : self.cb_error_mode, |
+ 'short': 'E', |
+ 'help' : '''In error mode, checkers without error messages are \ |
+disabled and for others, only the ERROR messages are displayed, and no reports \ |
+are done by default'''}), |
+ |
+ ('profile', |
+ {'type' : 'yn', 'metavar' : '<y_or_n>', |
+ 'default': False, 'hide': True, |
+ 'help' : 'Profiled execution.'}), |
+ |
+ ), option_groups=self.option_groups, |
+ reporter=reporter, pylintrc=self._rcfile) |
+ # register standard checkers |
+ linter.load_default_plugins() |
+ # load command line plugins |
+ linter.load_plugin_modules(self._plugins) |
+ # add some help section |
+ linter.add_help_section('Environment variables', config.ENV_HELP, level=1) |
+ linter.add_help_section('Output', ''' |
+Using the default text output, the message format is : |
+ |
+ MESSAGE_TYPE: LINE_NUM:[OBJECT:] MESSAGE |
+ |
+There are 5 kind of message types : |
+ * (C) convention, for programming standard violation |
+ * (R) refactor, for bad code smell |
+ * (W) warning, for python specific problems |
+ * (E) error, for probable bugs in the code |
+ * (F) fatal, if an error occurred which prevented pylint from doing further |
+processing. |
+ ''', level=1) |
+ linter.add_help_section('Output status code', ''' |
+Pylint should leave with following status code: |
+ * 0 if everything went fine |
+ * 1 if a fatal message was issued |
+ * 2 if an error message was issued |
+ * 4 if a warning message was issued |
+ * 8 if a refactor message was issued |
+ * 16 if a convention message was issued |
+ * 32 on usage error |
+ |
+status 1 to 16 will be bit-ORed so you can know which different categories has |
+been issued by analysing pylint output status code |
+ ''', level=1) |
+ # read configuration |
+ linter.disable('W0704') |
+ linter.read_config_file() |
+ # is there some additional plugins in the file configuration, in |
+ config_parser = linter.cfgfile_parser |
+ if config_parser.has_option('MASTER', 'load-plugins'): |
+ plugins = splitstrip(config_parser.get('MASTER', 'load-plugins')) |
+ linter.load_plugin_modules(plugins) |
+ # now we can load file config and command line, plugins (which can |
+ # provide options) have been registered |
+ linter.load_config_file() |
+ if reporter: |
+ # if a custom reporter is provided as argument, it may be overridden |
+ # by file parameters, so re-set it here, but before command line |
+ # parsing so it's still overrideable by command line option |
+ linter.set_reporter(reporter) |
+ try: |
+ args = linter.load_command_line_configuration(args) |
+ except SystemExit, exc: |
+ if exc.code == 2: # bad options |
+ exc.code = 32 |
+ raise |
+ if not args: |
+ print linter.help() |
+ sys.exit(32) |
+ # insert current working directory to the python path to have a correct |
+ # behaviour |
+ sys.path.insert(0, os.getcwd()) |
+ if self.linter.config.profile: |
+ print >> sys.stderr, '** profiled run' |
+ import cProfile, pstats |
+ cProfile.runctx('linter.check(%r)' % args, globals(), locals(), 'stones.prof' ) |
+ data = pstats.Stats('stones.prof') |
+ data.strip_dirs() |
+ data.sort_stats('time', 'calls') |
+ data.print_stats(30) |
+ else: |
+ linter.check(args) |
+ sys.path.pop(0) |
+ if exit: |
+ sys.exit(self.linter.msg_status) |
+ |
+ def cb_set_rcfile(self, name, value): |
+ """callback for option preprocessing (i.e. before optik parsing)""" |
+ self._rcfile = value |
+ |
+ def cb_add_plugins(self, name, value): |
+ """callback for option preprocessing (i.e. before optik parsing)""" |
+ self._plugins.extend(splitstrip(value)) |
+ |
+ def cb_error_mode(self, *args, **kwargs): |
+ """error mode: |
+ * disable all but error messages |
+ * disable the 'miscellaneous' checker which can be safely deactivated in |
+ debug |
+ * disable reports |
+ * do not save execution information |
+ """ |
+ self.linter.error_mode() |
+ |
+ def cb_generate_config(self, *args, **kwargs): |
+ """optik callback for sample config file generation""" |
+ self.linter.generate_config(skipsections=('COMMANDS',)) |
+ sys.exit(0) |
+ |
+ def cb_generate_manpage(self, *args, **kwargs): |
+ """optik callback for sample config file generation""" |
+ from pylint import __pkginfo__ |
+ self.linter.generate_manpage(__pkginfo__) |
+ sys.exit(0) |
+ |
+ def cb_help_message(self, option, optname, value, parser): |
+ """optik callback for printing some help about a particular message""" |
+ self.linter.help_message(splitstrip(value)) |
+ sys.exit(0) |
+ |
+ def cb_full_documentation(self, option, optname, value, parser): |
+ """optik callback for printing full documentation""" |
+ self.linter.print_full_documentation() |
+ sys.exit(0) |
+ |
+ def cb_list_messages(self, option, optname, value, parser): # FIXME |
+ """optik callback for printing available messages""" |
+ self.linter.list_messages() |
+ sys.exit(0) |
+ |
+def cb_init_hook(option, optname, value, parser): |
+ """exec arbitrary code to set sys.path for instance""" |
+ exec value |
+ |
+ |
+if __name__ == '__main__': |
+ Run(sys.argv[1:]) |