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

Issue 2238353002: Additional fixes to zero_termination_test (Closed)

Created:
4 years, 4 months ago by Siggi Cherem (dart-lang)
Modified:
4 years, 4 months ago
Reviewers:
Bill Hesse, sra1
CC:
reviews_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -4 lines) Patch
M tests/compiler/dart2js/zero_termination_test.dart View 1 chunk +8 lines, -4 lines 3 comments Download

Messages

Total messages: 6 (2 generated)
Siggi Cherem (dart-lang)
https://codereview.chromium.org/2238353002/diff/1/tests/compiler/dart2js/zero_termination_test.dart File tests/compiler/dart2js/zero_termination_test.dart (left): https://codereview.chromium.org/2238353002/diff/1/tests/compiler/dart2js/zero_termination_test.dart#oldcode51 tests/compiler/dart2js/zero_termination_test.dart:51: String command = path.join(path.dirname(executable), 'dart2js'); this was dead code. ...
4 years, 4 months ago (2016-08-12 20:26:37 UTC) #2
sra1
lgtm
4 years, 4 months ago (2016-08-12 21:32:40 UTC) #3
Siggi Cherem (dart-lang)
Committed patchset #1 (id:1) manually as 02fa5aad8ef05bd057e5cd8fc74ebc45d5fe26aa (presubmit successful).
4 years, 4 months ago (2016-08-12 21:33:29 UTC) #5
Bill Hesse
4 years, 4 months ago (2016-08-15 15:48:36 UTC) #6
Message was sent while issue was closed.
lgtm

https://codereview.chromium.org/2238353002/diff/1/tests/compiler/dart2js/zero...
File tests/compiler/dart2js/zero_termination_test.dart (right):

https://codereview.chromium.org/2238353002/diff/1/tests/compiler/dart2js/zero...
tests/compiler/dart2js/zero_termination_test.dart:54: if (Platform.packageRoot
!= null) {
On 2016/08/12 20:26:37, Siggi Cherem (dart-lang) wrote:
> this addresses Bill's comments from the other CL.

Acknowledged.

Powered by Google App Engine
This is Rietveld 408576698