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

Unified Diff: test/message_generator_test.dart

Issue 1829573002: Fix all strong mode warnings in protoc-plugin (Closed) Base URL: git@github.com:dart-lang/dart-protoc-plugin.git@master
Patch Set: Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: test/message_generator_test.dart
diff --git a/test/message_generator_test.dart b/test/message_generator_test.dart
index df82b3fa7713b123ce27b83cf72d66fd43a33298..86e5f6d6ca2df0ef5983270f425c88681d0c0df4 100755
--- a/test/message_generator_test.dart
+++ b/test/message_generator_test.dart
@@ -28,8 +28,8 @@ class PhoneNumber_PhoneType extends ProtobufEnum {
WORK,
];
- static final Map<int, PhoneNumber_PhoneType> _byValue = ProtobufEnum.initByValue(values);
- static PhoneNumber_PhoneType valueOf(int value) => _byValue[value];
+ static final Map<int, dynamic> _byValue = ProtobufEnum.initByValue(values);
+ static PhoneNumber_PhoneType valueOf(int value) => _byValue[value] as PhoneNumber_PhoneType;
static void $checkItem(PhoneNumber_PhoneType v) {
if (v is !PhoneNumber_PhoneType) checkItemFailed(v, 'PhoneNumber_PhoneType');
}
@@ -60,17 +60,17 @@ class PhoneNumber extends GeneratedMessage {
if (v is !PhoneNumber) checkItemFailed(v, 'PhoneNumber');
}
- String get number => $_get(0, 1, '');
+ String get number => $_get(0, 1, '') as String;
void set number(String v) { $_setString(0, 1, v); }
bool hasNumber() => $_has(0, 1);
void clearNumber() => clearField(1);
- PhoneNumber_PhoneType get type => $_get(1, 2, null);
+ PhoneNumber_PhoneType get type => $_get(1, 2, null) as PhoneNumber_PhoneType;
void set type(PhoneNumber_PhoneType v) { setField(2, v); }
bool hasType() => $_has(1, 2);
void clearType() => clearField(2);
- String get name => $_get(2, 3, '\$');
+ String get name => $_get(2, 3, '\$') as String;
void set name(String v) { $_setString(2, 3, v); }
bool hasName() => $_has(2, 3);
void clearName() => clearField(3);
@@ -81,44 +81,45 @@ class _ReadonlyPhoneNumber extends PhoneNumber with ReadonlyMessageMixin {}
''';
FileDescriptorProto fd = new FileDescriptorProto();
EnumDescriptorProto ed = new EnumDescriptorProto()
- ..name = 'PhoneType'
- ..value.addAll([
- new EnumValueDescriptorProto()
- ..name = 'MOBILE'
- ..number = 0,
- new EnumValueDescriptorProto()
- ..name = 'HOME'
- ..number = 1,
- new EnumValueDescriptorProto()
- ..name = 'WORK'
- ..number = 2,
- new EnumValueDescriptorProto()
- ..name = 'BUSINESS'
- ..number = 2]);
+ ..name = 'PhoneType'
+ ..value.addAll([
+ new EnumValueDescriptorProto()
+ ..name = 'MOBILE'
+ ..number = 0,
+ new EnumValueDescriptorProto()
+ ..name = 'HOME'
+ ..number = 1,
+ new EnumValueDescriptorProto()
+ ..name = 'WORK'
+ ..number = 2,
+ new EnumValueDescriptorProto()
+ ..name = 'BUSINESS'
+ ..number = 2
+ ]);
DescriptorProto md = new DescriptorProto()
- ..name = 'PhoneNumber'
- ..field.addAll([
- // optional PhoneType type = 2 [default = HOME];
- new FieldDescriptorProto()
- ..name = 'type'
- ..number = 2
- ..label = FieldDescriptorProto_Label.LABEL_OPTIONAL
- ..type = FieldDescriptorProto_Type.TYPE_ENUM
- ..typeName = '.PhoneNumber.PhoneType',
- // required string number = 1;
- new FieldDescriptorProto()
- ..name = 'number'
- ..number = 1
- ..label = FieldDescriptorProto_Label.LABEL_REQUIRED
- ..type = FieldDescriptorProto_Type.TYPE_STRING,
- new FieldDescriptorProto()
- ..name = 'name'
- ..number = 3
- ..label = FieldDescriptorProto_Label.LABEL_OPTIONAL
- ..type = FieldDescriptorProto_Type.TYPE_STRING
- ..defaultValue = r'$'
- ])
- ..enumType.add(ed);
+ ..name = 'PhoneNumber'
+ ..field.addAll([
+ // optional PhoneType type = 2 [default = HOME];
+ new FieldDescriptorProto()
+ ..name = 'type'
+ ..number = 2
+ ..label = FieldDescriptorProto_Label.LABEL_OPTIONAL
+ ..type = FieldDescriptorProto_Type.TYPE_ENUM
+ ..typeName = '.PhoneNumber.PhoneType',
+ // required string number = 1;
+ new FieldDescriptorProto()
+ ..name = 'number'
+ ..number = 1
+ ..label = FieldDescriptorProto_Label.LABEL_REQUIRED
+ ..type = FieldDescriptorProto_Type.TYPE_STRING,
+ new FieldDescriptorProto()
+ ..name = 'name'
+ ..number = 3
+ ..label = FieldDescriptorProto_Label.LABEL_OPTIONAL
+ ..type = FieldDescriptorProto_Type.TYPE_STRING
+ ..defaultValue = r'$'
+ ])
+ ..enumType.add(ed);
var options = parseGenerationOptions(
new CodeGeneratorRequest(), new CodeGeneratorResponse());

Powered by Google App Engine
This is Rietveld 408576698