[webkit-changes] [WebKit/WebKit] cc3bdd: REGRESSION(257823 at main): 4X Test262-test are const...

Michael Saboff noreply at github.com
Tue Dec 20 09:21:52 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: cc3bdd6f9825b5eff922b0d912f5cdbdcdd54646
      https://github.com/WebKit/WebKit/commit/cc3bdd6f9825b5eff922b0d912f5cdbdcdd54646
  Author: Michael Saboff <msaboff at apple.com>
  Date:   2022-12-20 (Tue, 20 Dec 2022)

  Changed paths:
    M JSTests/stress/regexp-lookbehind.js
    M JSTests/test262/config.yaml
    M Source/JavaScriptCore/yarr/YarrErrorCode.cpp
    M Source/JavaScriptCore/yarr/YarrErrorCode.h
    M Source/JavaScriptCore/yarr/YarrParser.h

  Log Message:
  -----------
  REGRESSION(257823 at main): 4X Test262-test are constant failures
https://bugs.webkit.org/show_bug.cgi?id=249624
rdar://103537619

Reviewed by Yusuke Suzuki.

As specified, RegExp lookbehinds do not allow for quantifiers.
This change fixes the Yarr parser to disallow Quantifiers for Lookbehinds and adds a new error if one
is specified.  The error message has the same text as V8.

Added a new test case and re-enabled the corresponding Test262 tests.

* JSTests/stress/regexp-lookbehind.js:
(testRegExpSyntaxError):
* JSTests/test262/config.yaml:
* Source/JavaScriptCore/yarr/YarrErrorCode.cpp:
(JSC::Yarr::errorMessage):
(JSC::Yarr::errorToThrow):
* Source/JavaScriptCore/yarr/YarrErrorCode.h:
* Source/JavaScriptCore/yarr/YarrParser.h:
(JSC::Yarr::Parser::parseEscape):
(JSC::Yarr::Parser::parseAtomEscape):
(JSC::Yarr::Parser::parseParenthesesBegin):
(JSC::Yarr::Parser::parseParenthesesEnd):
(JSC::Yarr::Parser::parseQuantifier):
(JSC::Yarr::Parser::parseTokens):

Canonical link: https://commits.webkit.org/258139@main




More information about the webkit-changes mailing list