Title: [273613] trunk/JSTests
Revision
273613
Author
msab...@apple.com
Date
2021-02-26 18:51:41 -0800 (Fri, 26 Feb 2021)

Log Message

Followup test fix to r273594
https://bugs.webkit.org/show_bug.cgi?id=222506

Reviewed by Yusuke Suzuki.

* stress/regexp-max-size.js:
(testTooBigBMPRegExp):
(testTooBigNonBMPRegExp):
(testAll):

Modified Paths

Diff

Modified: trunk/JSTests/ChangeLog (273612 => 273613)


--- trunk/JSTests/ChangeLog	2021-02-27 02:42:24 UTC (rev 273612)
+++ trunk/JSTests/ChangeLog	2021-02-27 02:51:41 UTC (rev 273613)
@@ -1,3 +1,15 @@
+2021-02-26  Michael Saboff  <msab...@apple.com>
+
+        Followup test fix to r273594
+        https://bugs.webkit.org/show_bug.cgi?id=222506
+
+        Reviewed by Yusuke Suzuki.
+
+        * stress/regexp-max-size.js:
+        (testTooBigBMPRegExp):
+        (testTooBigNonBMPRegExp):
+        (testAll):
+
 2021-02-26  Saam Barati  <sbar...@apple.com>
 
         Remove bad assertion of AI ArrayMode state in various "by val" opcodes

Modified: trunk/JSTests/stress/regexp-max-size.js (273612 => 273613)


--- trunk/JSTests/stress/regexp-max-size.js	2021-02-27 02:42:24 UTC (rev 273612)
+++ trunk/JSTests/stress/regexp-max-size.js	2021-02-27 02:51:41 UTC (rev 273613)
@@ -17,7 +17,7 @@
 }
 
 function testTooBigBMPRegExp() {
-    let patt = '\u{1234}{4294967294}\u{4567}';
+    let patt = '\u{1234}{4294967295}\u{4567}';
     const re = RegExp(patt, 'u');
     return "\u{1234}\u{1234}\u{4567}".match(re);
 }
@@ -29,13 +29,13 @@
 }
 
 function testTooBigNonBMPRegExp() {
-    let patt = '\u{10234}{2147483646}\u{10100}';
+    let patt = '\u{10234}{2147483647}\u{10100}';
     const re = RegExp(patt, 'u');
     return "\u{10234}\u{10234}\u{10100}".match(re);
 }
 
-let shouldCompile = [testMaxRegExp, testMaxRegExp, testMaxRegExp];
-let shouldntCompile = [testTooBigRegExp, testTooBigRegExp, testTooBigRegExp];
+let shouldCompile = [testMaxRegExp, testMaxBMPRegExp, testMaxNonBMPRegExp];
+let shouldntCompile = [testTooBigRegExp, testTooBigBMPRegExp, testTooBigNonBMPRegExp];
 
 function testAll()
 {
@@ -56,7 +56,7 @@
         }
 
         if (notSyntaxError)
-            throw "This RegExp: " + shouldntCompile + " should throw a Syntax Error when it is compiled";
+            throw "This RegExp: " + shouldntCompile[i] + " should throw a Syntax Error when it is compiled";
     }
 }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to