Bug 170651 - test262: test262/test/annexB/language/literals/regexp/identity-escape.js
Summary: test262: test262/test/annexB/language/literals/regexp/identity-escape.js
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Joseph Pecoraro
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-04-09 00:25 PDT by Joseph Pecoraro
Modified: 2017-04-09 08:27 PDT (History)
7 users (show)

See Also:


Attachments
[PATCH] Proposed Fix (7.44 KB, patch)
2017-04-09 00:39 PDT, Joseph Pecoraro
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.