Sort the output (for now just the classes and instance members).

lib/compiler/implementation/emitter.dart

Index: lib/compiler/implementation/emitter.dart
diff --git a/lib/compiler/implementation/emitter.dart b/lib/compiler/implementation/emitter.dart
index dda702ae1b39e367d535a9e8d545d9378d995eb1..a89d44a4ac91eb8a2504fe855f2b321ab10a3b59 100644
--- a/lib/compiler/implementation/emitter.dart
+++ b/lib/compiler/implementation/emitter.dart
@@ -426,12 +426,19 @@ function() {
       buffer.add(' $name: $value,\n');
     }
 
+    List<Element> instanceMembers = <Element>[];
     classElement.forEachMember(includeBackendMembers: true,
                                f: (ClassElement enclosing, Element member) {
       if (member.isInstanceMember()) {
-        addInstanceMember(member, defineInstanceMember);
+        instanceMembers.add(member);
       }
     });
+    instanceMembers.sort((Element member1, Element member2) {
+      return member1.compareTo(member2);

[ngeoffray, 2012/05/08 07:58:02]

Should the list already be sorted? After all, these elements are stored in the
ClassElement, and we should build that list sorted when creating the
ClassElement.

[floitsch, 2012/05/09 10:55:07]

Removed sorting of instance members.

+    });
+    for (Element member in instanceMembers) {
+      addInstanceMember(member, defineInstanceMember);
+    }
 
     generateTypeTests(classElement, (Element other) {
       if (nativeEmitter.requiresNativeIsCheck(other)) {
@@ -487,7 +494,12 @@ function() {
         neededClasses.add(superclass);
       }
     }
-    for (ClassElement element in neededClasses) {
+    List<ClassElement> sortedClasses =
+        new List<ClassElement>.from(neededClasses);
+    sortedClasses.sort((ClassElement class1, ClassElement class2) {

[ngeoffray, 2012/05/08 07:58:02]

I guess this one is hard to sort eagerly. How about assigning an index to the
ClassElement when it's created, so the order is only dependent on the way we
scan files?

[floitsch, 2012/05/09 10:55:07]

Done.

+      return class1.compareTo(class2);
+    });
+    for (ClassElement element in sortedClasses) {
       generateClass(element, buffer);
     }
 
@@ -570,7 +582,9 @@ function() {
     // The closure class.
     SourceString name = const SourceString("BoundClosure");
     ClassElement closureClassElement =
-        new ClosureClassElement(compiler, member.getCompilationUnit());
+        new ClosureClassElement(compiler,
+                                member.getCompilationUnit(),
+                                member.position());

[ngeoffray, 2012/05/08 07:58:02]

Maybe a FunctionElement should contain a list of closure elements?

[floitsch, 2012/05/09 10:55:07]

change reverted.

     String mangledName = namer.getName(closureClassElement);
     String superName = namer.getName(closureClassElement.superclass);
     needsClosureClass = true;