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

Issue 2738113002: Add strong mode error for mixins defining conflicting private names (issue 28809) (Closed)

Created:
3 years, 9 months ago by Brian Wilkerson
Modified:
3 years, 9 months ago
Reviewers:
vsm
CC:
reviews_dartlang.org, keertip
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Add strong mode error for mixins defining conflicting private names (issue 28809) R=vsm@google.com Committed: https://github.com/dart-lang/sdk/commit/97cc40e0a79eab42b2f5dee7d4f380079b4e5b3a

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+188 lines, -0 lines) Patch
M pkg/analyzer/lib/error/error.dart View 1 chunk +1 line, -0 lines 0 comments Download
M pkg/analyzer/lib/src/error/codes.dart View 1 chunk +11 lines, -0 lines 0 comments Download
M pkg/analyzer/lib/src/generated/error_verifier.dart View 2 chunks +77 lines, -0 lines 0 comments Download
M pkg/analyzer/test/generated/compile_time_error_code_test.dart View 1 chunk +99 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
Brian Wilkerson
The only difference from the last time this code was reviewed is the addition of ...
3 years, 9 months ago (2017-03-09 01:18:30 UTC) #2
vsm
lgtm
3 years, 9 months ago (2017-03-09 01:45:14 UTC) #3
Brian Wilkerson
3 years, 9 months ago (2017-03-22 15:42:35 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
97cc40e0a79eab42b2f5dee7d4f380079b4e5b3a (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698