I've pushed those three commits, with a small change to the second one
(removing the leading ^ and trailing '\$' in a regexp) to make that
test succeed also with -F.
Sorry, you are right. the declaration of kwset_exact_matches shouldn't
be removed.
On Sat, Jan 25, 2014 at 8:45 PM, Norihiro Tanaka wrote:
> Hi Jim,
>
> I thank you for your review for the patch.
>
> I have any requests of any changes for the modified comments and commit
> log.
>
> However, can you merge an additional patch, which is attached on this
> mail, into the commit? No
Hi Jim,
I thank you for your review for the patch.
I have any requests of any changes for the modified comments and commit
log.
However, can you merge an additional patch, which is attached on this
mail, into the commit? No longer `kwsincr_case' is called with
case-insensitive matching in a mul
On Mon, Jan 13, 2014 at 8:43 AM, Jim Meyering wrote:
> Thank you for the patch. I will take a look in the next day or two.
Sorry about the delay.
I have divided your patch into two separate commits: one modifies
dfa.c and the other modifies dfasearch.c. I've included 5 commits
below. The first
Thank you for the patch. I will take a look in the next day or two.
I'm sorry the content of the attachment is incorrect.
I send the correct file.
grep-ignore-icase.txt
Description: Binary data
Package: grep
Tags: patch
Case-insensitive matching is expensive in multi-byte locales because of
conversion of targeted text to lower case.
However, I seem that awk which uses dfa.c as well as grep don't covert
target text to lower case. I seem that if grep don't use kwset, it
doesn't also have
