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

Issue 171038: Fix issue 420: accept truncated log files. (Closed)

Created:
11 years, 4 months ago by mnaganov (inactive)
Modified:
9 years, 7 months ago
CC:
v8-dev
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 2

Patch Set 2 : Comment addressed #

Unified diffs Side-by-side diffs Delta from patch set Stats (+27 lines, -4 lines) Patch
M test/mjsunit/mjsunit.js View 1 chunk +6 lines, -2 lines 0 comments Download
M test/mjsunit/tools/logreader.js View 1 1 chunk +16 lines, -0 lines 0 comments Download
M tools/logreader.js View 1 chunk +5 lines, -2 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
mnaganov (inactive)
11 years, 4 months ago (2009-08-17 09:16:45 UTC) #1
Søren Thygesen Gjesse
LGTM http://codereview.chromium.org/171038/diff/1/3 File test/mjsunit/tools/logreader.js (right): http://codereview.chromium.org/171038/diff/1/3#newcode85 Line 85: (function testIssue420() { Please call this function ...
11 years, 4 months ago (2009-08-17 09:25:18 UTC) #2
mnaganov (inactive)
11 years, 4 months ago (2009-08-17 09:31:35 UTC) #3
http://codereview.chromium.org/171038/diff/1/3
File test/mjsunit/tools/logreader.js (right):

http://codereview.chromium.org/171038/diff/1/3#newcode85
Line 85: (function testIssue420() {
On 2009/08/17 09:25:19, Søren Gjesse wrote:
> Please call this function something like testTruncatedLog. Alternatively add a
> regress sub-directory with the file regress-420.js.

OK. Renamed and added a link to an issue.

Powered by Google App Engine
This is Rietveld 408576698