Report E266 instead of E265 when the block common starts with multiple #; issue #270

Author: florentx

CHANGES.txt

@@ -9,6 +9,9 @@ Bug fixes:
 
 * Skip the traceback on "Broken pipe" signal. (Issue #275)
 
+* Report E266 instead of E265 when the block comment starts with
+  multiple ``#``. (Issue #270)
+
 
 1.5.6 (2014-04-14)
 ------------------

docs/intro.rst

@@ -263,6 +263,8 @@ This is the current list of error and warning codes:
 +----------+----------------------------------------------------------------------+
 | E265     | block comment should start with '# '                                 |
 +----------+----------------------------------------------------------------------+
+| E266     | too many leading '#' for block comment                               |
++----------+----------------------------------------------------------------------+
 +----------+----------------------------------------------------------------------+
 | E271     | multiple spaces after keyword                                        |
 +----------+----------------------------------------------------------------------+

pep8.py

@@ -787,6 +787,7 @@ def whitespace_before_comment(logical_line, tokens):
     E262: x = x + 1  #Increment x
     E262: x = x + 1  #  Increment x
     E265: #Block comment
+    E266: ### Block comment
     """
     prev_end = (0, 0)
     for token_type, text, start, end, line in tokens:
@@ -797,13 +798,15 @@ def whitespace_before_comment(logical_line, tokens):
                     yield (prev_end,
                            "E261 at least two spaces before inline comment")
             symbol, sp, comment = text.partition(' ')
-            bad_prefix = symbol not in ('#', '#:')
+            bad_prefix = symbol not in '#:' and (symbol.lstrip('#')[:1] or '#')
             if inline_comment:
-                if bad_prefix or comment[:1].isspace():
+                if bad_prefix or comment[:1] in WHITESPACE:
                     yield start, "E262 inline comment should start with '# '"
-            elif bad_prefix:
-                if text.rstrip('#') and (start[0] > 1 or symbol[1] != '!'):
+            elif bad_prefix and (bad_prefix != '!' or start[0] > 1):
+                if bad_prefix != '#':
                     yield start, "E265 block comment should start with '# '"
+                elif comment:
+                    yield start, "E266 too many leading '#' for block comment"
         elif token_type != tokenize.NL:
             prev_end = end
 

testsuite/E26.py

@@ -13,6 +13,21 @@
 m = 42
 #! This is important
 mx = 42 - 42
+#: E266:3:5 E266:6:5
+def how_it_feel(r):
+
+    ### This is a variable ###
+    a = 42
+
+    ### Of course it is unused
+    return
+#: E265:1:1 E266:2:1
+##if DEBUG:
+##    logging.error()
+#: W291:1:42
+######################################### 
+#:
+
 #: Okay
 #!/usr/bin/env python
 
@@ -34,3 +49,11 @@ def oof():
     """
     #foo not parsed
     """
+
+    ###########################################################################
+    #                               A SEPARATOR                               #
+    ###########################################################################
+
+    # ####################################################################### #
+    # ########################## another separator ########################## #
+    # ####################################################################### #