Dart 2 fixes. (#106)

* Breaking change: Generated RpcClient stubs use the generic invoke method. Requires package:protobuf version 0.8.0 or newer.
* Dart 2 fixes.

Author: jakobr-google

.travis.yml

@@ -5,9 +5,9 @@ sudo: false
 # them against each Dart version.
 matrix:
   include:
-  - dart: stable
+  - dart: dev
     dart_task: dartfmt
-  - dart: stable
+  - dart: dev
     env: PROTOC_VERSION=3.0.0
     install: ./tool/travis/setup.sh
     script: ./tool/travis/test.sh

CHANGELOG.md

@@ -1,3 +1,9 @@
+## 0.8.0 - 2018-05-17
+
+* Breaking change: Generated RpcClient stubs use the generic invoke method.
+  Requires package:protobuf version 0.8.0 or newer.
+* Dart 2 fixes.
+
 ## 0.7.11 - 2018-04-09
 
 * Dart 2 fix.

benchmark/lib/report.dart

@@ -4,7 +4,7 @@
 
 library protoc.benchmark.report;
 
-import 'dart:convert' show JSON;
+import 'dart:convert' show jsonEncode;
 
 import 'generated/benchmark.pb.dart' as pb;
 
@@ -159,15 +159,15 @@ String _stringifyMap(StringBuffer out, Map json, String indent) {
 
     var value = json[key];
     out.write(childIndent);
-    out.write(JSON.encode(key));
+    out.write(jsonEncode(key));
     out.write(": ");
 
     if (value is List) {
       _stringifyList(out, value, childIndent);
     } else if (value is Map && indent.length < 4) {
       _stringifyMap(out, value, childIndent);
     } else {
-      out.write(JSON.encode(value));
+      out.write(jsonEncode(value));
     }
   }
   out.write("\n$indent}");
@@ -182,7 +182,7 @@ void _stringifyList(StringBuffer out, List json, String indent) {
     if (!first) out.write(",\n");
     first = false;
     out.write(childIndent);
-    out.write(JSON.encode(item));
+    out.write(jsonEncode(item));
   }
   out.write("\n$indent]");
 }

lib/base_type.dart

@@ -70,7 +70,7 @@ class BaseType {
             "double", r"$_setDouble", null);
       case FieldDescriptorProto_Type.TYPE_INT32:
         return const BaseType._raw(FieldDescriptorProto_Type.TYPE_INT32, "3",
-            "int", r"$_setUnsignedInt32", null);
+            "int", r"$_setSignedInt32", null);
       case FieldDescriptorProto_Type.TYPE_UINT32:
         return const BaseType._raw(FieldDescriptorProto_Type.TYPE_UINT32, "U3",
             "int", r"$_setUnsignedInt32", null);

lib/client_generator.dart

@@ -37,7 +37,8 @@ class ClientApiGenerator {
         'ClientContext ctx, $inputType request) {',
         '}', () {
       out.println('var emptyResponse = new $outputType();');
-      out.println('return _client.invoke(ctx, \'${service._descriptor.name}\', '
+      out.println(
+          'return _client.invoke<$outputType>(ctx, \'${service._descriptor.name}\', '
           '\'${m.name}\', request, emptyResponse);');
     });
   }

lib/protobuf_field.dart

@@ -203,11 +203,11 @@ class ProtobufField {
             '0' == descriptor.defaultValue) {
           return null;
         } else if (descriptor.defaultValue == 'inf') {
-          return 'double.INFINITY';
+          return 'double.infinity';
         } else if (descriptor.defaultValue == '-inf') {
-          return 'double.NEGATIVE_INFINITY';
+          return 'double.negativeInfinity';
         } else if (descriptor.defaultValue == 'nan') {
-          return 'double.NAN';
+          return 'double.nan';
         } else if (HEX_LITERAL_REGEX.hasMatch(descriptor.defaultValue)) {
           return '(${descriptor.defaultValue}).toDouble()';
         } else if (INTEGER_LITERAL_REGEX.hasMatch(descriptor.defaultValue)) {

lib/src/descriptor.pb.dart

@@ -1,5 +1,5 @@
 name: protoc_plugin
-version: 0.7.11
+version: 0.8.0
 author: Dart Team <misc@dartlang.org>
 description: Protoc compiler plugin to generate Dart code
 homepage: https://github.com/dart-lang/dart-protoc-plugin
@@ -8,7 +8,7 @@ environment:
 dependencies:
   fixnum: ^0.10.5
   path: ^1.0.0
-  protobuf: ^0.7.1
+  protobuf: ^0.8.0
   dart_style: ^1.0.6
 dev_dependencies:
   browser: any

lib/src/plugin.pb.dart

@@ -21,11 +21,11 @@ class TestApi {
 
   Future<SomeReply> aMethod(ClientContext ctx, SomeRequest request) {
     var emptyResponse = new SomeReply();
-    return _client.invoke(ctx, 'Test', 'AMethod', request, emptyResponse);
+    return _client.invoke<SomeReply>(ctx, 'Test', 'AMethod', request, emptyResponse);
   }
   Future<$foo$bar.AnotherReply> anotherMethod(ClientContext ctx, $foo$bar.EmptyMessage request) {
     var emptyResponse = new $foo$bar.AnotherReply();
-    return _client.invoke(ctx, 'Test', 'AnotherMethod', request, emptyResponse);
+    return _client.invoke<$foo$bar.AnotherReply>(ctx, 'Test', 'AnotherMethod', request, emptyResponse);
   }
 }