From: Daniel Hahler Date: Sat, 23 May 2015 22:24:22 +0000 (+0200) Subject: Fix regression with colon-at-eol detection (#44) X-Git-Url: https://git.madduck.net/etc/vim.git/commitdiff_plain/f4f95ee041370e3e79bb72cb9b360a4ec5707785?hp=--cc Fix regression with colon-at-eol detection (#44) Instead of using `search` it now goes back and breaks if a char is found that should not get skipped. Fixes https://github.com/hynek/vim-python-pep8-indent/issues/44 Closes https://github.com/hynek/vim-python-pep8-indent/issues/45 --- f4f95ee041370e3e79bb72cb9b360a4ec5707785 diff --git a/indent/python.vim b/indent/python.vim index a1dd43f..c3c6c03 100644 --- a/indent/python.vim +++ b/indent/python.vim @@ -243,17 +243,16 @@ function! s:indent_like_previous_line(lnum) let ignore_last_char = eval(s:skip_special_chars) " Search for final colon that is not inside something to be ignored. - while search(':', 'bcW', lnum) + while 1 let curpos = getpos(".")[2] if curpos == 1 | break | endif - if eval(s:skip_special_chars) + if eval(s:skip_special_chars) || text[curpos-1] =~ '\s' normal! h continue - endif - if !s:match_expr_on_line(s:skip_special_chars, lnum, curpos) + elseif text[curpos-1] == ':' return base + s:sw() endif - normal! h + break endwhile if text =~ '\\$' && !ignore_last_char diff --git a/spec/indent/indent_spec.rb b/spec/indent/indent_spec.rb index 1999d27..196e119 100644 --- a/spec/indent/indent_spec.rb +++ b/spec/indent/indent_spec.rb @@ -153,6 +153,14 @@ shared_examples_for "vim" do end end + describe "when the previous line has a list slice" do + it "does not indent" do + vim.feedkeys 'ib = a[2:]\' + indent.should == 0 + proposed_indent.should == 0 + end + end + describe "when after an '(' that is followed by an unfinished string" do before { vim.feedkeys 'itest("""' }