Fix bug 486, Cyrillic character ranges in case independent regexps....

Fix bug 486, Cyrillic character ranges in case independent regexps.
http://code.google.com/p/v8/issues/detail?id=486

Committed: http://code.google.com/p/v8/source/detail?r=3236

[Christian Plesner Hansen, 2009-11-06 08:27:59]

Lgtm

http://codereview.chromium.org/361033/diff/1/2
File test/mjsunit/regress/regress-486.js (right):

http://codereview.chromium.org/361033/diff/1/2#newcode29
Line 29: var cyrillicMatch = /^[\u0430-\u044fa-z]+$/i.test(st);  // a-ja a-z.
Does this really cover all the new code?

[Erik Corry, 2009-11-06 10:24:26]

On 2009/11/06 08:27:59, Christian Plesner Hansen wrote:
> Lgtm
> 
> http://codereview.chromium.org/361033/diff/1/2
> File test/mjsunit/regress/regress-486.js (right):
> 
> http://codereview.chromium.org/361033/diff/1/2#newcode29
> Line 29: var cyrillicMatch = /^[\u0430-\u044fa-z]+$/i.test(st);  // a-ja a-z.
> Does this really cover all the new code?

Nope.  It didn't work either.

New version uploaded with new test.  There are a couple of obscure cases where
we differ from JSC and Tracemonkey.  Please verify that we are doing the right
thing there.

[Christian Plesner Hansen, 2009-11-06 10:36:09]

Lgtm

http://codereview.chromium.org/361033/diff/4001/4002
File test/mjsunit/regress/regress-486.js (right):

http://codereview.chromium.org/361033/diff/4001/4002#newcode29
Line 29: var cyrillicMatch = /^[\u0430-\u044fa-z]+$/i.test(st);  // a-ja a-z.
So... are you going to extend it?

[Erik Corry, 2009-11-06 11:15:18]

http://codereview.chromium.org/361033/diff/4001/4002
File test/mjsunit/regress/regress-486.js (right):

http://codereview.chromium.org/361033/diff/4001/4002#newcode29
Line 29: var cyrillicMatch = /^[\u0430-\u044fa-z]+$/i.test(st);  // a-ja a-z.
On 2009/11/06 10:36:09, Christian Plesner Hansen wrote:
> So... are you going to extend it?

No.  It's a regression test so it tests the issue as reported.  I added the
other test to get better coverage.

[Christian Plesner Hansen, 2009-11-06 11:21:30]

Ah, indeed you did.