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

Issue 2294753002: Handle parts as input in resolver. (Closed)

Created:
4 years, 3 months ago by Johnni Winther
Modified:
4 years, 3 months ago
Reviewers:
Harry Terkelsen
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 5

Patch Set 2 : Updated cf. comments. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+86 lines, -109 lines) Patch
M pkg/compiler/bin/resolver.dart View 3 chunks +8 lines, -54 lines 0 comments Download
M pkg/compiler/lib/src/constants/constant_constructors.dart View 1 chunk +12 lines, -0 lines 0 comments Download
A + pkg/compiler/lib/src/dart2js_resolver.dart View 1 2 chunks +28 lines, -42 lines 0 comments Download
M tests/compiler/dart2js/analyze_test_test.dart View 2 chunks +19 lines, -13 lines 0 comments Download
A tests/compiler/dart2js/dart2js_resolver_test.dart View 1 chunk +19 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
Johnni Winther
https://codereview.chromium.org/2294753002/diff/1/pkg/compiler/lib/src/constants/constant_constructors.dart File pkg/compiler/lib/src/constants/constant_constructors.dart (right): https://codereview.chromium.org/2294753002/diff/1/pkg/compiler/lib/src/constants/constant_constructors.dart#newcode271 pkg/compiler/lib/src/constants/constant_constructors.dart:271: ConstantExpression visitEquals(Send node, Node left, Node right, _) { ...
4 years, 3 months ago (2016-08-30 11:22:42 UTC) #2
Harry Terkelsen
lgtm https://codereview.chromium.org/2294753002/diff/1/pkg/compiler/lib/src/dart2js_resolver.dart File pkg/compiler/lib/src/dart2js_resolver.dart (right): https://codereview.chromium.org/2294753002/diff/1/pkg/compiler/lib/src/dart2js_resolver.dart#newcode40 pkg/compiler/lib/src/dart2js_resolver.dart:40: platformConfig != null ? libraryRoot.resolve(platformConfig) : null); nit: ...
4 years, 3 months ago (2016-08-30 16:31:50 UTC) #3
Johnni Winther
https://codereview.chromium.org/2294753002/diff/1/pkg/compiler/lib/src/dart2js_resolver.dart File pkg/compiler/lib/src/dart2js_resolver.dart (right): https://codereview.chromium.org/2294753002/diff/1/pkg/compiler/lib/src/dart2js_resolver.dart#newcode40 pkg/compiler/lib/src/dart2js_resolver.dart:40: platformConfig != null ? libraryRoot.resolve(platformConfig) : null); On 2016/08/30 ...
4 years, 3 months ago (2016-08-31 08:49:51 UTC) #4
Johnni Winther
4 years, 3 months ago (2016-08-31 09:04:50 UTC) #6
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
64a2a03f0e0784aaa8f29418ea32c93989b8e9b6 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698