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 Apr 23 11:48:12 2005  
 
Category: NonePriority: 5 - Normal
Status: DonePrivacy: Public
Assigned to: NoneOpen/Closed: Closed

Wed Oct 3 09:23:23 2012, comment #4:

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

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

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

Jaroslav Škarvada <yarda>
Mon Mar 8 17:16:24 2010, comment #2:

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

Paolo Bonzini <bonzini>
Tue Apr 26 14:25:25 2005, 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 Apr 23 11:48:12 2005, 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 Oct 3 09:23:23 2012bonziniStatusReady For Test=>Done
      Open/ClosedOpen=>Closed
    Mon Mar 8 17:16:24 2010bonziniStatusNone=>Ready For Test
    Tue Apr 26 14:27:33 2005sick_soulAttached File-=>Added grep-P-utf8.Naurp.p1.patch, #4523
    Tue Apr 26 14:25:25 2005sick_soulAttached File-=>Added grep-P-utf8-bug.sh, #4522
    Sat Apr 23 12:03:49 2005sick_soulAttached File-=>Added grep-P-utf8.Naurp.p1.patch, #4493
    Sat Apr 23 11:48:13 2005sick_soulAttached File-=>Added grep-P-utf8.Naurp.p1.patch, #4492
      Carbon-Copy-=>Added -unavailable-

    Back to the top


    Powered by Savane 3.1-cleanup