[debbugs-tracker] bug#17034: closed ([PATCH] grep: open CSET and transform into the upper case when MB_CUR_MAX == 1 in dfamust)

[GNU bug Tracking System]

Your message dated Fri, 11 Apr 2014 18:36:50 -0700
with message-id <address@hidden>
and subject line Re: bug#17034: [PATCH] grep: open CSET and transform into the 
upper case when MB_CUR_MAX == 1 in dfamust
has caused the debbugs.gnu.org bug report #17034,
regarding [PATCH] grep: open CSET and transform into the upper case when 
MB_CUR_MAX == 1 in dfamust
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
17034: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=17034
GNU Bug Tracking System
Contact address@hidden with problems

--- Begin Message --- Subject: [PATCH] grep: open CSET and transform into the upper case when MB_CUR_MAX == 1 in dfamust Date: Tue, 18 Mar 2014 22:38:54 +0900

Package: grep
Tags: patch

In MB_CUR_MAX == 1 and case-insensitive matching, KWSet doesn't so
helpful, because through parse alphabets of single-byte are changed into
CSET which isn't extracted by dfamust.

This patch opens CSET in dfamust, and makes it possible to take out
a longer character fixed string from tokens.

Norihiro

patch.txt
Description: Text document

--- End Message ---

--- Begin Message --- Subject: Re: bug#17034: [PATCH] grep: open CSET and transform into the upper case when MB_CUR_MAX == 1 in dfamust Date: Fri, 11 Apr 2014 18:36:50 -0700 User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0

Norihiro Tanaka wrote:

I fixed the bug in the patch.  Added call of resetmust().

Thanks, I installed that, along with the attached fixeup patch, and am marking this as done. The most interesting part of the fixup patch is its replacement of 'for (; j < NOTCHAR; j++)' with 'while (++j < NOTCHAR)', which should work a bit better when analyzing patterns like '[aaa]'.

0001-grep-improvements-for-the-open-CSET-patch.patch
Description: Text document

--- End Message ---