Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(137)

Issue 7867036: d8 external array c'tors: allow parameters that can be converted to numbers (Closed)

Created:
9 years, 3 months ago by Jakob Kummerow
Modified:
9 years, 3 months ago
Reviewers:
Yang
CC:
v8-dev
Visibility:
Public.

Description

d8 external array c'tors: allow parameters that can be converted to numbers BUG=v8:1681 TEST=d8 accepts: var a = new Int32Array("2"); Committed: http://code.google.com/p/v8/source/detail?r=9243

Patch Set 1 #

Patch Set 2 : add test #

Unified diffs Side-by-side diffs Delta from patch set Stats (+19 lines, -4 lines) Patch
M src/d8.cc View 2 chunks +6 lines, -4 lines 0 comments Download
M test/mjsunit/external-array.js View 1 1 chunk +13 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Jakob Kummerow
PTAL.
9 years, 3 months ago (2011-09-12 12:18:17 UTC) #1
Yang
On 2011/09/12 12:18:17, Jakob wrote: > PTAL. LGTM. Not sure if we should have test ...
9 years, 3 months ago (2011-09-12 12:30:22 UTC) #2
Jakob Kummerow
9 years, 3 months ago (2011-09-12 12:41:30 UTC) #3
Well, yeah. Test added, landing.

Powered by Google App Engine
This is Rietveld 408576698