Skip to content

Commit

Permalink
Avoid private override via mixin
Browse files Browse the repository at this point in the history
This is now an error in strong mode. See:
dart-lang/sdk#28809

BUG=
R=kevmoo@google.com

Review-Url: https://codereview.chromium.org//2778923002 .
  • Loading branch information
Brian Slesinsky committed Mar 27, 2017
1 parent 8010b70 commit 814653b
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
2 changes: 1 addition & 1 deletion lib/src/protobuf/generated_message.dart
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ abstract class GeneratedMessage {

UnknownFieldSet get unknownFields => _fieldSet._ensureUnknownFields();

bool get _isReadOnly => false;
bool get _isReadOnly => this is ReadonlyMessageMixin;

bool hasRequiredFields() => info_.hasRequiredFields;

Expand Down
2 changes: 0 additions & 2 deletions lib/src/protobuf/readonly_message.dart
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,6 @@ part of protobuf;
abstract class ReadonlyMessageMixin {
BuilderInfo get info_;

bool get _isReadOnly => true;

void addExtension(Extension extension, var value) =>
_readonly("addExtension");

Expand Down

0 comments on commit 814653b

Please sign in to comment.