Run dartfmt --fix to drop optional new (#224)

- Update the SDK constraint to >=2.0.0
- Update README to drop `new` keyword.
- Hold back lib/**/*.g.dart files since they are generated.

Author: natebosch

README.md

@@ -20,14 +20,14 @@ import 'package:code_builder/code_builder.dart';
 import 'package:dart_style/dart_style.dart';
 
 void main() {
-  final animal = new Class((b) => b
+  final animal = Class((b) => b
     ..name = 'Animal'
     ..extend = refer('Organism')
-    ..methods.add(new Method.returnsVoid((b) => b
+    ..methods.add(Method.returnsVoid((b) => b
       ..name = 'eat'
       ..body = const Code('print(\'Yum\')'))));
-  final emitter = new DartEmitter();
-  print(new DartFormatter().format('${animal.accept(emitter)}'));
+  final emitter = DartEmitter();
+  print(DartFormatter().format('${animal.accept(emitter)}'));
 }
 ```
 
@@ -47,18 +47,18 @@ import 'package:code_builder/code_builder.dart';
 import 'package:dart_style/dart_style.dart';
 
 void main() {
-  final library = new Library((b) => b.body.addAll([
-        new Method((b) => b
+  final library = Library((b) => b.body.addAll([
+        Method((b) => b
           ..body = const Code('')
           ..name = 'doThing'
           ..returns = refer('Thing', 'package:a/a.dart')),
-        new Method((b) => b
+        Method((b) => b
           ..body = const Code('')
           ..name = 'doOther'
           ..returns = refer('Other', 'package:b/b.dart')),
       ]));
-  final emitter = new DartEmitter(new Allocator.simplePrefixing());
-  print(new DartFormatter().format('${library.accept(emitter)}'));
+  final emitter = DartEmitter(Allocator.simplePrefixing());
+  print(DartFormatter().format('${library.accept(emitter)}'));
 }
 ```
 

example/example.dart

@@ -5,7 +5,7 @@
 import 'package:code_builder/code_builder.dart';
 import 'package:dart_style/dart_style.dart';
 
-final _dartfmt = new DartFormatter();
+final _dartfmt = DartFormatter();
 
 void main() {
   print('animalClass():\n${'=' * 40}\n${animalClass()}');
@@ -20,13 +20,13 @@ void main() {
 /// }
 /// ```
 String animalClass() {
-  final animal = new Class((b) => b
+  final animal = Class((b) => b
     ..name = 'Animal'
     ..extend = refer('Organism')
-    ..methods.add(new Method.returnsVoid((b) => b
+    ..methods.add(Method.returnsVoid((b) => b
       ..name = 'eat'
       ..body = refer('print').call([literalString('Yum!')]).code)));
-  return _dartfmt.format('${animal.accept(new DartEmitter())}');
+  return _dartfmt.format('${animal.accept(DartEmitter())}');
 }
 
 /// Outputs:
@@ -40,15 +40,15 @@ String animalClass() {
 /// ```
 String scopedLibrary() {
   final methods = [
-    new Method((b) => b
+    Method((b) => b
       ..body = const Code('')
       ..name = 'doThing'
       ..returns = refer('Thing', 'package:a/a.dart')),
-    new Method((b) => b
+    Method((b) => b
       ..body = const Code('')
       ..name = 'doOther'
       ..returns = refer('Other', 'package:b/b.dart')),
   ];
-  final library = new Library((b) => b.body.addAll(methods));
-  return _dartfmt.format('${library.accept(new DartEmitter.scoped())}');
+  final library = Library((b) => b.body.addAll(methods));
+  return _dartfmt.format('${library.accept(DartEmitter.scoped())}');
 }

lib/src/allocator.dart

@@ -12,7 +12,7 @@ import 'specs/reference.dart';
 /// to resolve all symbols in your generated code.
 abstract class Allocator {
   /// An allocator that does not prefix symbols nor collects imports.
-  static const Allocator none = const _NullAllocator();
+  static const Allocator none = _NullAllocator();
 
   /// Creates a new default allocator that applies no prefixing.
   factory Allocator() = _Allocator;
@@ -46,7 +46,7 @@ abstract class Allocator {
 }
 
 class _Allocator implements Allocator {
-  final _imports = new Set<String>();
+  final _imports = Set<String>();
 
   @override
   String allocate(Reference reference) {
@@ -58,7 +58,7 @@ class _Allocator implements Allocator {
 
   @override
   Iterable<Directive> get imports {
-    return _imports.map((u) => new Directive.import(u));
+    return _imports.map((u) => Directive.import(u));
   }
 }
 
@@ -73,7 +73,7 @@ class _NullAllocator implements Allocator {
 }
 
 class _PrefixedAllocator implements Allocator {
-  static const _doNotPrefix = const ['dart:core'];
+  static const _doNotPrefix = ['dart:core'];
 
   final _imports = <String, int>{};
   var _keys = 1;
@@ -92,7 +92,7 @@ class _PrefixedAllocator implements Allocator {
   @override
   Iterable<Directive> get imports {
     return _imports.keys.map(
-      (u) => new Directive.import(u, as: '_i${_imports[u]}'),
+      (u) => Directive.import(u, as: '_i${_imports[u]}'),
     );
   }
 }

lib/src/base.dart

@@ -9,7 +9,7 @@ abstract class Spec {
 }
 
 /// Returns a generic [Spec] that is lazily generated when visited.
-Spec lazySpec(Spec Function() generate) => new _LazySpec(generate);
+Spec lazySpec(Spec Function() generate) => _LazySpec(generate);
 
 class _LazySpec implements Spec {
   final Spec Function() generate;

lib/src/emitter.dart

@@ -57,7 +57,7 @@ class DartEmitter extends Object
 
   /// Creates a new instance of [DartEmitter] with simple automatic imports.
   factory DartEmitter.scoped() {
-    return new DartEmitter(new Allocator.simplePrefixing());
+    return DartEmitter(Allocator.simplePrefixing());
   }
 
   static bool _isLambdaBody(Code code) =>
@@ -74,15 +74,15 @@ class DartEmitter extends Object
 
   @override
   visitAnnotation(Expression spec, [StringSink output]) {
-    (output ??= new StringBuffer()).write('@');
+    (output ??= StringBuffer()).write('@');
     spec.accept(this, output);
     output.write(' ');
     return output;
   }
 
   @override
   visitClass(Class spec, [StringSink output]) {
-    output ??= new StringBuffer();
+    output ??= StringBuffer();
     spec.docs.forEach(output.writeln);
     spec.annotations.forEach((a) => visitAnnotation(a, output));
     if (spec.abstract) {
@@ -128,7 +128,7 @@ class DartEmitter extends Object
 
   @override
   visitConstructor(Constructor spec, String clazz, [StringSink output]) {
-    output ??= new StringBuffer();
+    output ??= StringBuffer();
     spec.docs.forEach(output.writeln);
     spec.annotations.forEach((a) => visitAnnotation(a, output));
     if (spec.external) {
@@ -212,7 +212,7 @@ class DartEmitter extends Object
 
   @override
   visitDirective(Directive spec, [StringSink output]) {
-    output ??= new StringBuffer();
+    output ??= StringBuffer();
     if (spec.type == DirectiveType.import) {
       output.write('import ');
     } else {
@@ -240,7 +240,7 @@ class DartEmitter extends Object
 
   @override
   visitField(Field spec, [StringSink output]) {
-    output ??= new StringBuffer();
+    output ??= StringBuffer();
     spec.docs.forEach(output.writeln);
     spec.annotations.forEach((a) => visitAnnotation(a, output));
     if (spec.static) {
@@ -274,9 +274,9 @@ class DartEmitter extends Object
 
   @override
   visitLibrary(Library spec, [StringSink output]) {
-    output ??= new StringBuffer();
+    output ??= StringBuffer();
     // Process the body first in order to prime the allocators.
-    final body = new StringBuffer();
+    final body = StringBuffer();
     for (final spec in spec.body) {
       spec.accept(this, body);
       if (spec is Method && _isLambdaMethod(spec)) {
@@ -296,7 +296,7 @@ class DartEmitter extends Object
 
   @override
   visitFunctionType(FunctionType spec, [StringSink output]) {
-    output ??= new StringBuffer();
+    output ??= StringBuffer();
     if (spec.returnType != null) {
       spec.returnType.accept(this, output);
       output.write(' ');
@@ -336,7 +336,7 @@ class DartEmitter extends Object
 
   @override
   visitMethod(Method spec, [StringSink output]) {
-    output ??= new StringBuffer();
+    output ??= StringBuffer();
     spec.docs.forEach(output.writeln);
     spec.annotations.forEach((a) => visitAnnotation(a, output));
     if (spec.external) {
@@ -452,15 +452,15 @@ class DartEmitter extends Object
 
   @override
   visitReference(Reference spec, [StringSink output]) {
-    return (output ??= new StringBuffer())..write(allocator.allocate(spec));
+    return (output ??= StringBuffer())..write(allocator.allocate(spec));
   }
 
   @override
   visitSpec(Spec spec, [StringSink output]) => spec.accept(this, output);
 
   @override
   visitType(TypeReference spec, [StringSink output]) {
-    output ??= new StringBuffer();
+    output ??= StringBuffer();
     // Intentionally not .accept to avoid stack overflow.
     visitReference(spec, output);
     if (spec.bound != null) {
@@ -473,7 +473,7 @@ class DartEmitter extends Object
 
   @override
   visitTypeParameters(Iterable<Reference> specs, [StringSink output]) {
-    output ??= new StringBuffer();
+    output ??= StringBuffer();
     if (specs.isNotEmpty) {
       output
         ..write('<')

lib/src/matchers.dart

@@ -16,7 +16,7 @@ Matcher equalsDart(
   String source, [
   DartEmitter emitter,
 ]) =>
-    new EqualsDart._(EqualsDart._format(source), emitter ?? new DartEmitter());
+    EqualsDart._(EqualsDart._format(source), emitter ?? DartEmitter());
 
 /// Implementation detail of using the [equalsDart] matcher.
 ///

lib/src/specs/class.dart

@@ -72,22 +72,22 @@ abstract class ClassBuilder extends Object
   bool abstract = false;
 
   @override
-  ListBuilder<Expression> annotations = new ListBuilder<Expression>();
+  ListBuilder<Expression> annotations = ListBuilder<Expression>();
 
   @override
-  ListBuilder<String> docs = new ListBuilder<String>();
+  ListBuilder<String> docs = ListBuilder<String>();
 
   Reference extend;
 
-  ListBuilder<Reference> implements = new ListBuilder<Reference>();
-  ListBuilder<Reference> mixins = new ListBuilder<Reference>();
+  ListBuilder<Reference> implements = ListBuilder<Reference>();
+  ListBuilder<Reference> mixins = ListBuilder<Reference>();
 
   @override
-  ListBuilder<Reference> types = new ListBuilder<Reference>();
+  ListBuilder<Reference> types = ListBuilder<Reference>();
 
-  ListBuilder<Constructor> constructors = new ListBuilder<Constructor>();
-  ListBuilder<Method> methods = new ListBuilder<Method>();
-  ListBuilder<Field> fields = new ListBuilder<Field>();
+  ListBuilder<Constructor> constructors = ListBuilder<Constructor>();
+  ListBuilder<Method> methods = ListBuilder<Method>();
+  ListBuilder<Field> fields = ListBuilder<Field>();
 
   /// Name of the class.
   String name;

lib/src/specs/code.dart

@@ -50,7 +50,7 @@ abstract class Block implements Built<Block, BlockBuilder>, Code, Spec {
   factory Block([void updates(BlockBuilder b)]) = _$Block;
 
   factory Block.of(Iterable<Code> statements) {
-    return new Block((b) => b..statements.addAll(statements));
+    return Block((b) => b..statements.addAll(statements));
   }
 
   Block._();
@@ -75,7 +75,7 @@ abstract class BlockBuilder implements Builder<Block, BlockBuilder> {
     statements.add(expression.statement);
   }
 
-  ListBuilder<Code> statements = new ListBuilder<Code>();
+  ListBuilder<Code> statements = ListBuilder<Code>();
 }
 
 /// Knowledge of different types of blocks of code in Dart.
@@ -94,21 +94,21 @@ abstract class CodeEmitter implements CodeVisitor<StringSink> {
 
   @override
   visitBlock(Block block, [StringSink output]) {
-    output ??= new StringBuffer();
+    output ??= StringBuffer();
     return visitAll<Code>(block.statements, output, (statement) {
       statement.accept(this, output);
     }, '\n');
   }
 
   @override
   visitStaticCode(StaticCode code, [StringSink output]) {
-    output ??= new StringBuffer();
+    output ??= StringBuffer();
     return output..write(code.code);
   }
 
   @override
   visitScopedCode(ScopedCode code, [StringSink output]) {
-    output ??= new StringBuffer();
+    output ??= StringBuffer();
     return output..write(code.code(allocator.allocate));
   }
 }
@@ -126,7 +126,7 @@ class LazyCode implements Code {
 }
 
 /// Returns a generic [Code] that is lazily generated when visited.
-Code lazyCode(Code Function() generate) => new _LazyCode(generate);
+Code lazyCode(Code Function() generate) => _LazyCode(generate);
 
 class _LazyCode implements Code {
   final Code Function() generate;

lib/src/specs/constructor.dart

@@ -72,19 +72,19 @@ abstract class ConstructorBuilder extends Object
   ConstructorBuilder._();
 
   @override
-  ListBuilder<Expression> annotations = new ListBuilder<Expression>();
+  ListBuilder<Expression> annotations = ListBuilder<Expression>();
 
   @override
-  ListBuilder<String> docs = new ListBuilder<String>();
+  ListBuilder<String> docs = ListBuilder<String>();
 
   /// Optional parameters.
-  ListBuilder<Parameter> optionalParameters = new ListBuilder<Parameter>();
+  ListBuilder<Parameter> optionalParameters = ListBuilder<Parameter>();
 
   /// Required parameters.
-  ListBuilder<Parameter> requiredParameters = new ListBuilder<Parameter>();
+  ListBuilder<Parameter> requiredParameters = ListBuilder<Parameter>();
 
   /// Constructor initializer statements.
-  ListBuilder<Code> initializers = new ListBuilder<Code>();
+  ListBuilder<Code> initializers = ListBuilder<Code>();
 
   /// Body of the constructor.
   Code body;

lib/src/specs/directive.dart

@@ -20,7 +20,7 @@ abstract class Directive implements Built<Directive, DirectiveBuilder>, Spec {
     List<String> show = const [],
     List<String> hide = const [],
   }) =>
-      new Directive((builder) => builder
+      Directive((builder) => builder
         ..as = as
         ..type = DirectiveType.import
         ..url = url
@@ -33,7 +33,7 @@ abstract class Directive implements Built<Directive, DirectiveBuilder>, Spec {
     List<String> show = const [],
     List<String> hide = const [],
   }) =>
-      new Directive((builder) => builder
+      Directive((builder) => builder
         ..as = as
         ..type = DirectiveType.import
         ..url = url
@@ -46,7 +46,7 @@ abstract class Directive implements Built<Directive, DirectiveBuilder>, Spec {
     List<String> show = const [],
     List<String> hide = const [],
   }) =>
-      new Directive((builder) => builder
+      Directive((builder) => builder
         ..type = DirectiveType.export
         ..url = url
         ..show.addAll(show)