Rather than put things in a HashSet and then use the "sorted iterator", which

really copies everything to an ArrayList and sorts it,
let's just throw the elements into a TreeSet and iterate over that.


git-svn-id: svn+ssh://src.earth.threerings.net/vilya/trunk@942 c613c5cb-e716-0410-b11b-feb51c14d237

src/java/com/threerings/stage/tools/editor/SceneInfoPanel.java

@@ -21,8 +21,7 @@
 
 package com.threerings.stage.tools.editor;
 
-import java.util.HashSet;
+import java.util.TreeSet;
-import java.util.Iterator;
 
 import java.awt.event.ActionEvent;
 import java.awt.event.ActionListener;
@@ -40,7 +39,6 @@ import javax.swing.event.PopupMenuListener;
 
 import com.google.common.collect.Sets;
 
-import com.samskivert.util.Collections;
 import com.samskivert.util.ComparableArrayList;
 
 import com.samskivert.swing.GroupLayout;
@@ -173,7 +171,7 @@ public class SceneInfoPanel extends JPanel
     {
         // add all possible colorization names to the list
         final TileManager tilemgr = _ctx.getTileManager();
-        final HashSet<String> set = Sets.newHashSet();
+        final TreeSet<String> set = Sets.newTreeSet();
         StageMisoSceneModel msmodel = StageMisoSceneModel.getSceneModel(
             _scene.getSceneModel());
         msmodel.visitObjects(new ObjectVisitor() {
@@ -203,9 +201,8 @@ public class SceneInfoPanel extends JPanel
         DefaultComboBoxModel model =
             (DefaultComboBoxModel) _colorClasses.getModel();
         model.removeAllElements();
-        for (Iterator<String> itr = Collections.getSortedIterator(set);
+        for (String zation : set) {
-                itr.hasNext(); ) {
+            model.addElement(zation);
-            model.addElement(itr.next());
         }
         if (selected != null) {
             _colorClasses.setSelectedItem(selected);