Merge pull request #78 from aarondill/master

Author: aarondill

autoload/suda.vim

@@ -3,15 +3,20 @@
 " {opts} should be *sudo-specific*, while {cmd} is passed to any suda#executable
 " Note: {cmd} can not have any sudo flags. Put these into {opts}, as '--' is passed before {cmd}
 " Similarly, {opts} should *not* contain '--'
+" Returns a string that is safe to pass to `system` on both vim and neovim
 function! s:get_command(opts, cmd)
   if g:suda#executable ==# 'sudo' 
-    return [g:suda#executable] + a:opts + ['--'] + a:cmd
+    let ret = [g:suda#executable] + a:opts + ['--'] + a:cmd
+  else
+    " TODO:
+    " Should we pass '--' before cmd when using a custom suda#executable?
+    " Should suda#executable be split? Should we allow suda#executable to be a list instead?
+    " This behavior is entirely undocumented
+    let ret =  [g:suda#executable] + a:cmd
   endif
-  " TODO:
-  " Should we pass '--' before cmd when using a custom suda#executable?
-  " Should suda#executable be split? Should we allow suda#executable to be a list instead?
-  " This behavior is entirely undocumented
-  return [g:suda#executable] + a:cmd
+
+  " TODO: Should we detect `has('neovim')` and return a list to avoid a shell?
+  return join(map(ret, { k, v -> shellescape(v) }), ' ')
 endfunction
 
 " {cmd} is a argv list for the process