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

 
 

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

patch #3934: grep -P : fix for UTF8

Submitter:  Claudio Fontana <sick_soul>
Submitted:  Sat 23 Apr 2005 11:48:12 AM UTC
   
 
Category:  None Priority:  5 - Normal
Status:  Done Privacy:  Public
Assigned to:  None Open/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: 
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>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #8736:  grep-P-utf8.Naurp.p1.patch added by sick_soul (4KiB - 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 (3KiB - application/octet-stream - fix: do not check validity of utf8 string)
file #8711:  grep-P-utf8.Naurp.p1.patch added by sick_soul (3KiB - 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
  • -email is unavailable- added by yarda (Posted a comment)
  • -email is unavailable- added by bonzini (Posted a comment)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 8 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2012-10-03 bonzini StatusReady For Test Done
        Open/ClosedOpen Closed
    2010-03-08 bonzini StatusNone Ready For Test
    2005-04-26 sick_soul Attached File- Added grep-P-utf8.Naurp.p1.patch, #4523
    2005-04-26 sick_soul Attached File- Added grep-P-utf8-bug.sh, #4522
    2005-04-23 sick_soul Attached File- Added grep-P-utf8.Naurp.p1.patch, #4493
    2005-04-23 sick_soul Attached File- Added grep-P-utf8.Naurp.p1.patch, #4492
        Carbon-Copy- Added -email is unavailable-

    Back to the top

    Powered by Savane 3.14-50d9.
    Corresponding source code