Null sequence check

[jalview.git] / src / jalview / gui / SequenceRenderer.java

diff --git a/src/jalview/gui/SequenceRenderer.java b/src/jalview/gui/SequenceRenderer.java
index 8e8b066..57cd0fc 100755 (executable)
--- a/src/jalview/gui/SequenceRenderer.java
+++ b/src/jalview/gui/SequenceRenderer.java
@@ -24,7 +24,6 @@ import jalview.schemes.*;
 \r
 import java.awt.*;\r
 \r
-\r
 /**\r
  * DOCUMENT ME!\r
  *\r
@@ -61,22 +60,27 @@ public class SequenceRenderer
         renderGaps = b;\r
     }\r
 \r
-    /**\r
-     * DOCUMENT ME!\r
-     *\r
-     * @param cs DOCUMENT ME!\r
-     * @param seq DOCUMENT ME!\r
-     * @param i DOCUMENT ME!\r
-     *\r
-     * @return DOCUMENT ME!\r
-     */\r
-    public Color getResidueBoxColour(ColourSchemeI cs, SequenceI seq, int i)\r
+\r
+    public Color getResidueBoxColour(SequenceI seq, int i)\r
     {\r
-        getBoxColour(cs, seq, i);\r
+      allGroups = av.alignment.findAllGroups(seq);\r
+\r
+      if (inCurrentSequenceGroup(i))\r
+      {\r
+        if (currentSequenceGroup.getDisplayBoxes())\r
+        {\r
+          getBoxColour(currentSequenceGroup.cs, seq, i);\r
+        }\r
+      }\r
+      else if (av.getShowBoxes())\r
+      {\r
+          getBoxColour(av.globalColourScheme, seq, i);\r
+      }\r
 \r
-        return resBoxColour;\r
+      return resBoxColour;\r
     }\r
 \r
+\r
     /**\r
      * DOCUMENT ME!\r
      *\r
@@ -133,61 +137,62 @@ public class SequenceRenderer
      * @param width DOCUMENT ME!\r
      * @param height DOCUMENT ME!\r
      */\r
-    public void drawBoxes(SequenceI seq, int start, int end, int x1, int y1,\r
+    public synchronized void drawBoxes(SequenceI seq, int start, int end, int x1, int y1,\r
         int width, int height)\r
     {\r
-        int i = start;\r
-        int length = seq.getLength();\r
+      int i = start;\r
+      int length = seq.getLength();\r
 \r
-        int curStart = -1;\r
-        int curWidth = width;\r
+      int curStart = -1;\r
+      int curWidth = width;\r
 \r
-        Color tempColour = null;\r
+      Color tempColour = null;\r
 \r
-        while (i <= end)\r
-        {\r
-          resBoxColour = Color.white;\r
+      while (i <= end)\r
+      {\r
+        resBoxColour = Color.white;\r
 \r
-          if (i < length)\r
+        if (i < length)\r
+        {\r
+          if (inCurrentSequenceGroup(i))\r
           {\r
-            if (inCurrentSequenceGroup(i))\r
+            if (currentSequenceGroup.getDisplayBoxes())\r
             {\r
-              if (currentSequenceGroup.getDisplayBoxes())\r
-              {\r
-                getBoxColour(currentSequenceGroup.cs, seq, i);\r
-              }\r
-            }\r
-            else if (av.getShowBoxes())\r
-            {\r
-              getBoxColour(av.getGlobalColourScheme(), seq, i);\r
+              getBoxColour(currentSequenceGroup.cs, seq, i);\r
             }\r
-\r
+          }\r
+          else if (av.getShowBoxes())\r
+          {\r
+            getBoxColour(av.globalColourScheme, seq, i);\r
           }\r
 \r
-            if (resBoxColour != tempColour)\r
-            {\r
-                if (tempColour != null)\r
-                {\r
-                    graphics.fillRect(x1 + (width * (curStart - start)), y1,\r
-                        curWidth, height);\r
-                }\r
+        }\r
 \r
-                graphics.setColor(resBoxColour);\r
+          if (resBoxColour != tempColour)\r
+          {\r
+              if (tempColour != null)\r
+              {\r
+                  graphics.fillRect(x1 + (width * (curStart - start)), y1,\r
+                      curWidth, height);\r
+              }\r
 \r
-                curStart = i;\r
-                curWidth = width;\r
-                tempColour = resBoxColour;\r
-            }\r
-            else\r
-            {\r
-                curWidth += width;\r
-            }\r
+              graphics.setColor(resBoxColour);\r
 \r
-            i++;\r
-        }\r
+              curStart = i;\r
+              curWidth = width;\r
+              tempColour = resBoxColour;\r
+          }\r
+          else\r
+          {\r
+              curWidth += width;\r
+          }\r
+\r
+          i++;\r
+      }\r
+\r
+      graphics.fillRect(x1 + (width * (curStart - start)), y1, curWidth,\r
+          height);\r
 \r
-        graphics.fillRect(x1 + (width * (curStart - start)), y1, curWidth,\r
-            height);\r
     }\r
 \r
     /**\r
@@ -208,13 +213,6 @@ public class SequenceRenderer
         int charOffset = 0;\r
         char s;\r
 \r
-        float fwidth = width + ((float)av.charWidth/(float)width)/width;\r
-\r
-\r
-        if(av.charWidth != (av.charWidth/width)*width)\r
-        {\r
-\r
-        }\r
 \r
         // Need to find the sequence position here.\r
         String sequence = seq.getSequence();\r
@@ -255,7 +253,7 @@ public class SequenceRenderer
 \r
                 if (av.getColourText())\r
                 {\r
-                    getBoxColour(av.getGlobalColourScheme(), seq, i);\r
+                    getBoxColour(av.globalColourScheme, seq, i);\r
 \r
                     if (av.getShowBoxes())\r
                     {\r
@@ -270,7 +268,8 @@ public class SequenceRenderer
 \r
             charOffset = (width - fm.charWidth(s)) / 2;\r
             graphics.drawString(String.valueOf(s),\r
-                charOffset + x1 + (int)(fwidth * (i - start)), (y1 + height) - pady);\r
+                charOffset + x1 + (int)(width * (i - start)), (y1 + height) - pady);\r
+\r
         }\r
     }\r
 \r