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

Unified Diff: lib/protobuf_field.dart

Issue 813373003: Protobuf changes for smaller dart2js code, Int64 fixes (Closed) Base URL: https://github.com/dart-lang/dart-protoc-plugin@master
Patch Set: bump version Created 6 years 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: lib/protobuf_field.dart
diff --git a/lib/protobuf_field.dart b/lib/protobuf_field.dart
index ca6d4d24fb777f0711ea28f2be6d3ee7dad30725..7936f9c739aec8105e29584016e29580e49bf39a 100644
--- a/lib/protobuf_field.dart
+++ b/lib/protobuf_field.dart
@@ -139,7 +139,7 @@ class ProtobufField {
codedStreamType = 'Bool';
if (!repeats) {
if (field.hasDefaultValue() && 'false' != field.defaultValue) {
- initialization = '()${SP}=>${SP}${field.defaultValue}';
+ initialization = '${field.defaultValue}';
}
}
break;
@@ -155,19 +155,18 @@ class ProtobufField {
if (field.hasDefaultValue() &&
('0.0' != field.defaultValue || '0' != field.defaultValue)) {
if (field.defaultValue == 'inf') {
- initialization = '()${SP}=>${SP}double.INFINITY';
+ initialization = 'double.INFINITY';
} else if (field.defaultValue == '-inf') {
- initialization = '()${SP}=>${SP}double.NEGATIVE_INFINITY';
+ initialization = 'double.NEGATIVE_INFINITY';
} else if (field.defaultValue == 'nan') {
- initialization = '()${SP}=>${SP}double.NAN';
+ initialization = 'double.NAN';
} else if (HEX_LITERAL_REGEX.hasMatch(field.defaultValue)) {
- initialization = '()${SP}=>${SP}(${field.defaultValue})'
- '.toDouble()';
+ initialization = '(${field.defaultValue}).toDouble()';
} else if (INTEGER_LITERAL_REGEX.hasMatch(field.defaultValue)) {
- initialization = '()${SP}=>${SP}${field.defaultValue}.0';
+ initialization = '${field.defaultValue}.0';
} else if (DECIMAL_LITERAL_REGEX_A.hasMatch(field.defaultValue)
|| DECIMAL_LITERAL_REGEX_B.hasMatch(field.defaultValue)) {
- initialization = '()${SP}=>${SP}${field.defaultValue}';
+ initialization = '${field.defaultValue}';
} else {
throw new InvalidDefaultValue.double(
field.name, field.defaultValue);
@@ -202,7 +201,7 @@ class ProtobufField {
}
if (!repeats) {
if (field.hasDefaultValue() && '0' != field.defaultValue) {
- initialization = '()${SP}=>${SP}${field.defaultValue}';
+ initialization = '${field.defaultValue}';
}
}
break;
@@ -234,7 +233,11 @@ class ProtobufField {
if (!repeats) {
final defaultValue = field.hasDefaultValue() ?
field.defaultValue : '0';
- initialization = '()${SP}=>${SP}makeLongInt($defaultValue)';
+ if (defaultValue == '0') {
+ initialization = 'Int64.ZERO';
+ } else {
+ initialization = "parseLongInt('$defaultValue')";
+ }
}
break;
case FieldDescriptorProto_Type.TYPE_STRING:
@@ -244,7 +247,7 @@ class ProtobufField {
if (!repeats) {
if (field.hasDefaultValue() && !field.defaultValue.isEmpty) {
String defaultValue = field.defaultValue.replaceAll(r'$', r'\$');
- initialization = '()${SP}=>${SP}\'$defaultValue\'';
+ initialization = '\'$defaultValue\'';
}
}
break;
@@ -296,8 +299,8 @@ class ProtobufField {
} else {
throw 'FAILURE: Unknown message type reference ${field.typeName}';
}
- initialization = '()${SP}=>${SP}new ${baseType}()';
- prefixedInitialization = '()${SP}=>${SP}new ${prefixedBaseType}()';
+ initialization = '${baseType}.create';
+ prefixedInitialization = '${prefixedBaseType}.create';
break;
case FieldDescriptorProto_Type.TYPE_ENUM:
EnumGenerator enumType = context[field.typeName];
@@ -316,16 +319,14 @@ class ProtobufField {
if (!repeats) {
if (field.hasDefaultValue() && !field.defaultValue.isEmpty) {
initialization =
- '()${SP}=>${SP}${baseType}.${field.defaultValue}';
+ '${baseType}.${field.defaultValue}';
prefixedInitialization =
- '()${SP}=>${SP}${prefixedBaseType}.${field.defaultValue}';
+ '${prefixedBaseType}.${field.defaultValue}';
} else if (!enumType._canonicalValues.isEmpty) {
initialization =
- '()${SP}=>${SP}${baseType}.'
- '${enumType._canonicalValues[0].name}';
+ '${baseType}.${enumType._canonicalValues[0].name}';
prefixedInitialization =
- '()${SP}=>${SP}${prefixedBaseType}.'
- '${enumType._canonicalValues[0].name}';
+ '${prefixedBaseType}.${enumType._canonicalValues[0].name}';
}
}
} else {

Powered by Google App Engine
This is Rietveld 408576698