Use real generic syntax instead of comment-based. (#89)

Also update pubspec to support v2 dev SDKs.

Author: jakobr-google

CHANGELOG.md

@@ -1,3 +1,8 @@
+## 0.7.5 - 2017-08-04
+
+* Use real generic syntax instead of comment-based.
+* Support 2.0.0 dev SDKs.
+
 ## 0.7.4 - 2017-07-07
 
 * Added call options to gRPC client stubs.

lib/protobuf_field.dart

@@ -106,24 +106,24 @@ class ProtobufField {
 
     if (isRepeated) {
       if (baseType.isMessage || baseType.isGroup) {
-        return '..pp/*<$type>*/($number, $quotedName, $typeConstant,'
+        return '..pp<$type>($number, $quotedName, $typeConstant,'
             ' $type.$checkItem, $type.create)';
       } else if (baseType.isEnum) {
-        return '..pp/*<$type>*/($number, $quotedName, $typeConstant,'
+        return '..pp<$type>($number, $quotedName, $typeConstant,'
             ' $type.$checkItem, null, $type.valueOf)';
       } else {
-        return '..p/*<$type>*/($number, $quotedName, $typeConstant)';
+        return '..p<$type>($number, $quotedName, $typeConstant)';
       }
     }
 
     String makeDefault = generateDefaultFunction(package);
     if (baseType.isEnum) {
       String valueOf = '$type.valueOf';
-      return '..e/*<$type>*/('
+      return '..e<$type>('
           '$number, $quotedName, $typeConstant, $makeDefault, $valueOf)';
     }
 
-    String prefix = '..a/*<$type>*/($number, $quotedName, $typeConstant';
+    String prefix = '..a<$type>($number, $quotedName, $typeConstant';
     if (makeDefault == null) return prefix + ')';
 
     if (baseType.isMessage || baseType.isGroup) {

lib/src/dart_options.pb.dart

@@ -1,14 +1,14 @@
 name: protoc_plugin
-version: 0.7.4
+version: 0.7.5
 author: Dart Team <[email protected]>
 description: Protoc compiler plugin to generate Dart code
 homepage: https://github.com/dart-lang/dart-protoc-plugin
 environment:
-  sdk: '>=1.13.0 <2.0.0'
+  sdk: '>=1.21.0 <2.0.0-dev.infinity'
 dependencies:
   fixnum: ^0.10.5
   path: ^1.0.0
-  protobuf: ^0.5.4
+  protobuf: ^0.5.5
   dart_style: ^1.0.6
 dev_dependencies:
   browser: any

lib/src/descriptor.pb.dart

@@ -87,9 +87,9 @@ import 'package:protobuf/protobuf.dart';
 
 class PhoneNumber extends GeneratedMessage {
   static final BuilderInfo _i = new BuilderInfo('PhoneNumber')
-    ..a/*<String>*/(1, 'number', PbFieldType.QS)
-    ..a/*<int>*/(2, 'type', PbFieldType.O3)
-    ..a/*<String>*/(3, 'name', PbFieldType.OS, '\$')
+    ..a<String>(1, 'number', PbFieldType.QS)
+    ..a<int>(2, 'type', PbFieldType.O3)
+    ..a<String>(3, 'name', PbFieldType.OS, '\$')
   ;
 
   PhoneNumber() : super();
@@ -693,9 +693,9 @@ import 'package2.pb.dart' as p2;
 
 class M extends GeneratedMessage {
   static final BuilderInfo _i = new BuilderInfo('M')
-    ..a/*<M>*/(1, 'm', PbFieldType.OM, M.getDefault, M.create)
-    ..a/*<p1.M>*/(2, 'm1', PbFieldType.OM, p1.M.getDefault, p1.M.create)
-    ..a/*<p2.M>*/(3, 'm2', PbFieldType.OM, p2.M.getDefault, p2.M.create)
+    ..a<M>(1, 'm', PbFieldType.OM, M.getDefault, M.create)
+    ..a<p1.M>(2, 'm1', PbFieldType.OM, p1.M.getDefault, p1.M.create)
+    ..a<p2.M>(3, 'm2', PbFieldType.OM, p2.M.getDefault, p2.M.create)
     ..hasRequiredFields = false
   ;
 

lib/src/plugin.pb.dart

@@ -43,9 +43,9 @@ class PhoneNumber_PhoneType extends ProtobufEnum {
     String expected = r'''
 class PhoneNumber extends GeneratedMessage {
   static final BuilderInfo _i = new BuilderInfo('PhoneNumber')
-    ..a/*<String>*/(1, 'number', PbFieldType.QS)
-    ..e/*<PhoneNumber_PhoneType>*/(2, 'type', PbFieldType.OE, PhoneNumber_PhoneType.MOBILE, PhoneNumber_PhoneType.valueOf)
-    ..a/*<String>*/(3, 'name', PbFieldType.OS, '\$')
+    ..a<String>(1, 'number', PbFieldType.QS)
+    ..e<PhoneNumber_PhoneType>(2, 'type', PbFieldType.OE, PhoneNumber_PhoneType.MOBILE, PhoneNumber_PhoneType.valueOf)
+    ..a<String>(3, 'name', PbFieldType.OS, '\$')
   ;
 
   PhoneNumber() : super();