Fix a false positive E124 for hanging indent: issue #254

Author: florentx

CHANGES.txt

@@ -15,6 +15,8 @@ Changelog
 * Fix ``StyleGuide`` to parse the local configuration if the
   keyword argument ``paths`` is specified. (Issue #246)
 
+* Fix a false positive E124 for hanging indent. (Issue #254)
+
 
 1.4.6 (2013-07-02)
 ------------------

pep8.py

@@ -473,22 +473,23 @@ def continued_indentation(logical_line, tokens, indent_level, hang_closing,
                 # closing bracket matches indentation of opening bracket's line
                 if hang_closing:
                     yield start, "E133 closing bracket is missing indentation"
+            elif indent[depth] and start[1] < indent[depth]:
+                if visual_indent is not True:
+                    # visual indent is broken
+                    yield (start, "E128 continuation line "
+                           "under-indented for visual indent")
+            elif hang == 4 or (indent_next and rel_indent[row] == 8):
+                # hanging indent is verified
+                if close_bracket and not hang_closing:
+                    yield (start, "E123 closing bracket does not match "
+                           "indentation of opening bracket's line")
             elif visual_indent is True:
                 # visual indent is verified
                 if not indent[depth]:
                     indent[depth] = start[1]
             elif visual_indent in (text, str):
                 # ignore token lined up with matching one from a previous line
                 pass
-            elif indent[depth] and start[1] < indent[depth]:
-                # visual indent is broken
-                yield (start, "E128 continuation line "
-                       "under-indented for visual indent")
-            elif hang == 4 or (indent_next and rel_indent[row] == 8):
-                # hanging indent is verified
-                if close_bracket and not hang_closing:
-                    yield (start, "E123 closing bracket does not match "
-                           "indentation of opening bracket's line")
             else:
                 # indent is broken
                 if hang <= 0:

testsuite/E12.py

@@ -340,4 +340,17 @@ def qualify_by_address(self, cr, uid, ids, context=None,
 rv.update(d=('a' + 'b', 'c'),
           e=42, f=(42
                  + 42))
+#: E123
+if True:
+    def example_issue254():
+        return [node.copy(
+            (
+                replacement
+                # First, look at all the node's current children.
+                for child in node.children
+                # Replace them.
+                for replacement in replace(child)
+                ),
+            dict(name=token.undefined)
+        )]
 #:

testsuite/E12not.py

@@ -588,3 +588,32 @@ def f():
     1).zap(
     2)
 #
+if True:
+
+    def example_issue254():
+        return [node.copy(
+            (
+                replacement
+                # First, look at all the node's current children.
+                for child in node.children
+                # Replace them.
+                for replacement in replace(child)
+            ),
+            dict(name=token.undefined)
+        )]
+
+
+def valid_example():
+    return [node.copy(properties=dict(
+            (key, val if val is not None else token.undefined)
+            for key, val in node.items()
+            ))]
+
+
+def other_example():
+    return [node.copy(properties=dict(
+        (key, val if val is not None else token.undefined)
+        for key, val in node.items()
+    ))]
+
+#