]> granicus.if.org Git - vim/commitdiff
patch 8.0.0851: 'smartindent' is used even when 'indentexpr' is set v8.0.0851
authorBram Moolenaar <Bram@vim.org>
Thu, 3 Aug 2017 15:54:03 +0000 (17:54 +0200)
committerBram Moolenaar <Bram@vim.org>
Thu, 3 Aug 2017 15:54:03 +0000 (17:54 +0200)
Problem:    'smartindent' is used even when 'indentexpr' is set.
Solution:   Ignore 'smartindent' when 'indentexpr' is set. (Hirohito Higashi)

src/misc1.c
src/testdir/test_smartindent.vim
src/version.c

index 4f33015a794a96348954d8c023c6a67dfb5c42b3..f19c2dc8111639a7d68a962088880ab4a2759c8c 100644 (file)
@@ -626,6 +626,9 @@ open_line(
     int                do_si = (!p_paste && curbuf->b_p_si
 # ifdef FEAT_CINDENT
                                        && !curbuf->b_p_cin
+# endif
+# ifdef FEAT_EVAL
+                                       && *curbuf->b_p_inde == NUL
 # endif
                        );
     int                no_si = FALSE;          /* reset did_si afterwards */
index d00eac97981f22940a501cc670882d815f7dcfee..9e93a55eb0fb20c901f3e35d686ffa597b09f43b 100644 (file)
@@ -1,3 +1,4 @@
+" Tests for smartindent
 
 " Tests for not doing smart indenting when it isn't set.
 function! Test_nosmartindent()
@@ -12,3 +13,29 @@ function! Test_nosmartindent()
   call assert_equal("          #test", getline(1))
   enew! | close
 endfunction
+
+function MyIndent()
+endfunction
+
+" When 'indentexpr' is set, setting 'si' has no effect.
+function Test_smartindent_has_no_effect()
+  new
+  exe "normal! i\<Tab>one\<Esc>"
+  set noautoindent
+  set smartindent
+  set indentexpr=
+  exe "normal! Gotwo\<Esc>"
+  call assert_equal("\ttwo", getline("$"))
+
+  set indentexpr=MyIndent
+  exe "normal! Gothree\<Esc>"
+  call assert_equal("three", getline("$"))
+
+  delfunction! MyIndent
+  set autoindent&
+  set smartindent&
+  set indentexpr&
+  bwipe!
+endfunction
+
+" vim: shiftwidth=2 sts=2 expandtab
index c5c39fea2f65bab215ba53607bf23e61598f7ba6..a5a3d68a193492ebeffdaeeebb843c1013468670 100644 (file)
@@ -769,6 +769,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    851,
 /**/
     850,
 /**/