sort classes in dependency order, or load lazily if needed, fixes #78

test/codegen/expect/dart/core.js

 var core;
 (function(exports) {
   'use strict';
+  class Object {
+    Object() {
+      var name = this.constructor.name;
+      var init = this[name];
+      var result = void 0;
+      if (init)
+        result = init.apply(this, arguments);
+      return result === void 0 ? this : result;
+    }
+    ['=='](other) {
+      return identical(this, other);
+    }
+    get hashCode() {
+      return _js_helper.Primitives.objectHashCode(this);
+    }
+    toString() {
+      return _js_helper.Primitives.objectToString(this);
+    }
+    noSuchMethod(invocation) {
+      throw new NoSuchMethodError(this, invocation.memberName, invocation.positionalArguments, invocation.namedArguments);
+    }
+    get runtimeType() {
+      return _js_helper.getRuntimeType(this);
+    }
+  }
   class Deprecated extends Object {
     Deprecated(expires) {
       this.expires = expires;
     }
     toString() {
       return `Deprecated feature. Will be removed ${this.expires}`;
     }
   }
   class _Override extends Object {
     _Override() {
@@ skipping 293 matching lines @@
   DateTime.MAY = 5;
   DateTime.JUNE = 6;
   DateTime.JULY = 7;
   DateTime.AUGUST = 8;
   DateTime.SEPTEMBER = 9;
   DateTime.OCTOBER = 10;
   DateTime.NOVEMBER = 11;
   DateTime.DECEMBER = 12;
   DateTime.MONTHS_PER_YEAR = 12;
   DateTime._MAX_MILLISECONDS_SINCE_EPOCH = 8640000000000000;
+  let _onParseErrorInt = Symbol('_onParseErrorInt');
+  let _onParseErrorDouble = Symbol('_onParseErrorDouble');
+  class num extends Object {
+    static parse(input, onError) {
+      if (onError === void 0)
+        onError = null;
+      let source = input.trim();
+      _parseError = false;
+      let result = int.parse(source, {onError: _onParseErrorInt});
+      if (!dart.notNull(_parseError))
+        return result;
+      _parseError = false;
+      result = double.parse(source, _onParseErrorDouble);
+      if (!dart.notNull(_parseError))
+        return result;
+      if (onError === null)
+        throw new FormatException(input);
+      return onError(input);
+    }
+    static [_onParseErrorInt](_) {
+      _parseError = true;
+      return 0;
+    }
+    static [_onParseErrorDouble](_) {
+      _parseError = true;
+      return 0.0;
+    }
+  }
+  num._parseError = false;
   class double extends num {
     static parse(source, onError) {
       if (onError === void 0)
         onError = null;
       return _js_helper.Primitives.parseDouble(source, onError);
     }
   }
   double.NAN = 0.0 / 0.0;
   double.INFINITY = 1.0 / 0.0;
   double.NEGATIVE_INFINITY = -dart.notNull(double.INFINITY);
@@ skipping 741 matching lines @@
           return this;
         return new _GeneratorIterable.slice(this[_start], newEnd, this[_generator]);
       }
       static [_id](n) {
         return n;
       }
     }
     dart.defineNamedConstructor(_GeneratorIterable, 'slice');
     return _GeneratorIterable;
   });
-  let _GeneratorIterable = _GeneratorIterable$(dart.dynamic);
+  dart.defineLazyClassGeneric(exports, '_GeneratorIterable', {get: _GeneratorIterable$});
   let _index = Symbol('_index');
   let _current = Symbol('_current');
   let _GeneratorIterator$ = dart.generic(function(E) {
     class _GeneratorIterator extends Object {
       _GeneratorIterator($_index, $_end, $_generator) {
         this[_index] = $_index;
         this[_end] = $_end;
         this[_generator] = $_generator;
         this[_current] = null;
       }
@@ skipping 105 matching lines @@
   let Map = Map$(dart.dynamic, dart.dynamic);
   class Null extends Object {
     Null$_uninstantiable() {
       throw new UnsupportedError('class Null cannot be instantiated');
     }
     toString() {
       return "null";
     }
   }
   dart.defineNamedConstructor(Null, '_uninstantiable');
-  let _onParseErrorInt = Symbol('_onParseErrorInt');
-  let _onParseErrorDouble = Symbol('_onParseErrorDouble');
-  class num extends Object {
-    static parse(input, onError) {
-      if (onError === void 0)
-        onError = null;
-      let source = input.trim();
-      _parseError = false;
-      let result = int.parse(source, {onError: _onParseErrorInt});
-      if (!dart.notNull(_parseError))
-        return result;
-      _parseError = false;
-      result = double.parse(source, _onParseErrorDouble);
-      if (!dart.notNull(_parseError))
-        return result;
-      if (onError === null)
-        throw new FormatException(input);
-      return onError(input);
-    }
-    static [_onParseErrorInt](_) {
-      _parseError = true;
-      return 0;
-    }
-    static [_onParseErrorDouble](_) {
-      _parseError = true;
-      return 0.0;
-    }
-  }
-  num._parseError = false;
-  class Object {
-    Object() {
-      var name = this.constructor.name;
-      var init = this[name];
-      var result = void 0;
-      if (init)
-        result = init.apply(this, arguments);
-      return result === void 0 ? this : result;
-    }
-    ['=='](other) {
-      return identical(this, other);
-    }
-    get hashCode() {
-      return _js_helper.Primitives.objectHashCode(this);
-    }
-    toString() {
-      return _js_helper.Primitives.objectToString(this);
-    }
-    noSuchMethod(invocation) {
-      throw new NoSuchMethodError(this, invocation.memberName, invocation.positionalArguments, invocation.namedArguments);
-    }
-    get runtimeType() {
-      return _js_helper.getRuntimeType(this);
-    }
-  }
   class Pattern extends Object {
   }
   // Function print: (Object) → void
   function print(object) {
     let line = `${object}`;
     if (_internal.printToZone === null) {
       _internal.printToConsole(line);
     } else {
       dart.dinvokef(_internal.printToZone, line);
     }
@@ skipping 16 matching lines @@
         return new collection.LinkedHashSet.identity();
       }
       Set$from(elements) {
         return new collection.LinkedHashSet.from(elements);
       }
     }
     dart.defineNamedConstructor(Set, 'identity');
     dart.defineNamedConstructor(Set, 'from');
     return Set;
   });
-  let Set = Set$(dart.dynamic);
+  dart.defineLazyClassGeneric(exports, 'Set', {get: Set$});
   let Sink$ = dart.generic(function(T) {
     class Sink extends Object {
     }
     return Sink;
   });
   let Sink = Sink$(dart.dynamic);
   class StackTrace extends Object {
   }
   let _stop = Symbol('_stop');
   let _initTicker = Symbol('_initTicker');
@@ skipping 112 matching lines @@
           }
           list.add(it.current);
         }
       }
       return _js_helper.Primitives.stringFromCharCodes(list);
     }
   }
   dart.defineNamedConstructor(String, 'fromCharCodes');
   dart.defineNamedConstructor(String, 'fromCharCode');
   dart.defineNamedConstructor(String, 'fromEnvironment');
-  class Runes extends collection.IterableBase$(int) {
-    Runes(string) {
-      this.string = string;
-      super.IterableBase();
+  dart.defineLazyClass(exports, {
+    get Runes() {
+      return class Runes extends collection.IterableBase$(int) {

[Siggi Cherem (dart-lang), 2015/03/18 23:54:55]

mmm... overall I find the generic-class pattern slightly easier to read than
this one, I think that might be because of the 'return' here.

Could we do the same pattern for this? Or move the return to a separate line?

get Runes() {
  class Runes ... {}
  return Runes();
}

[Jennifer Messerly, 2015/03/19 16:03:29]

Sure thing. It's actually easier to emit that way too :)

+        Runes(string) {
+          this.string = string;
+          super.IterableBase();
+        }
+        get iterator() {
+          return new RuneIterator(this.string);
+        }
+        get last() {
+          if (this.string.length === 0) {
+            throw new StateError('No elements.');
+          }
+          let length = this.string.length;
+          let code = this.string.codeUnitAt(dart.notNull(length) - 1);
+          if (dart.notNull(_isTrailSurrogate(code)) && dart.notNull(this.string.length) > 1) {
+            let previousCode = this.string.codeUnitAt(dart.notNull(length) - 2);
+            if (_isLeadSurrogate(previousCode)) {
+              return _combineSurrogatePair(previousCode, code);
+            }
+          }
+          return code;
+        }
+      };
     }
-    get iterator() {
-      return new RuneIterator(this.string);
-    }
-    get last() {
-      if (this.string.length === 0) {
-        throw new StateError('No elements.');
-      }
-      let length = this.string.length;
-      let code = this.string.codeUnitAt(dart.notNull(length) - 1);
-      if (dart.notNull(_isTrailSurrogate(code)) && dart.notNull(this.string.length) > 1) {
-        let previousCode = this.string.codeUnitAt(dart.notNull(length) - 2);
-        if (_isLeadSurrogate(previousCode)) {
-          return _combineSurrogatePair(previousCode, code);
-        }
-      }
-      return code;
-    }
-  }
+  });
   // Function _isLeadSurrogate: (int) → bool
   function _isLeadSurrogate(code) {
     return (dart.notNull(code) & 64512) === 55296;
   }
   // Function _isTrailSurrogate: (int) → bool
   function _isTrailSurrogate(code) {
     return (dart.notNull(code) & 64512) === 56320;
   }
   // Function _combineSurrogatePair: (int, int) → int
   function _combineSurrogatePair(start, end) {
@@ skipping 1501 matching lines @@
   // Function _symbolMapToStringMap: (Map<Symbol, dynamic>) → dynamic
   function _symbolMapToStringMap(map) {
     if (map === null)
       return null;
     let result = new Map();
     map.forEach((key, value) => {
       result.set(_symbolToString(key), value);
     });
     return result;
   }
-  class _ListConstructorSentinel extends _interceptors.JSInt {
-    _ListConstructorSentinel() {
-      super.JSInt();
+  dart.defineLazyClass(exports, {
+    get _ListConstructorSentinel() {
+      return class _ListConstructorSentinel extends _interceptors.JSInt {
+        _ListConstructorSentinel() {
+          super.JSInt();
+        }
+      };
     }
-  }
+  });
   // Exports:
   exports.Deprecated = Deprecated;
+  exports.Object = Object;
   exports.deprecated = deprecated;
   exports.override = override;
   exports.proxy = proxy;
   exports.bool = bool;
   exports.Comparable = Comparable;
   exports.Comparable$ = Comparable$;
   exports.DateTime = DateTime;
   exports.double = double;
+  exports.num = num;
   exports.Duration = Duration;
   exports.Error = Error;
   exports.AssertionError = AssertionError;
   exports.TypeError = TypeError;
   exports.CastError = CastError;
   exports.NullThrownError = NullThrownError;
   exports.ArgumentError = ArgumentError;
   exports.RangeError = RangeError;
   exports.IndexError = IndexError;
   exports.FallThroughError = FallThroughError;
@@ skipping 20 matching lines @@
   exports.Iterable$ = Iterable$;
   exports.BidirectionalIterator = BidirectionalIterator;
   exports.BidirectionalIterator$ = BidirectionalIterator$;
   exports.Iterator = Iterator;
   exports.Iterator$ = Iterator$;
   exports.List = List;
   exports.List$ = List$;
   exports.Map = Map;
   exports.Map$ = Map$;
   exports.Null = Null;
-  exports.num = num;
-  exports.Object = Object;
   exports.Pattern = Pattern;
   exports.print = print;
   exports.Match = Match;
   exports.RegExp = RegExp;
   exports.Set = Set;
   exports.Set$ = Set$;
   exports.Sink = Sink;
   exports.Sink$ = Sink$;
   exports.StackTrace = StackTrace;
   exports.Stopwatch = Stopwatch;
   exports.String = String;
   exports.Runes = Runes;
   exports.RuneIterator = RuneIterator;
   exports.StringBuffer = StringBuffer;
   exports.StringSink = StringSink;
   exports.Symbol = Symbol;
   exports.Type = Type;
   exports.Uri = Uri;
 })(core || (core = {}));