Fix invalid range in SetexToAtx

preservim · Oct 25, 2022 · 67571a1 · 67571a1
1 parent b9c5c34
commit 67571a1
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 1 deletion.
diff --git a/ftplugin/markdown.vim b/ftplugin/markdown.vim
@@ -494,7 +494,9 @@ endfunction
 function! s:SetexToAtx(line1, line2)
  let l:originalNumLines = line('$')
  execute 'silent! ' . a:line1 . ',' . a:line2 . 'substitute/\v(.*\S.*)\n\=+$/# \1/'
- execute 'silent! ' . a:line1 . ',' . a:line2 . 'substitute/\v(.*\S.*)\n-+$/## \1/'
+
+ let l:changed = l:originalNumLines - line('$')
+ execute 'silent! ' . a:line1 . ',' . (a:line2 - l:changed) . 'substitute/\v(.*\S.*)\n-+$/## \1'
  return l:originalNumLines - line('$')
 endfunction
 

diff --git a/test/setextoatx.vader b/test/setextoatx.vader
@@ -20,3 +20,29 @@ Expect (convert setex-style headings to atx):
  ## b
 
  ## b
+
+Given markdown;
+a
+=
+
+b
+=
+
+c
+-
+
+d
+-
+
+Execute (SetexToAtx with range):
+ :1,8SetexToAtx
+
+Expect (only convert setex headings in original range):
+ # a
+
+ # b
+
+ ## c
+
+ d
+ -