GNU bug report logs - #17034
[PATCH] grep: open CSET and transform into the upper case when MB_CUR_MAX == 1 in dfamust

Previous Next

Package: grep;

Reported by: Norihiro Tanaka <noritnk <at> kcn.ne.jp>

Date: Tue, 18 Mar 2014 13:40:02 UTC

Severity: normal

Tags: patch

Done: Paul Eggert <eggert <at> cs.ucla.edu>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 17034 in the body.
You can then email your comments to 17034 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox

Report forwarded to bug-grep <at> gnu.org:
bug#17034; Package grep. (Tue, 18 Mar 2014 13:40:03 GMT) Full text and rfc822 format available.
Acknowledgement sent to Norihiro Tanaka <noritnk <at> kcn.ne.jp>:
New bug report received and forwarded. Copy sent to bug-grep <at> gnu.org. (Tue, 18 Mar 2014 13:40:04 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Norihiro Tanaka <noritnk <at> kcn.ne.jp>
To: submit <at> debbugs.gnu.org
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

[Message part 1 (text/plain, inline)]
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 (text/plain, attachment)]
Information forwarded to bug-grep <at> gnu.org:
bug#17034; Package grep. (Wed, 02 Apr 2014 23:51:01 GMT) Full text and rfc822 format available.

Message #8 received at 17034 <at> debbugs.gnu.org (full text, mbox):

From: Norihiro Tanaka <noritnk <at> kcn.ne.jp>
To: 17034 <at> debbugs.gnu.org
Subject: bug#17034: [PATCH] grep: open CSET and transform into the upper case
 when MB_CUR_MAX == 1 in dfamust
Date: Thu, 03 Apr 2014 08:50:37 +0900

[Message part 1 (text/plain, inline)]
I fixed the bug in the patch.  Added call of resetmust().

[patch.txt (text/plain, attachment)]
Reply sent to Paul Eggert <eggert <at> cs.ucla.edu>:
You have taken responsibility. (Sat, 12 Apr 2014 01:37:02 GMT) Full text and rfc822 format available.
Notification sent to Norihiro Tanaka <noritnk <at> kcn.ne.jp>:
bug acknowledged by developer. (Sat, 12 Apr 2014 01:37:03 GMT) Full text and rfc822 format available.

Message #13 received at 17034-done <at> debbugs.gnu.org (full text, mbox):

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: Norihiro Tanaka <noritnk <at> kcn.ne.jp>, 17034-done <at> debbugs.gnu.org
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

[Message part 1 (text/plain, inline)]
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 (text/plain, attachment)]
bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Sat, 10 May 2014 11:24:03 GMT) Full text and rfc822 format available.
This bug report was last modified 11 years and 47 days ago.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.