search

keeleysam / tenfourfox

Automatically exported from code.google.com/p/tenfourfox
0 stars 0 forks source link

[PATCH] jstests failure: ecma_3/RegExp/perlstress-001.js #120

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
I built js/ from 10.4Fx 9 and ran 'make check jstests'. It revealed a few 
failures, including several; in ecma_3/RegExp/perlstress-001.js, which can be 
reduced to

'A]'.match(/a]/i) == null /* should be 'A]' */

. I get the same result running the test by hand in the Web Console in 10.4Fx 9.

It turns out to be a minor endian bug in YARR; here's a patch.

Original issue reported on code.google.com by magef...@gmail.com on 21 Dec 2011 at 3:33

Attachments:

GoogleCodeExporter commented 9 years ago 
Looks reasonable enough. We might grab M660747 at the same time, though it 
doesn't really affect us. Scheduling for 10 (whenever that might be -_- ).

Original comment by classi...@floodgap.com on 21 Dec 2011 at 3:41

GoogleCodeExporter commented 9 years ago 
I'm going to push this upstream to see if Mozilla wants it. I don't have a name 
to credit you under though.

Original comment by classi...@floodgap.com on 22 Dec 2011 at 4:06

GoogleCodeExporter commented 9 years ago 
Ah, I'm Ben Stuhl.

Original comment by magef...@gmail.com on 22 Dec 2011 at 5:00

GoogleCodeExporter commented 9 years ago 
Got it!

Original comment by classi...@floodgap.com on 22 Dec 2011 at 5:07

GoogleCodeExporter commented 9 years ago 
https://bugzilla.mozilla.org/show_bug.cgi?id=712990

I added your account to the contributors list, btw. Thanks for your help so far 
:)

Original comment by classi...@floodgap.com on 22 Dec 2011 at 5:26

GoogleCodeExporter commented 9 years ago 
Landed in 10

Original comment by classi...@floodgap.com on 3 Jan 2012 at 1:59

GoogleCodeExporter commented 9 years ago 
Closing as Verified

Original comment by classi...@floodgap.com on 9 Jan 2012 at 12:36