Use systemlist instead of r!

Doesn't use a temporary file, which avoids a potential issue with the `egrep` backend. Also doesn't show stderr to the console.
AndrewRadev · Jun 5, 2020 · 352b783 · 352b783
1 parent f7f76df
commit 352b783
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/autoload/writable_search.vim b/autoload/writable_search.vim
@@ -45,7 +45,6 @@ function! writable_search#Rerun(...)
 
   %delete _
   call b:command.Read()
-  0delete _
 
   call writable_search#Parse()
 endfunction
@@ -221,7 +220,6 @@ function! s:Grep(query)
 
   %delete _
   call b:command.Read()
-  0delete _
 endfunction
 
 function! s:LastSelectedText()

diff --git a/autoload/writable_search/command.vim b/autoload/writable_search/command.vim
@@ -20,7 +20,8 @@ function! writable_search#command#Read() dict
   let saved_shellredir = &shellredir
   let &shellredir = substitute(&shellredir, '\V2>&1', '', 'g')
 
-  exe 'r!'.full_command
+  let output = systemlist(full_command)
+  call append(0, output)
 
   let &shellredir = saved_shellredir
 endfunction