Yegappan Lakshmanan wrote: > Hi all, > > >>> > > > > >>> > > > These are my initial observations. I'll keep on using my build with > >>> > > > the cdo.diff patch and report back with any further findings. > >>> > > > > >>> > > > Thanks, > >>> > > > Drew > >>> > > > > >>> > > > > >>> > > > On Monday, April 6, 2015 at 6:41:10 PM UTC+1, yega...@gmail.com > >>> > > > wrote: > >>> > > > > Hi all, > >>> > > > > > >>> > > > > On Mon, Apr 6, 2015 at 6:58 AM, Drew Neil <andr...@gmail.com> > >>> > > > > wrote: > >>> > > > > > On Sunday, March 22, 2015 at 2:43:22 AM UTC, yega...@gmail.com > >>> > > > > > wrote: > >>> > > > > >> Hi all, > >>> > > > > >> > >>> > > > > >> On Sun, Jun 2, 2013 at 2:09 PM, Bram Moolenaar wrote: > >>> > > > > >> > > >>> > > > > >> > Yegappan Lakshmanan wrote: > >>> > > > > >> > > >>> > > > > >> >> I am attaching a patch to add the :cdo and :ldo commands. > >>> > > > > >> >> The :cdo command > >>> > > > > >> >> can be used to execute commands over all the buffers in the > >>> > > > > >> >> quickfix list. > >>> > > > > >> >> The :ldo command can be used to execute commands over all > >>> > > > > >> >> the buffers in > >>> > > > > >> >> the location list. These are similar to the existing > >>> > > > > >> >> :bufdo, :windo > >>> > > > > >> >> and :tabdo commands. > >>> > > > > >> > > >>> > > > > >> > Thanks. I'll add it in the todo list. > >>> > > > > >> > > >>> > > > > >> > >>> > > > > >> I am attaching an updated patch with a lot of test cases > >>> > > > > >> against Vim 7.4.671. > >>> > > > > >> This patch introduces the ":cdo", ":ldo", ":cfdo" and > >>> > > > > >> ":lfdo"commands. > >>> > > > > >> The patch to the documentation is also included. > >>> > > > > > > >>> > > > > > Nice work Yegappan. I would love to see these features included > >>> > > > > > in Vim. > >>> > > > > > > >>> > > > > > >>> > > > > It looks like a lot of people are interested in this feature. It > >>> > > > > will > >>> > > > > help if people > >>> > > > > can apply this patch and try the new commands and give some > >>> > > > > feedback. > >>> > > > >>> > > I ported the patch to Neovim [1], where CI discovered that it causes > >>> > > test10 to fail. I tested against Vim 7.4.764, and test10 fails there > >>> > > as well (see [2]). > >>> > > >>> > I think this happens, because with this patch :cf does resolve to :cfdo > >>> > instead of :cfile. Swapping the order in ex_cmds.h should fix this. > >>> > > >>> > Best, > >>> > Christian > >>> > -- > >>> > Glück ist Scharfsinn für Gelegenheiten und die Fähigkeit sie zu > >>> > nutzen. > >>> > -- Samuel Goldwyn (eigentl. Goldfish) > >>> > >>> > >>> I can confirm that swapping the commands fixes the test error [1]. > >>> > >>> Cheers, > >>> Florian > >>> > >>> [1] https://travis-ci.org/fwalch/vim-ci/builds/69122901 > >> > >> I built Vim with this patch a week ago. I've been using the patched build > >> every day for a week and have not yet encountered any issues. > >> > > > > Thanks for testing the patch. I will send out an updated patch in a few > > days. > > Hopefully this time it will get included. This has been outstanding for more > > than two years. > > > > The updated patch (against vim 7.4.796) is attached.
Thanks. So now it's ready to include, right? -- XML is a nice language for computers. Not for humans. /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net \\\ /// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\ \\\ an exciting new programming language -- http://www.Zimbu.org /// \\\ help me help AIDS victims -- http://ICCF-Holland.org /// -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying to. For more information, visit http://www.vim.org/maillist.php --- You received this message because you are subscribed to the Google Groups "vim_dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to vim_dev+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
