On 2013-06-04 17:48, Bram Moolenaar wrote:
> Patch 7.3.1111
> Problem:    nfa_recognize_char_class() implementation is inefficient.
> Solution:   Use bits in an int instead of chars in a string. (Dominique Pelle)
> Files:            src/regexp_nfa.c, src/testdir/test36.in, 
> src/testdir/test36.ok

This patch fails to apply for me on Mac OS X:

$ patch --dry-run -p0 < /tmp/vim-patches/7.3.1111
patching file src/regexp_nfa.c
patching file src/testdir/test36.in
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file src/testdir/test36.in.rej
patching file src/testdir/test36.ok
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file src/testdir/test36.ok.rej
patching file src/version.

I tried both /usr/bin/patch which is 2.5.8 and a gpatch from MacPorts,
which is GNU patch 2.7.1.

I was unable to spot the exact problem with this patch. At first I
thought about locale issues due to the characters used in the test case,
but LC_ALL=C or anything similar did not make a difference.

Then I got the equivalent of this patch from mercurial with the command
'hg diff -cv7-3-1111' and the patch obtained from this applies just
fine. So I doubt it's a locale issue, it must be something else.

Did anybody else have a problem with this patch?

Rainer

-- 
-- 
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/groups/opt_out.


Raspunde prin e-mail lui