Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 1fe6b1dc37f170c19e6483afc9be1e9ebca59fbf
      
https://github.com/WebKit/WebKit/commit/1fe6b1dc37f170c19e6483afc9be1e9ebca59fbf
  Author: Michael Saboff <msab...@apple.com>
  Date:   2023-03-16 (Thu, 16 Mar 2023)

  Changed paths:
    M JSTests/stress/regexp-vflag-property-of-strings.js
    M Source/JavaScriptCore/yarr/YarrParser.h

  Log Message:
  -----------
  [JSC] Add ClassSetCharacter syntax tests for RegExp V flag and fix issues 
found
https://bugs.webkit.org/show_bug.cgi?id=254007
rdar://106789711

Reviewed by Mark Lam.

Added ClassCharacterSet tests.

Factored out the syntax checking of class set character from parseClassSet() to
consumeAndCheckIfValidClassSetCharacter() and used it in its original location 
as well as
parseClassStringDisjunction().  Added an additional check for the reserved 
double punctuators.
Also added explicit handling of '-' in parseClassStringDisjunction() since it 
is an
additional syntax error, where it is needed for constructing ranges in 
parseClassSet().

* JSTests/stress/regexp-vflag-property-of-strings.js:
* Source/JavaScriptCore/yarr/YarrParser.h:
(JSC::Yarr::Parser::consumeAndCheckIfValidClassSetCharacter):
(JSC::Yarr::Parser::parseClassSet):
(JSC::Yarr::Parser::parseClassStringDisjunction):

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


_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to