Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1261)

Unified Diff: third_party/pylint/lint.py

Issue 10447014: Add pylint to depot_tools. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/depot_tools
Patch Set: Fix unittests. Created 8 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « third_party/pylint/interfaces.py ('k') | third_party/pylint/pyreverse/__init__.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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:])
« no previous file with comments | « third_party/pylint/interfaces.py ('k') | third_party/pylint/pyreverse/__init__.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698