Fix false positive E121/E126 with multi-line string; issue #265

Author: florentx

CHANGES.txt

@@ -5,7 +5,7 @@ Changelog
 1.5.x (unreleased)
 ------------------
 
-
+* Fix false positive E121/E126 with multi-line strings. (Issue #265)
 
 
 1.5.1 (2014-03-27)

pep8.py

@@ -556,6 +556,8 @@ def continued_indentation(logical_line, tokens, indent_level, hang_closing,
                 indent_chances[start[1]] = text
 
         last_token_multiline = (start[0] != end[0])
+        if last_token_multiline:
+            rel_indent[end[0] - first_row] = rel_indent[row]
 
     if indent_next and expand_indent(line) == indent_level + 4:
         if visual_indent:

testsuite/E12not.py

@@ -632,3 +632,15 @@ def other_example():
         else 0,
 }
 #
+from textwrap import dedent
+
+
+print dedent(
+    '''
+        mkdir -p ./{build}/
+        mv ./build/ ./{build}/%(revision)s/
+    '''.format(
+        build='build',
+        # more stuff
+    )
+)

testsuite/test_shell.py

@@ -82,9 +82,11 @@ def test_check_simple(self):
             self.assertEqual(x, str(num))
             self.assertEqual(y, str(col))
             self.assertTrue(msg.startswith(' E11'))
-        # Config file read from the pep8's tox.ini
-        (config_filename,) = self._config_filenames
-        self.assertTrue(config_filename.endswith('tox.ini'))
+        # Config file read from the pep8's setup.cfg or tox.ini
+        config_filenames = self._config_filenames
+        self.assertEqual(len(config_filenames), 2)
+        self.assertTrue(config_filenames[0].endswith('setup.cfg'))
+        self.assertTrue(config_filenames[1].endswith('tox.ini'))
 
     def test_check_stdin(self):
         pep8.PROJECT_CONFIG = ()