Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(9)

Unified Diff: editor/tools/plugins/com.google.dart.engine.services/src/com/google/dart/engine/services/correction/MembersSorter.java

Issue 961443004: Issue 21978. Put function 'main' first when sorting members. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: editor/tools/plugins/com.google.dart.engine.services/src/com/google/dart/engine/services/correction/MembersSorter.java
diff --git a/editor/tools/plugins/com.google.dart.engine.services/src/com/google/dart/engine/services/correction/MembersSorter.java b/editor/tools/plugins/com.google.dart.engine.services/src/com/google/dart/engine/services/correction/MembersSorter.java
index c176f1db56735616fd5d2b4aefcad3605a72e15b..c220bdbe3d7d69b742c71cda3fb11ee5f8d9626f 100644
--- a/editor/tools/plugins/com.google.dart.engine.services/src/com/google/dart/engine/services/correction/MembersSorter.java
+++ b/editor/tools/plugins/com.google.dart.engine.services/src/com/google/dart/engine/services/correction/MembersSorter.java
@@ -111,6 +111,7 @@ public class MembersSorter {
}
private static enum MemberKind {
+ UNIT_FUNCTION_MAIN,
UNIT_ACCESSOR,
UNIT_FUNCTION,
UNIT_FUNCTION_TYPE,
@@ -151,6 +152,7 @@ public class MembersSorter {
private static final PriorityItem[] PRIORITY_ITEMS = {
// unit
+ new PriorityItem(false, MemberKind.UNIT_FUNCTION_MAIN, false),
new PriorityItem(false, MemberKind.UNIT_VARIABLE, false),
new PriorityItem(false, MemberKind.UNIT_VARIABLE, true),
new PriorityItem(false, MemberKind.UNIT_ACCESSOR, false),
@@ -445,7 +447,11 @@ public class MembersSorter {
kind = MemberKind.UNIT_ACCESSOR;
name += " setter";
} else {
- kind = MemberKind.UNIT_FUNCTION;
+ if (name.equals("main")) {
+ kind = MemberKind.UNIT_FUNCTION_MAIN;
+ } else {
+ kind = MemberKind.UNIT_FUNCTION;
+ }
}
}
if (member instanceof FunctionTypeAlias) {
« no previous file with comments | « no previous file | editor/tools/plugins/com.google.dart.engine.services_test/src/com/google/dart/engine/services/internal/correction/MembersSorterTest.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698