Bug 170651

Summary: test262: test262/test/annexB/language/literals/regexp/identity-escape.js
Product: WebKit Reporter: Joseph Pecoraro <joepeck>
Component: JavaScriptCoreAssignee: Joseph Pecoraro <joepeck>
Status: RESOLVED FIXED    
Severity: Normal CC: buildbot, commit-queue, joepeck, keith_miller, mark.lam, msaboff, saam
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
[PATCH] Proposed Fix none

Description Joseph Pecoraro 2017-04-09 00:25:38 PDT 
Summary:
DecimalEscape of /\8/ and /\9/ should just be the number, so /8/ or /9/ but it currently matches "\\8" or "\\9".

Test:
test262/test/annexB/language/literals/regexp/identity-escape.js

Spec:
https://tc39.github.io/ecma262/#sec-decimalescape

Where "MV" is just the mathematical value:

> The production DecimalEscape::NonZeroDigit evaluates as follows:
> 
>     Return the MV of NonZeroDigit.
>
> The production DecimalEscape::NonZeroDigitDecimalDigits evaluates as follows:
> 
>     Let n be the number of code points in DecimalDigits.
>     Return (the MV of NonZeroDigit × 10n) plus the MV of DecimalDigits.


Notes:
Both Chrome and Firefox match just the number.
Comment 1 Joseph Pecoraro 2017-04-09 00:39:10 PDT 
Created attachment 306607 [details]
[PATCH] Proposed Fix
Comment 2 WebKit Commit Bot 2017-04-09 08:27:11 PDT 
Comment on attachment 306607 [details]
[PATCH] Proposed Fix

Clearing flags on attachment: 306607

Committed r215161: <http://trac.webkit.org/changeset/215161>
Comment 3 WebKit Commit Bot 2017-04-09 08:27:13 PDT 
All reviewed patches have been landed.  Closing bug.