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

Issue 1430953004: Check field overrides (Closed)

Created:
5 years, 1 month ago by vsm
Modified:
5 years, 1 month ago
Reviewers:
Leaf, Jennifer Messerly
CC:
reviews_dartlang.org, devoncarew
Base URL:
https://github.com/dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Check field overrides Fixes https://github.com/dart-lang/dev_compiler/issues/52 This hits once in the SDK - naturally Devon hit it writing his version of hello world. Fix coming in DDC. R=leafp@google.com Committed: https://github.com/dart-lang/sdk/commit/19ff4ca2f76c220d30fce8605fc46111ed8279ff

Patch Set 1 #

Total comments: 1

Patch Set 2 : Properly handle private members #

Patch Set 3 : Fix typi #

Patch Set 4 : fix typo #

Unified diffs Side-by-side diffs Delta from patch set Stats (+184 lines, -37 lines) Patch
M pkg/analyzer/lib/src/task/strong/checker.dart View 1 2 11 chunks +71 lines, -18 lines 0 comments Download
M pkg/analyzer/lib/src/task/strong/info.dart View 1 chunk +9 lines, -0 lines 0 comments Download
M pkg/analyzer/test/src/task/strong/checker_test.dart View 1 8 chunks +92 lines, -7 lines 0 comments Download
M pkg/analyzer/test/src/task/strong/inferred_type_test.dart View 7 chunks +12 lines, -12 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
vsm
5 years, 1 month ago (2015-11-06 14:33:01 UTC) #3
vsm
On 2015/11/06 14:33:01, vsm wrote: PTAL - extra test + fix for private members
5 years, 1 month ago (2015-11-06 17:11:12 UTC) #4
Leaf
lgtm https://codereview.chromium.org/1430953004/diff/1/pkg/analyzer/lib/src/task/strong/checker.dart File pkg/analyzer/lib/src/task/strong/checker.dart (right): https://codereview.chromium.org/1430953004/diff/1/pkg/analyzer/lib/src/task/strong/checker.dart#newcode308 pkg/analyzer/lib/src/task/strong/checker.dart:308: // Disallow any overridding if the base class ...
5 years, 1 month ago (2015-11-06 21:08:36 UTC) #5
vsm
5 years, 1 month ago (2015-11-06 22:04:55 UTC) #6
Message was sent while issue was closed.
Committed patchset #4 (id:60001) manually as
19ff4ca2f76c220d30fce8605fc46111ed8279ff (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698