JAL-1114 - refactor methods handling Vectors and Hashtables to Lists and Maps, and...

[jalview.git] / src / jalview / appletgui / SliderPanel.java

diff --git a/src/jalview/appletgui/SliderPanel.java b/src/jalview/appletgui/SliderPanel.java
index f7522be..2046df6 100755 (executable)
--- a/src/jalview/appletgui/SliderPanel.java
+++ b/src/jalview/appletgui/SliderPanel.java
@@ -18,6 +18,7 @@
 package jalview.appletgui;
 
 import java.util.*;
+import java.util.List;
 
 import java.awt.*;
 import java.awt.event.*;
@@ -195,24 +196,22 @@ public class SliderPanel extends Panel implements ActionListener,
     }
 
     ColourSchemeI toChange = null;
-    Vector allGroups = null;
-    int groupIndex = 0;
+    Iterator<SequenceGroup> allGroups = null;
 
     if (allGroupsCheck.getState())
     {
-      allGroups = ap.av.getAlignment().getGroups();
-      groupIndex = allGroups.size() - 1;
+      allGroups = ap.av.getAlignment().getGroups().listIterator();
     }
     else
     {
       toChange = cs;
     }
 
-    while (groupIndex > -1)
+    do 
     {
       if (allGroups != null)
       {
-        toChange = ((SequenceGroup) allGroups.elementAt(groupIndex)).cs;
+        toChange = allGroups.next().cs;
       }
 
       if (forConservation)
@@ -224,8 +223,7 @@ public class SliderPanel extends Panel implements ActionListener,
         toChange.setThreshold(i, ap.av.getIgnoreGapsConsensus());
       }
 
-      groupIndex--;
-    }
+    } while (allGroups!=null && allGroups.hasNext());
 
     ap.seqPanel.seqCanvas.repaint();