Ingo Karkat wrote:
> On 25-Feb-09 6:25, Tony Mechelynck wrote: > > Bug? Seems so to me. even two of them (but both related to the 'keymap' > > option). > > > > The one is in menu.vim (Last change: 2008 Aug 22): at lines 365 and 377 > > it uses ":set" rather than ":setlocal", which sets the keymap for _all_ > > future new buffers, not only the current one. > > > > ... > > Just by opening menu.vim (with folding of if..endif enabled, cp. > http://vim.wikia.com/wiki/Syntax_folding_of_Vim_scripts), I found another > bug: > There's a missing 'endif' between lines 140-141: > > 138 func! s:FnameEscape(fname) > 139 if exists('*fnameescape') > 140 return fnameescape(a:fname) > 141 return escape(a:fname, " \t\n*?[{`$\\%#'\"|!<") > 142 endfunc > > I'm not sure what the version support policy for runtime files is; in > case they now only need to support VIM 7.2, one could get rid of this > wrapper around fnameescape() completely?! We can get rid of the wrapper as soon as people who run an older Vim but do fetch the latest runtime files upgrade to a newer Vim. It's hard to say when that happens. And the workaround is easy, so there is no push to removing it. I'll fix it, of course. -- Don't drink and drive. You might hit a bump and spill your beer. /// 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 -~----------~----~----~----~------~----~------~--~---
