diff options
| author | karl williamson <public@khwilliamson.com> | 2008-11-05 11:42:16 -0700 |
|---|---|---|
| committer | Yves Orton <demerphq@gmail.com> | 2008-11-06 11:32:25 +0000 |
| commit | a4893424bce3ee5da2b2e8e1c256b806c74bfb50 (patch) | |
| tree | 51e1f7ee29386f7a58ac1d2b56270ff914210000 /regcomp.c | |
| parent | aff3edb5cf5195431c87166ebfe06c4721c5c783 (diff) | |
| download | perl-a4893424bce3ee5da2b2e8e1c256b806c74bfb50.tar.gz | |
Reolve perlbug #59328: In re's, \N{U+...} doesn't match for ... > 256
Subject: PATCH [perl #59328] In re's, \N{U+...} doesn't match for ... > 256
Message-ID: <49124B78.2000907@khwilliamson.com>
Date: Wed, 05 Nov 2008 18:42:16 -0700
p4raw-id: //depot/perl@34747
Diffstat (limited to 'regcomp.c')
| -rw-r--r-- | regcomp.c | 22 |
1 files changed, 16 insertions, 6 deletions
@@ -6617,20 +6617,30 @@ S_reg_namedseq(pTHX_ RExC_state_t *pRExC_state, UV *valuep) | PERL_SCAN_DISALLOW_PREFIX | (SIZE_ONLY ? PERL_SCAN_SILENT_ILLDIGIT : 0); UV cp; - char string; len = (STRLEN)(endbrace - name - 2); cp = grok_hex(name + 2, &len, &fl, NULL); if ( len != (STRLEN)(endbrace - name - 2) ) { cp = 0xFFFD; } - if (cp > 0xff) - RExC_utf8 = 1; if ( valuep ) { + if (cp > 0xff) RExC_utf8 = 1; *valuep = cp; return NULL; } - string = (char)cp; - sv_str= newSVpvn(&string, 1); + + /* Need to convert to utf8 if either: won't fit into a byte, or the re + * is going to be in utf8 and the representation changes under utf8. */ + if (cp > 0xff || (RExC_utf8 && ! UNI_IS_INVARIANT(cp))) { + U8 string[UTF8_MAXBYTES+1]; + U8 *tmps; + RExC_utf8 = 1; + tmps = uvuni_to_utf8(string, cp); + sv_str = newSVpvn_utf8((char*)string, tmps - string, TRUE); + } else { /* Otherwise, no need for utf8, can skip that step */ + char string; + string = (char)cp; + sv_str= newSVpvn(&string, 1); + } } else { /* fetch the charnames handler for this scope */ HV * const table = GvHV(PL_hintgv); @@ -6809,7 +6819,7 @@ S_reg_namedseq(pTHX_ RExC_state_t *pRExC_state, UV *valuep) Set_Node_Cur_Length(ret); /* MJD */ RExC_parse--; nextchar(pRExC_state); - } else { + } else { /* zero length */ ret = reg_node(pRExC_state,NOTHING); } if (!cached) { |