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

Issue 2897003003: Fix various semantic nits that Fasta will detect. (Closed)

Created:
3 years, 7 months ago by ahe
Modified:
3 years, 6 months ago
CC:
reviews_dartlang.org, dart-fe-team+reviews_google.com
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 (+3 lines, -3 lines) Patch
M pkg/front_end/test/src/incremental/mock_sdk.dart View 1 chunk +1 line, -1 line 1 comment Download
M sdk/lib/_internal/js_runtime/lib/js_helper.dart View 1 chunk +1 line, -1 line 1 comment Download
M sdk/lib/_internal/js_runtime/lib/js_number.dart View 1 chunk +1 line, -1 line 1 comment Download

Depends on Patchset:

Dependent Patchsets:

Messages

Total messages: 8 (3 generated)
ahe
https://codereview.chromium.org/2897003003/diff/1/pkg/front_end/test/src/incremental/mock_sdk.dart File pkg/front_end/test/src/incremental/mock_sdk.dart (right): https://codereview.chromium.org/2897003003/diff/1/pkg/front_end/test/src/incremental/mock_sdk.dart#newcode28 pkg/front_end/test/src/incremental/mock_sdk.dart:28: const Object(); A const constructor can't have a body. ...
3 years, 7 months ago (2017-05-23 14:44:27 UTC) #2
Paul Berry
lgtm
3 years, 7 months ago (2017-05-23 15:52:17 UTC) #4
scheglov
LGTM
3 years, 7 months ago (2017-05-23 16:05:11 UTC) #5
Johnni Winther
lgtm
3 years, 6 months ago (2017-05-24 08:02:10 UTC) #6
ahe
3 years, 6 months ago (2017-05-24 09:43:25 UTC) #8
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
b6a78795e4e264fca4e816f6c83a3b05905767de (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698