On Wednesday, July 1, 2015 at 12:54:45 PM UTC+1, Florian Walch wrote: > On Wednesday, July 1, 2015 at 12:48:50 AM UTC+3, Christian Brabandt wrote: > > On Mo, 29 Jun 2015, Florian Walch wrote: > > > > > On Monday, June 29, 2015 at 12:40:28 PM UTC+3, Drew Neil wrote: > > > > I finally got round to building Vim with the cdo.diff patch and I'm > > > > currently test-driving it. > > > > > > > > First thing I tried was populating the quickfix list using :Ggrep (a > > > > wrapper for git-grep supplied by the fugitive.vim plugin). Then I made > > > > a change to every line in the quickfix list: > > > > > > > > :cdo normal A -- it works! > > > > > > > > Then I updated each of the files listed in the quickfix list: > > > > > > > > :cfdo update > > > > > > > > Both commands worked as expected. > > > > > > > > I did notice one oddity: after running :cdo, the syntax highlighting is > > > > disabled for each file listed in the quickfix list with the exception > > > > of the last file, which has syntax highlighting enabled. The > > > > documentation says that disabling syntax highlighting gives a > > > > performance boost, which is good to know. My guess is that syntax > > > > highlighting is supposed to be re-enabled after running :cdo, but > > > > there's some bug that causes it to be re-enabled only for the last file > > > > in the list. I have observed similar behaviour when using the :argdo > > > > command, so perhaps this bug (if it is a bug) is out of scope for this > > > > particular patch. > > > > > > > > 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, [email protected] wrote: > > > > > Hi all, > > > > > > > > > > On Mon, Apr 6, 2015 at 6:58 AM, Drew Neil <[email protected]> wrote: > > > > > > On Sunday, March 22, 2015 at 2:43:22 AM UTC, [email protected] > > > > > > 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. -- -- 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 [email protected]. For more options, visit https://groups.google.com/d/optout.
