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

Joseph Pecoraro
Reported 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.
Attachments
[PATCH] Proposed Fix (7.44 KB, patch)
2017-04-09 00:39 PDT, Joseph Pecoraro 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Joseph Pecoraro
Comment 1 2017-04-09 00:39:10 PDT
Created attachment 306607 [details] [PATCH] Proposed Fix
WebKit Commit Bot
Comment 2 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>
WebKit Commit Bot
Comment 3 2017-04-09 08:27:13 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.