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

Issue 10961065: Split test/mjsunit/numops-fuzz into smaller chunks (Closed)

Created:
8 years, 3 months ago by Jakob Kummerow
Modified:
8 years, 3 months ago
Reviewers:
Michael Starzinger
CC:
v8-dev
Visibility:
Public.

Description

Split test/mjsunit/numops-fuzz into smaller chunks Committed: https://code.google.com/p/v8/source/detail?r=12591

Patch Set 1 #

Total comments: 4
Unified diffs Side-by-side diffs Delta from patch set Stats (+4705 lines, -4609 lines) Patch
D test/mjsunit/numops-fuzz.js View 1 chunk +0 lines, -4609 lines 0 comments Download
A test/mjsunit/numops-fuzz-part1.js View 1 chunk +1172 lines, -0 lines 1 comment Download
A test/mjsunit/numops-fuzz-part2.js View 1 chunk +1178 lines, -0 lines 1 comment Download
A test/mjsunit/numops-fuzz-part3.js View 1 chunk +1178 lines, -0 lines 1 comment Download
A test/mjsunit/numops-fuzz-part4.js View 1 chunk +1177 lines, -0 lines 1 comment Download

Messages

Total messages: 3 (0 generated)
Jakob Kummerow
Yo dawg, I heard you like big code reviews... :-) No lines were actually added ...
8 years, 3 months ago (2012-09-24 08:56:20 UTC) #1
Michael Starzinger
LGTM (rubber-stamped with a nit) https://codereview.chromium.org/10961065/diff/1/test/mjsunit/numops-fuzz-part1.js File test/mjsunit/numops-fuzz-part1.js (right): https://codereview.chromium.org/10961065/diff/1/test/mjsunit/numops-fuzz-part1.js#newcode1 test/mjsunit/numops-fuzz-part1.js:1: // Copyright 2011 the ...
8 years, 3 months ago (2012-09-24 09:02:09 UTC) #2
Jakob Kummerow
8 years, 3 months ago (2012-09-24 09:11:08 UTC) #3
As discussed offline, I left the copyright years unchanged intentionally.

Powered by Google App Engine
This is Rietveld 408576698