Bug 46882

Summary:

The case-insensitivity backreference checking isn't working with YARR Interpreter

Product:

WebKit

Reporter:

Peter Varga <pvarga>

Component:

JavaScriptCore

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

barraclough, commit-queue, msaboff, zherczeg

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
proposed patch	none

Description Peter Varga 2010-09-30 02:07:20 PDT

The Interpreter::tryConsumeBackReference() function doesn't support the ignorecase checking of characters. 

Eg.:
"Abab".match(/(ab)\1/i);
result: null
expected: Abab, Ab

Comment 1 Peter Varga 2010-09-30 02:14:23 PDT

Created attachment 69319 [details]
proposed patch

Comment 2 Gavin Barraclough 2010-09-30 04:32:12 PDT

Comment on attachment 69319 [details]
proposed patch

Looks great, thank you!

Comment 3 WebKit Commit Bot 2010-09-30 04:45:01 PDT

Comment on attachment 69319 [details]
proposed patch

Clearing flags on attachment: 69319

Committed r68771: <http://trac.webkit.org/changeset/68771>

Comment 4 WebKit Commit Bot 2010-09-30 04:45:07 PDT

All reviewed patches have been landed.  Closing bug.