Formatting
[jalview.git] / src / jalview / appletgui / FeatureRenderer.java
index 47fbb0d..7ad5f4d 100755 (executable)
@@ -1,7 +1,6 @@
-\r
 /*\r
  * Jalview - A Sequence Alignment Editor and Viewer\r
- * Copyright (C) 2006 AM Waterhouse, J Procter, G Barton, M Clamp, S Searle\r
+ * Copyright (C) 2007 AM Waterhouse, J Procter, G Barton, M Clamp, S Searle\r
  *\r
  * This program is free software; you can redistribute it and/or\r
  * modify it under the terms of the GNU General Public License\r
  */\r
 package jalview.appletgui;\r
 \r
-import jalview.datamodel.*;\r
+import java.util.*;\r
 \r
 import java.awt.*;\r
 \r
-import java.util.*;\r
+import jalview.datamodel.*;\r
 \r
 /**\r
  * DOCUMENT ME!\r
@@ -33,392 +32,426 @@ import java.util.*;
  */\r
 public class FeatureRenderer\r
 {\r
-    AlignViewport av;\r
+  AlignViewport av;\r
 \r
-    Hashtable featureColours = new Hashtable();\r
+  Hashtable featureColours = new Hashtable();\r
 \r
-    // A higher level for grouping features of a\r
-    // particular type\r
-    Hashtable featureGroups = null;\r
+  // A higher level for grouping features of a\r
+  // particular type\r
+  Hashtable featureGroups = null;\r
 \r
-    // Holds web links for feature groups and feature types\r
-    // in the form label|link\r
-    Hashtable featureLinks = null;\r
+  // Holds web links for feature groups and feature types\r
+  // in the form label|link\r
+  Hashtable featureLinks = null;\r
 \r
+  // This is actually an Integer held in the hashtable,\r
+  // Retrieved using the key feature type\r
+  Object currentColour;\r
 \r
-    // This is actually an Integer held in the hashtable,\r
-    // Retrieved using the key feature type\r
-    Object currentColour;\r
+  String[] renderOrder;\r
 \r
-    String [] renderOrder;\r
+  FontMetrics fm;\r
+  int charOffset;\r
 \r
-    FontMetrics fm;\r
-    int charOffset;\r
+  float transparency = 1f;\r
 \r
-    float transparency = 1f;\r
+  TransparencySetter transparencySetter = null;\r
 \r
-    TransparencySetter transparencySetter = null;\r
+  /**\r
+   * Creates a new FeatureRenderer object.\r
+   *\r
+   * @param av DOCUMENT ME!\r
+   */\r
+  public FeatureRenderer(AlignViewport av)\r
+  {\r
+    this.av = av;\r
 \r
-    /**\r
-     * Creates a new FeatureRenderer object.\r
-     *\r
-     * @param av DOCUMENT ME!\r
-     */\r
-    public FeatureRenderer(AlignViewport av)\r
+    if (!System.getProperty("java.version").startsWith("1.1"))\r
     {\r
-        this.av = av;\r
-\r
-        if(!System.getProperty("java.version").startsWith("1.1"))\r
-             transparencySetter = new TransparencySetter();\r
+      transparencySetter = new TransparencySetter();\r
     }\r
+  }\r
 \r
+  public void transferSettings(FeatureRenderer fr)\r
+  {\r
+    renderOrder = fr.renderOrder;\r
+    featureGroups = fr.featureGroups;\r
+    featureColours = fr.featureColours;\r
+    transparency = fr.transparency;\r
+  }\r
 \r
-    public void transferSettings(FeatureRenderer fr)\r
+  public Color findFeatureColour(Color initialCol, SequenceI seq, int i)\r
+  {\r
+    overview = true;\r
+    if (!av.showSequenceFeatures)\r
     {\r
-      renderOrder = fr.renderOrder;\r
-      featureGroups = fr.featureGroups;\r
-      featureColours = fr.featureColours;\r
-      transparency =  fr.transparency;\r
+      return initialCol;\r
     }\r
 \r
-\r
-    public Color findFeatureColour(Color initialCol, SequenceI seq, int i)\r
+    lastSequence = seq;\r
+    sequenceFeatures = lastSequence.getSequenceFeatures();\r
+    if (sequenceFeatures == null)\r
     {\r
-      overview = true;\r
-      if(!av.showSequenceFeatures)\r
-        return initialCol;\r
-\r
-        lastSequence = seq;\r
-        sequenceFeatures = lastSequence.getSequenceFeatures();\r
-        if(sequenceFeatures==null)\r
-          return initialCol;\r
-\r
-        sfSize = sequenceFeatures.length;\r
+      return initialCol;\r
+    }\r
 \r
-      if(jalview.util.Comparison.isGap(lastSequence.getCharAt(i)))\r
-        return Color.white;\r
+    sfSize = sequenceFeatures.length;\r
 \r
-      currentColour = null;\r
+    if (jalview.util.Comparison.isGap(lastSequence.getCharAt(i)))\r
+    {\r
+      return Color.white;\r
+    }\r
 \r
-      drawSequence(null, lastSequence, lastSequence.findPosition(i), -1,-1);\r
+    currentColour = null;\r
 \r
-      if(currentColour==null)\r
-        return initialCol;\r
+    drawSequence(null, lastSequence, lastSequence.findPosition(i), -1, -1);\r
 \r
-      return new Color( ((Integer)currentColour).intValue() );\r
+    if (currentColour == null)\r
+    {\r
+      return initialCol;\r
     }\r
 \r
-    /**\r
-     * This is used by the Molecule Viewer to get the accurate colour\r
-     * of the rendered sequence\r
-     */\r
-    boolean overview = false;\r
+    return new Color( ( (Integer) currentColour).intValue());\r
+  }\r
+\r
+  /**\r
+   * This is used by the Molecule Viewer to get the accurate colour\r
+   * of the rendered sequence\r
+   */\r
+  boolean overview = false;\r
 \r
-    int white = Color.white.getRGB();\r
-    public int findFeatureColour(int initialCol, int seqIndex, int column)\r
+  int white = Color.white.getRGB();\r
+  public int findFeatureColour(int initialCol, int seqIndex, int column)\r
+  {\r
+    if (!av.showSequenceFeatures)\r
     {\r
-      if(!av.showSequenceFeatures)\r
-        return initialCol;\r
+      return initialCol;\r
+    }\r
 \r
-      if(seqIndex!=lastSequenceIndex)\r
+    if (seqIndex != lastSequenceIndex)\r
+    {\r
+      lastSequence = av.alignment.getSequenceAt(seqIndex);\r
+      lastSequenceIndex = seqIndex;\r
+      sequenceFeatures = lastSequence.getSequenceFeatures();\r
+      if (sequenceFeatures == null)\r
       {\r
-        lastSequence = av.alignment.getSequenceAt(seqIndex);\r
-        lastSequenceIndex = seqIndex;\r
-        sequenceFeatures = lastSequence.getSequenceFeatures();\r
-        if(sequenceFeatures==null)\r
-          return initialCol;\r
-\r
-        sfSize = sequenceFeatures.length;\r
+        return initialCol;\r
       }\r
 \r
+      sfSize = sequenceFeatures.length;\r
+    }\r
 \r
-      if(jalview.util.Comparison.isGap(lastSequence.getCharAt(column)))\r
-        return Color.white.getRGB();\r
-\r
-      currentColour = null;\r
+    if (jalview.util.Comparison.isGap(lastSequence.getCharAt(column)))\r
+    {\r
+      return Color.white.getRGB();\r
+    }\r
 \r
-      drawSequence(null, lastSequence, lastSequence.findPosition(column), -1,-1);\r
+    currentColour = null;\r
 \r
-      if(currentColour==null)\r
-        return initialCol;\r
+    drawSequence(null, lastSequence, lastSequence.findPosition(column), -1, -1);\r
 \r
-      return  ((Integer)currentColour).intValue();\r
+    if (currentColour == null)\r
+    {\r
+      return initialCol;\r
     }\r
 \r
+    return ( (Integer) currentColour).intValue();\r
+  }\r
 \r
-    /**\r
-     * DOCUMENT ME!\r
-     *\r
-     * @param g DOCUMENT ME!\r
-     * @param seq DOCUMENT ME!\r
-     * @param sg DOCUMENT ME!\r
-     * @param start DOCUMENT ME!\r
-     * @param end DOCUMENT ME!\r
-     * @param x1 DOCUMENT ME!\r
-     * @param y1 DOCUMENT ME!\r
-     * @param width DOCUMENT ME!\r
-     * @param height DOCUMENT ME!\r
-     */\r
-   // String type;\r
-   // SequenceFeature sf;\r
-    int lastSequenceIndex=-1;\r
-    SequenceI lastSequence;\r
-    SequenceFeature [] sequenceFeatures;\r
-    int sfSize, sfindex, spos, epos;\r
-\r
-    public void drawSequence(Graphics g, SequenceI seq,\r
-                             int start, int end, int y1)\r
+  /**\r
+   * DOCUMENT ME!\r
+   *\r
+   * @param g DOCUMENT ME!\r
+   * @param seq DOCUMENT ME!\r
+   * @param sg DOCUMENT ME!\r
+   * @param start DOCUMENT ME!\r
+   * @param end DOCUMENT ME!\r
+   * @param x1 DOCUMENT ME!\r
+   * @param y1 DOCUMENT ME!\r
+   * @param width DOCUMENT ME!\r
+   * @param height DOCUMENT ME!\r
+   */\r
+  // String type;\r
+  // SequenceFeature sf;\r
+  int lastSequenceIndex = -1;\r
+  SequenceI lastSequence;\r
+  SequenceFeature[] sequenceFeatures;\r
+  int sfSize, sfindex, spos, epos;\r
+\r
+  public void drawSequence(Graphics g, SequenceI seq,\r
+                           int start, int end, int y1)\r
+  {\r
+    if (seq.getSequenceFeatures() == null\r
+        || seq.getSequenceFeatures().length == 0)\r
     {\r
-      if (   seq.getSequenceFeatures() == null\r
-          || seq.getSequenceFeatures().length==0)\r
-        return;\r
+      return;\r
+    }\r
 \r
-      if(transparencySetter!=null && g!=null)\r
+    if (transparencySetter != null && g != null)\r
+    {\r
+      transparencySetter.setTransparency(g, transparency);\r
+    }\r
+\r
+    if (av.featuresDisplayed == null || renderOrder == null)\r
+    {\r
+      findAllFeatures();\r
+      if (av.featuresDisplayed.size() < 1)\r
       {\r
-        transparencySetter.setTransparency(g, transparency);\r
+        return;\r
       }\r
 \r
-      if (av.featuresDisplayed == null || renderOrder==null)\r
-       {\r
-         findAllFeatures();\r
-         if(av.featuresDisplayed.size()<1)\r
-           return;\r
-\r
-         sequenceFeatures = seq.getSequenceFeatures();\r
-         sfSize = sequenceFeatures.length;\r
-       }\r
-       if(lastSequence==null || seq!=lastSequence)\r
+      sequenceFeatures = seq.getSequenceFeatures();\r
+      sfSize = sequenceFeatures.length;\r
+    }\r
+    if (lastSequence == null || seq != lastSequence)\r
+    {\r
+      lastSequence = seq;\r
+      sequenceFeatures = seq.getSequenceFeatures();\r
+      sfSize = sequenceFeatures.length;\r
+    }\r
+    if (!overview)\r
+    {\r
+      spos = lastSequence.findPosition(start);\r
+      epos = lastSequence.findPosition(end);\r
+      if (g != null)\r
       {\r
-        lastSequence = seq;\r
-        sequenceFeatures = seq.getSequenceFeatures();\r
-        sfSize = sequenceFeatures.length;\r
+        fm = g.getFontMetrics();\r
       }\r
-      if(!overview)\r
+    }\r
+    String type;\r
+    for (int renderIndex = 0; renderIndex < renderOrder.length; renderIndex++)\r
+    {\r
+      type = renderOrder[renderIndex];\r
+      if (!av.featuresDisplayed.containsKey(type))\r
       {\r
-        spos = lastSequence.findPosition(start);\r
-        epos = lastSequence.findPosition(end);\r
-        if(g!=null)\r
-          fm = g.getFontMetrics();\r
+        continue;\r
       }\r
-      String type;\r
-      for(int renderIndex=0; renderIndex<renderOrder.length; renderIndex++)\r
-       {\r
-        type =  renderOrder[renderIndex];\r
-        if(!av.featuresDisplayed.containsKey(type))\r
+\r
+      // loop through all features in sequence to find\r
+      // current feature to render\r
+      for (sfindex = 0; sfindex < sfSize; sfindex++)\r
+      {\r
+        if (!sequenceFeatures[sfindex].type.equals(type))\r
+        {\r
           continue;\r
+        }\r
 \r
-        // loop through all features in sequence to find\r
-        // current feature to render\r
-        for (sfindex = 0; sfindex < sfSize; sfindex++)\r
+        if (featureGroups != null\r
+            && sequenceFeatures[sfindex].featureGroup != null\r
+            &&\r
+            featureGroups.containsKey(sequenceFeatures[sfindex].featureGroup)\r
+            &&\r
+            ! ( (Boolean) featureGroups.get(sequenceFeatures[sfindex].\r
+                                            featureGroup)).\r
+            booleanValue())\r
         {\r
-          if (!sequenceFeatures[sfindex].type.equals(type))\r
-            continue;\r
-\r
-          if (featureGroups != null\r
-              && sequenceFeatures[sfindex].featureGroup != null\r
-              &&\r
-              featureGroups.containsKey(sequenceFeatures[sfindex].featureGroup)\r
-              &&\r
-              ! ( (Boolean) featureGroups.get(sequenceFeatures[sfindex].featureGroup)).\r
-              booleanValue())\r
-          {\r
-            continue;\r
-          }\r
+          continue;\r
+        }\r
 \r
-          if (!overview && (sequenceFeatures[sfindex].getBegin() > epos\r
-                            || sequenceFeatures[sfindex].getEnd() < spos))\r
-            continue;\r
+        if (!overview && (sequenceFeatures[sfindex].getBegin() > epos\r
+                          || sequenceFeatures[sfindex].getEnd() < spos))\r
+        {\r
+          continue;\r
+        }\r
 \r
-          if (overview)\r
+        if (overview)\r
+        {\r
+          if (sequenceFeatures[sfindex].begin <= start &&\r
+              sequenceFeatures[sfindex].end >= start)\r
           {\r
-            if (sequenceFeatures[sfindex].begin <= start &&\r
-                sequenceFeatures[sfindex].end >= start)\r
-            {\r
-              currentColour = av.featuresDisplayed.get(sequenceFeatures[sfindex].\r
-                  type);\r
-            }\r
-\r
+            currentColour = av.featuresDisplayed.get(sequenceFeatures[sfindex].\r
+                type);\r
           }\r
-          else if (sequenceFeatures[sfindex].type.equals("disulfide bond"))\r
-          {\r
 \r
-            renderFeature(g, seq,\r
-                          seq.findIndex(sequenceFeatures[sfindex].begin) - 1,\r
-                          seq.findIndex(sequenceFeatures[sfindex].begin) - 1,\r
-                          new Color( ( (Integer) av.featuresDisplayed.get(\r
-                sequenceFeatures[sfindex].type)).intValue()),\r
-                          start, end, y1);\r
-            renderFeature(g, seq,\r
-                          seq.findIndex(sequenceFeatures[sfindex].end) - 1,\r
-                          seq.findIndex(sequenceFeatures[sfindex].end) - 1,\r
-                          new Color( ( (Integer) av.featuresDisplayed.get(\r
-                sequenceFeatures[sfindex].type)).intValue()),\r
-                          start, end, y1);\r
+        }\r
+        else if (sequenceFeatures[sfindex].type.equals("disulfide bond"))\r
+        {\r
 \r
-          }\r
-          else\r
-            renderFeature(g, seq,\r
-                          seq.findIndex(sequenceFeatures[sfindex].begin) - 1,\r
-                          seq.findIndex(sequenceFeatures[sfindex].end) - 1,\r
-                          getColour(sequenceFeatures[sfindex].type),\r
-                          start, end, y1);\r
+          renderFeature(g, seq,\r
+                        seq.findIndex(sequenceFeatures[sfindex].begin) - 1,\r
+                        seq.findIndex(sequenceFeatures[sfindex].begin) - 1,\r
+                        new Color( ( (Integer) av.featuresDisplayed.get(\r
+                            sequenceFeatures[sfindex].type)).intValue()),\r
+                        start, end, y1);\r
+          renderFeature(g, seq,\r
+                        seq.findIndex(sequenceFeatures[sfindex].end) - 1,\r
+                        seq.findIndex(sequenceFeatures[sfindex].end) - 1,\r
+                        new Color( ( (Integer) av.featuresDisplayed.get(\r
+                            sequenceFeatures[sfindex].type)).intValue()),\r
+                        start, end, y1);\r
 \r
         }\r
-      }\r
+        else\r
+        {\r
+          renderFeature(g, seq,\r
+                        seq.findIndex(sequenceFeatures[sfindex].begin) - 1,\r
+                        seq.findIndex(sequenceFeatures[sfindex].end) - 1,\r
+                        getColour(sequenceFeatures[sfindex].type),\r
+                        start, end, y1);\r
+        }\r
 \r
-      if(transparencySetter!=null && g!=null)\r
-      {\r
-        transparencySetter.setTransparency(g, 1.0f);\r
       }\r
     }\r
 \r
+    if (transparencySetter != null && g != null)\r
+    {\r
+      transparencySetter.setTransparency(g, 1.0f);\r
+    }\r
+  }\r
+\r
+  char s;\r
+  int i;\r
+  void renderFeature(Graphics g, SequenceI seq,\r
+                     int fstart, int fend, Color featureColour, int start,\r
+                     int end, int y1)\r
+  {\r
 \r
-    char s;\r
-    int i;\r
-    void renderFeature(Graphics g, SequenceI seq,\r
-                       int fstart, int fend, Color featureColour, int start, int end,  int y1)\r
+    if ( ( (fstart <= end) && (fend >= start)))\r
     {\r
+      if (fstart < start)\r
+      { // fix for if the feature we have starts before the sequence start,\r
+        fstart = start; // but the feature end is still valid!!\r
+      }\r
 \r
-      if (((fstart <= end) && (fend >= start)))\r
+      if (fend >= end)\r
       {\r
-        if (fstart < start)\r
-        { // fix for if the feature we have starts before the sequence start,\r
-          fstart = start; // but the feature end is still valid!!\r
-        }\r
+        fend = end;\r
+      }\r
+\r
+      for (i = fstart; i <= fend; i++)\r
+      {\r
+        s = seq.getCharAt(i);\r
 \r
-        if (fend >= end)\r
+        if (jalview.util.Comparison.isGap(s))\r
         {\r
-          fend = end;\r
+          continue;\r
         }\r
 \r
-          for (i = fstart; i <= fend; i++)\r
-          {\r
-            s = seq.getCharAt(i);\r
-\r
-            if (jalview.util.Comparison.isGap(s))\r
-            {\r
-              continue;\r
-            }\r
+        g.setColor(featureColour);\r
 \r
-            g.setColor(featureColour);\r
+        g.fillRect( (i - start) * av.charWidth, y1, av.charWidth, av.charHeight);\r
 \r
-            g.fillRect( (i - start) * av.charWidth, y1, av.charWidth, av.charHeight);\r
-\r
-            if(!av.validCharWidth)\r
-              continue;\r
+        if (!av.validCharWidth)\r
+        {\r
+          continue;\r
+        }\r
 \r
-            g.setColor(Color.white);\r
-            charOffset = (av.charWidth - fm.charWidth(s)) / 2;\r
-            g.drawString(String.valueOf(s),\r
-                         charOffset + (av.charWidth * (i - start)),\r
-                         (y1 + av.charHeight) - av.charHeight / 5); //pady = height / 5;\r
+        g.setColor(Color.white);\r
+        charOffset = (av.charWidth - fm.charWidth(s)) / 2;\r
+        g.drawString(String.valueOf(s),\r
+                     charOffset + (av.charWidth * (i - start)),\r
+                     (y1 + av.charHeight) - av.charHeight / 5); //pady = height / 5;\r
 \r
-          }\r
       }\r
     }\r
+  }\r
+\r
+  void findAllFeatures()\r
+  {\r
+    jalview.schemes.UserColourScheme ucs = new\r
+        jalview.schemes.UserColourScheme();\r
 \r
-    void findAllFeatures()\r
+    av.featuresDisplayed = new Hashtable();\r
+    Vector allfeatures = new Vector();\r
+    for (int i = 0; i < av.alignment.getHeight(); i++)\r
     {\r
-      jalview.schemes.UserColourScheme ucs = new\r
-          jalview.schemes.UserColourScheme();\r
+      SequenceFeature[] features = av.alignment.getSequenceAt(i).\r
+          getSequenceFeatures();\r
 \r
-      av.featuresDisplayed = new Hashtable();\r
-      Vector allfeatures = new Vector();\r
-      for (int i = 0; i < av.alignment.getHeight(); i++)\r
+      if (features == null)\r
       {\r
-        SequenceFeature [] features = av.alignment.getSequenceAt(i).getSequenceFeatures();\r
-\r
-        if (features == null)\r
-          continue;\r
+        continue;\r
+      }\r
 \r
-        int index = 0;\r
-        while (index < features.length)\r
+      int index = 0;\r
+      while (index < features.length)\r
+      {\r
+        if (!av.featuresDisplayed.containsKey(features[index].getType()))\r
         {\r
-          if (!av.featuresDisplayed.containsKey(features[index].getType()))\r
+          if (getColour(features[index].getType()) == null)\r
           {\r
-            if (getColour(features[index].getType()) == null)\r
-            {\r
-              featureColours.put(features[index].getType(),\r
-                                 ucs.createColourFromName(features[index].\r
-                  getType()));\r
-            }\r
-\r
-\r
-            av.featuresDisplayed.put(features[index].getType(),\r
-                                  new Integer( getColour(features[index].getType()).getRGB()) );\r
-            allfeatures.addElement(features[index].getType());\r
+            featureColours.put(features[index].getType(),\r
+                               ucs.createColourFromName(features[index].\r
+                getType()));\r
           }\r
-          index++;\r
-        }\r
-      }\r
 \r
-      renderOrder = new String[allfeatures.size()];\r
-      Enumeration en = allfeatures.elements();\r
-      int i = allfeatures.size()-1;\r
-      while(en.hasMoreElements())\r
-      {\r
-        renderOrder[i] = en.nextElement().toString();\r
-        i--;\r
+          av.featuresDisplayed.put(features[index].getType(),\r
+                                   new Integer(getColour(features[index].\r
+              getType()).getRGB()));\r
+          allfeatures.addElement(features[index].getType());\r
+        }\r
+        index++;\r
       }\r
     }\r
 \r
-    public Color getColour(String featureType)\r
+    renderOrder = new String[allfeatures.size()];\r
+    Enumeration en = allfeatures.elements();\r
+    int i = allfeatures.size() - 1;\r
+    while (en.hasMoreElements())\r
     {\r
-      return (Color)featureColours.get(featureType);\r
+      renderOrder[i] = en.nextElement().toString();\r
+      i--;\r
     }\r
+  }\r
 \r
-    public void addNewFeature(String name, Color col)\r
-    {\r
-\r
-      setColour(name, col);\r
-      if(av.featuresDisplayed==null)\r
-        av.featuresDisplayed = new Hashtable();\r
+  public Color getColour(String featureType)\r
+  {\r
+    return (Color) featureColours.get(featureType);\r
+  }\r
 \r
+  public void addNewFeature(String name, Color col)\r
+  {\r
 \r
-      av.featuresDisplayed.put(name, "NOGROUP");\r
+    setColour(name, col);\r
+    if (av.featuresDisplayed == null)\r
+    {\r
+      av.featuresDisplayed = new Hashtable();\r
     }\r
 \r
-    public void setColour(String featureType, Color col)\r
+    av.featuresDisplayed.put(name, "NOGROUP");\r
+  }\r
+\r
+  public void setColour(String featureType, Color col)\r
+  {\r
+    featureColours.put(featureType, col);\r
+  }\r
+\r
+  public void setFeaturePriority(Object[][] data)\r
+  {\r
+    // The feature table will display high priority\r
+    // features at the top, but theses are the ones\r
+    // we need to render last, so invert the data\r
+    if (av.featuresDisplayed != null)\r
     {\r
-      featureColours.put(featureType, col);\r
+      av.featuresDisplayed.clear();\r
     }\r
 \r
-    public void setFeaturePriority(Object [][] data)\r
-    {\r
-        // The feature table will display high priority\r
-        // features at the top, but theses are the ones\r
-        // we need to render last, so invert the data\r
-        if(av.featuresDisplayed!=null)\r
-          av.featuresDisplayed.clear();\r
+    renderOrder = new String[data.length];\r
 \r
-        renderOrder = new String[data.length];\r
+    if (data.length > 0)\r
+    {\r
+      for (int i = 0; i < data.length; i++)\r
+      {\r
+        String type = data[i][0].toString();\r
+        setColour(type, (Color) data[i][1]);\r
+        if ( ( (Boolean) data[i][2]).booleanValue())\r
+        {\r
+          av.featuresDisplayed.put(type, new Integer(getColour(type).getRGB()));\r
+        }\r
 \r
-        if (data.length > 0)\r
-          for (int i = 0; i < data.length; i++)\r
-          {\r
-            String type = data[i][0].toString();\r
-            setColour(type, (Color) data[i][1]);\r
-            if ( ( (Boolean) data[i][2]).booleanValue())\r
-            {\r
-              av.featuresDisplayed.put(type, new Integer(getColour(type).getRGB()));\r
-            }\r
-\r
-            renderOrder[data.length - i - 1] = type;\r
-          }\r
+        renderOrder[data.length - i - 1] = type;\r
+      }\r
     }\r
+  }\r
 }\r
 \r
-  class TransparencySetter\r
+class TransparencySetter\r
+{\r
+  void setTransparency(Graphics g, float value)\r
   {\r
-    void setTransparency(Graphics g, float value)\r
-    {\r
     /* Graphics2D g2 = (Graphics2D) g;\r
      g2.setComposite(\r
         AlphaComposite.getInstance(\r
              AlphaComposite.SRC_OVER, value));*/\r
-    }\r
   }\r
-\r
-\r
-\r
-\r
+}\r