patchgrep - Patches: patch #3934, grep -P : fix for UTF8

 
 

You are not allowed to post comments on this tracker with your current authentification level.

patch #3934: grep -P : fix for UTF8

Submitted by:  Claudio Fontana <sick_soul>
Submitted on:  Sat 23 Apr 2005 11:48:12 AM UTC  
 
Category: NonePriority: 5 - Normal
Status: DonePrivacy: Public
Assigned to: NoneOpen/Closed: Closed

Wed 03 Oct 2012 09:23:23 AM UTC, comment #4:

Similar patch applied (with testcases) as commit 003797b2d498fd16f67790a3c1129df9d0eb4722.

Paolo Bonzini <bonzini>
Wed 03 Oct 2012 07:52:35 AM UTC, comment #3:

There is also: http://bugzilla.redhat.com/show_bug.cgi?id=683753 including fix.

Jaroslav Škarvada <yarda>
Mon 08 Mar 2010 05:16:24 PM UTC, comment #2:

Patch needs to be brought up to date, but it looks fine.

Paolo Bonzini <bonzini>
Tue 26 Apr 2005 02:25:25 PM UTC, comment #1:

For a script reproducing the problem, download attached grep-P-utf8-bug.sh.

If you have arabic fonts, you can see how the UTF8 string gets broken very clearly, however it is not needed. Thanks to Charles for pointing out the script(1) tool.

For the discussion leading to this patch see this thread:
http://lists.gnu.org/archive/html/bug-grep/2005-04/msg00049.html

Note that the original submission is for some reason not in the archive. Also note that although the user reported it as a --color error, the problem is general and has nothing in particular to do with --color.

Claudio

Claudio Fontana <sick_soul>
Sat 23 Apr 2005 11:48:12 AM UTC, original submission:

This patch addresses problems in grep -P under utf8.

Claudio Fontana <sick_soul>

 

Attached Files
file #8736:  grep-P-utf8.Naurp.p1.patch added by sick_soul (4kB - application/octet-stream - The latest version of the grep-P-utf8 patch (2005-04-26))
file #8735:  grep-P-utf8-bug.sh added by sick_soul (554B - application/octet-stream - Use this script to verify the bug)
file #8712:  grep-P-utf8.Naurp.p1.patch added by sick_soul (3kB - application/octet-stream - fix: do not check validity of utf8 string)
file #8711:  grep-P-utf8.Naurp.p1.patch added by sick_soul (3kB - application/octet-stream - fix for grep -P under utf8. Apply against latest cvs with patch -p1 from source directory.)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by yarda (Posted a comment)
  • -unavailable- added by bonzini (Posted a comment)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 8 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 03 Oct 2012 09:23:23 AM UTCbonziniStatusReady For Test=>Done
      Open/ClosedOpen=>Closed
    Mon 08 Mar 2010 05:16:24 PM UTCbonziniStatusNone=>Ready For Test
    Tue 26 Apr 2005 02:27:33 PM UTCsick_soulAttached File-=>Added grep-P-utf8.Naurp.p1.patch, #4523
    Tue 26 Apr 2005 02:25:25 PM UTCsick_soulAttached File-=>Added grep-P-utf8-bug.sh, #4522
    Sat 23 Apr 2005 12:03:49 PM UTCsick_soulAttached File-=>Added grep-P-utf8.Naurp.p1.patch, #4493
    Sat 23 Apr 2005 11:48:13 AM UTCsick_soulAttached File-=>Added grep-P-utf8.Naurp.p1.patch, #4492
      Carbon-Copy-=>Added -unavailable-

    Back to the top


    Powered by Savane 3.1-cleanup