Bug 7253 – Inline regex produces wrong results vs. creating new RegExp object

• Status: RESOLVED
• Resolution: DUPLICATE
• Component: JavaScriptCore

[Parag Shah]

Basically creating a regex like this:

var regex = /^([^#<\u2264]+)([#<\u2264])(.*)$/;

does not give any results vs. creating a regex like this:

var regex = new RegExp("^([^#<\u2264]+)([#<\u2264])(.*)$");


This probably isnt a bug in most cases but see attachment for narrowed down testcase where this is happening - comment out line 10 and uncomment line 9 (and vice versa) to see the bug. 

FF/IE work fine in both cases. Tested on Safari v2.0.3 (417.8) on OS X v10.4.4

[Parag Shah]

Created attachment 6484 [details]
Narrowed down testcase

[Alexey Proskuryakov]

Confirmed with ToT

[Alexey Proskuryakov]

Console log says:
KJS: pcre_compile() failed with 'PCRE does not support \L, \l, \N, \U, or \u'

[Geoffrey Garen]

Maks posted a patch about this. Can't seem to find it right now.

[Alexey Proskuryakov]

Probably bug 6257 - not sure if this should be marked as a duplicate.

[Alexey Proskuryakov]

*** Bug 10611 has been marked as a duplicate of this bug. ***

[David Kilzer (:ddkilzer)]

Per Bug 10611 Comment #2, this bug is in Radar as:

rdar://4694011

Bug 10611 also has its own reduced test case.

[W. Andy Carrel]

The patch landed for bug 7445 (r17354) makes the reduction for this bug function properly. This should be marked as a duplicate.

[David Kilzer (:ddkilzer)]

(In reply to comment #8)
> The patch landed for bug 7445 (r17354) makes the reduction for this bug
> function properly. This should be marked as a duplicate.

It would probably be best to land the reduction (Attachment 6484 [details]) as a test before closing this bug.

[David Kilzer (:ddkilzer)]

(In reply to comment #9)
> It would probably be best to land the reduction (Attachment 6484 [details]) as a test
> before closing this bug.

Especially since the fix for Bug 7445 didn't include a layout test.

[David Kilzer (:ddkilzer)]

(In reply to comment #10)
> (In reply to comment #9)
> > It would probably be best to land the reduction (Attachment 6484 [details] [edit]) as a test
> > before closing this bug.
> 
> Especially since the fix for Bug 7445 didn't include a layout test.

Test cases will be forthcoming for Bug 7445 per Bug 7445 Comment #18, so closing this as a duplicate per Comment #8.


*** This bug has been marked as a duplicate of 7445 ***