Address Flake8's E127 with "if (" (#102)

> continuation line over-indented for visual indent

It appears that the "if (" is just the 4 columns wide that triggers the
extra indent with sw=4.

Might also change/adjust b:control_statement to ignore `if (`, but this
appears to be more flexible/correct.

Author: blueyed

indent/python.vim

@@ -225,10 +225,13 @@ function! s:indent_like_opening_paren(lnum)
     " indent further to distinguish the continuation line
     " from the next logical line.
     if text =~# b:control_statement && res == base + s:sw()
-        return base + s:sw() * 2
-    else
-        return res
+        " But only if not inside parens itself (Flake's E127).
+        let [paren_lnum, _] = s:find_opening_paren(paren_lnum)
+        if paren_lnum <= 0
+            return res + s:sw()
+        endif
     endif
+    return res
 endfunction
 
 " Match indent of first block of this type.

spec/indent/indent_spec.rb

@@ -251,6 +251,11 @@
       indent.should == 0
     end
 
+    it "handles nested expressions (Flake8's E127)" do
+      vim.feedkeys 'i[\<CR>x for x in foo\<CR>if (\<CR>'
+      indent.should == shiftwidth * 2
+    end
+
     it "still handles multiple parens correctly" do
       vim.feedkeys 'iif (111 and (222 and 333\<CR>'
       indent.should == 13