Christian Brabandt wrote:

> when working with csv-files, I have often wished for more than 10 
> capturing groups. So here is a patch, that allows 99 capturing groups in 
> the replacement part. This uses the perl-like syntax ${1} until ${99} 
> for the capturing groups in the replacement part. If there does not 
> exist a capturing group, this will resolve to the empty string.
> 
> I have tested it locally and it works for me[1]. This obviously needs to 
> be very well tested, before included. 
> 
> [1] running make test as well as trying several different :s commands.

Besides what I said about the string to be used:
- You don't need four bytes to store the group nr, use one byte.
- We need proper tests.  I'm not accepting any changes to the regexp
  changes without tests.  The code is too complicated for just a few
  manual checks.

-- 
hundred-and-one symptoms of being an internet addict:
35. Your husband tells you he's had the beard for 2 months.

 /// Bram Moolenaar -- [email protected] -- 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

Raspunde prin e-mail lui