Ben Fritz wrote:
[...] > > Now the question is what will break if we change the default for > > 'shellxquote'. I'm not sure about that. > > I doubt that changing the default would break anything built into Vim, > but I suppose that remains to be seen. Changing the default would > certainly add a whole class of use cases that work "out of the box". > Any corner cases that for some reason do not work properly could > easily be worked around as done in the plugin mentioned, by setting > and restoring shellxquote (and possibly shellcmdflag). I haven't found > a way to break system() with shellxquote=\", but there is certainly a > way to break it with the current default. I have admittedly not done > much with shellxquote for uses other than system(). I see potential > for a problem here, especially when using redirection, but I _think_ > it will work. > > Plugins are another question though. For testing, I placed my proposed > values of shellxquote and shellcmdflag in my vimrc, and immediately > broke the TagList plugin. > > This seems to be caused by a bug in TagList, actually. I don't think > the following code is needed, and it is the root of the bug. But, I'm > not actually sure what the intent of it is: > > if &shellxquote == '"' > " Double-quotes within double-quotes will not work in the > " command-line.If the 'shellxquote' option is set to double- > quotes, > " then escape the double-quotes in the ctags command-line. > let ctags_cmd = escape(ctags_cmd, '"') > endif > > Later (in Windows), Taglist writes ctags_cmd to a temporary batch file > and executes the batch file with system(). > > I wonder how many other plugins do something similar. The problem here appears to be that the plugin assumes a sh like 'shell'. Escaping double quotes with a backslash doesn't work for cmd.exe, it sees backslashes as path separators.. -- Some of the well know MS-Windows errors: ESLEEP Operator fell asleep ENOERR No error yet EDOLLAR OS too expensive EWINDOWS MS-Windows loaded, system in danger /// Bram Moolenaar -- [EMAIL PROTECTED] -- http://www.Moolenaar.net \\\ /// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\ \\\ download, build and distribute -- http://www.A-A-P.org /// \\\ help me help AIDS victims -- http://ICCF-Holland.org /// --~--~---------~--~----~------------~-------~--~----~ You received this message from the "vim_dev" maillist. For more information, visit http://www.vim.org/maillist.php -~----------~----~----~----~------~----~------~--~---