Plugins/CheckerPlugins/Pep8/Pep8Checker.py

changeset 2862
a1448560d7dc
parent 2302
f29e9405c851
child 2863
62171fa4a6a4
diff -r cdcbca0cea82 -r a1448560d7dc Plugins/CheckerPlugins/Pep8/Pep8Checker.py
--- a/Plugins/CheckerPlugins/Pep8/Pep8Checker.py	Fri Aug 23 15:20:55 2013 +0200
+++ b/Plugins/CheckerPlugins/Pep8/Pep8Checker.py	Fri Aug 23 20:08:42 2013 +0200
@@ -8,7 +8,7 @@
 """
 
 import os
-import optparse
+##import optparse
 
 from PyQt4.QtCore import QProcess, QCoreApplication
 
@@ -20,102 +20,102 @@
 from eric5config import getConfig
 
 
-class Pep8Checker(pep8.Checker):
-    """
-    Class implementing the PEP 8 checker.
-    """
-    def __init__(self, filename, lines, repeat=False,
-                 select="", ignore=""):
-        """
-        Constructor
-        
-        @param filename name of the file to check (string)
-        @param lines source of the file (list of strings)
-        @keyparam repeat flag indicating to repeat message categories (boolean)
-        @keyparam select list of message IDs to check for
-            (comma separated string)
-        @keyparam ignore list of message IDs to ignore
-            (comma separated string)
-        """
-        pep8.options = optparse.Values()
-        
-        pep8.options.verbose = 0
-        
-        pep8.options.repeat = repeat
-        if select:
-            pep8.options.select = [s.strip() for s in select.split(',')
-                                   if s.strip()]
-        else:
-            pep8.options.select = []
-        if ignore:
-            pep8.options.ignore = [i.strip() for i in ignore.split(',')
-                                   if i.strip()]
-        else:
-            pep8.options.ignore = []
-        pep8.options.physical_checks = pep8.find_checks('physical_line')
-        pep8.options.logical_checks = pep8.find_checks('logical_line')
-        pep8.options.counters = dict.fromkeys(pep8.BENCHMARK_KEYS, 0)
-        pep8.options.messages = {}
-        
-        pep8.Checker.__init__(self, filename, lines)
-        
-        self.messages = []
-        self.statistics = {}
-    
-    def __ignore_code(self, code):
-        """
-        Private method to check, if the message for the given code should
-        be ignored.
-        
-        If codes are selected and the code has a selected prefix and does not
-        have an ignored prefix, it is not ignored. If codes are selected and
-        the code does not have a selected prefix, it is ignored. If no codes
-        are selected, the code is ignored, if it has a prefix, that is
-        contained in the ignored codes.
-        
-        @param code code to be checked (string)
-        @return flag indicating, that the code should be ignored (boolean)
-        """
-        if pep8.options.select:
-            if code.startswith(tuple(pep8.options.select)):
-                if code.startswith(tuple(pep8.options.ignore)):
-                    return True
-                else:
-                    return False
-            else:
-                return True
-        else:
-            if code.startswith(tuple(pep8.options.ignore)):
-                return True
-            else:
-                return False
-    
-    def report_error_args(self, line_number, offset, code, check, *args):
-        """
-        Public method to collect the error messages.
-        
-        @param line_number line number of the issue (integer)
-        @param offset position within line of the issue (integer)
-        @param code message code (string)
-        @param check reference to the checker function (function)
-        @param args arguments for the message (list)
-        """
-        if self.__ignore_code(code):
-            return
-        
-        text = pep8.getMessage(code, *args)
-        if code in self.statistics:
-            self.statistics[code] += 1
-        else:
-            self.statistics[code] = 1
-        self.file_errors += 1
-        if self.statistics[code] == 1 or pep8.options.repeat:
-            self.messages.append(
-                (self.filename, self.line_offset + line_number,
-                 offset + 1, text)
-            )
-
-
+##class Pep8Checker(pep8.Checker):
+##    """
+##    Class implementing the PEP 8 checker.
+##    """
+##    def __init__(self, filename, lines, repeat=False,
+##                 select="", ignore=""):
+##        """
+##        Constructor
+##        
+##        @param filename name of the file to check (string)
+##        @param lines source of the file (list of strings)
+##        @keyparam repeat flag indicating to repeat message categories (boolean)
+##        @keyparam select list of message IDs to check for
+##            (comma separated string)
+##        @keyparam ignore list of message IDs to ignore
+##            (comma separated string)
+##        """
+##        pep8.options = optparse.Values()
+##        
+##        pep8.options.verbose = 0
+##        
+##        pep8.options.repeat = repeat
+##        if select:
+##            pep8.options.select = [s.strip() for s in select.split(',')
+##                                   if s.strip()]
+##        else:
+##            pep8.options.select = []
+##        if ignore:
+##            pep8.options.ignore = [i.strip() for i in ignore.split(',')
+##                                   if i.strip()]
+##        else:
+##            pep8.options.ignore = []
+##        pep8.options.physical_checks = pep8.find_checks('physical_line')
+##        pep8.options.logical_checks = pep8.find_checks('logical_line')
+##        pep8.options.counters = dict.fromkeys(pep8.BENCHMARK_KEYS, 0)
+##        pep8.options.messages = {}
+##        
+##        pep8.Checker.__init__(self, filename, lines)
+##        
+##        self.messages = []
+##        self.statistics = {}
+##    
+##    def __ignore_code(self, code):
+##        """
+##        Private method to check, if the message for the given code should
+##        be ignored.
+##        
+##        If codes are selected and the code has a selected prefix and does not
+##        have an ignored prefix, it is not ignored. If codes are selected and
+##        the code does not have a selected prefix, it is ignored. If no codes
+##        are selected, the code is ignored, if it has a prefix, that is
+##        contained in the ignored codes.
+##        
+##        @param code code to be checked (string)
+##        @return flag indicating, that the code should be ignored (boolean)
+##        """
+##        if pep8.options.select:
+##            if code.startswith(tuple(pep8.options.select)):
+##                if code.startswith(tuple(pep8.options.ignore)):
+##                    return True
+##                else:
+##                    return False
+##            else:
+##                return True
+##        else:
+##            if code.startswith(tuple(pep8.options.ignore)):
+##                return True
+##            else:
+##                return False
+##    
+##    def report_error_args(self, line_number, offset, code, check, *args):
+##        """
+##        Public method to collect the error messages.
+##        
+##        @param line_number line number of the issue (integer)
+##        @param offset position within line of the issue (integer)
+##        @param code message code (string)
+##        @param check reference to the checker function (function)
+##        @param args arguments for the message (list)
+##        """
+##        if self.__ignore_code(code):
+##            return
+##        
+##        text = pep8.getMessage(code, *args)
+##        if code in self.statistics:
+##            self.statistics[code] += 1
+##        else:
+##            self.statistics[code] = 1
+##        self.file_errors += 1
+##        if self.statistics[code] == 1 or pep8.options.repeat:
+##            self.messages.append(
+##                (self.filename, self.line_offset + line_number,
+##                 offset + 1, text)
+##            )
+##
+##
 class Pep8Py2Checker(object):
     """
     Class implementing the PEP 8 checker interface for Python 2.

eric ide

mercurial