JAL-2286 use constant for fixed colours
authorgmungoc <g.m.carstairs@dundee.ac.uk>
Tue, 25 Oct 2016 15:35:51 +0000 (16:35 +0100)
committergmungoc <g.m.carstairs@dundee.ac.uk>
Tue, 25 Oct 2016 15:35:51 +0000 (16:35 +0100)
src/jalview/schemes/Blosum62ColourScheme.java

index 90f90f0..8cb6ff0 100755 (executable)
@@ -30,6 +30,9 @@ import java.util.Map;
 
 public class Blosum62ColourScheme extends ResidueColourScheme
 {
+  private static final Color LIGHT_BLUE = new Color(204, 204, 255);
+  private static final Color DARK_BLUE = new Color(154, 154, 255);
+
   public Blosum62ColourScheme()
   {
     super();
@@ -58,8 +61,7 @@ public class Blosum62ColourScheme extends ResidueColourScheme
 
       if (max.indexOf(res) > -1)
       {
-        // TODO use a constant here?
-        currentColour = new Color(154, 154, 255);
+        currentColour = DARK_BLUE;
       }
       else
       {
@@ -74,8 +76,7 @@ public class Blosum62ColourScheme extends ResidueColourScheme
 
         if (c > 0)
         {
-          // TODO use a constant here?
-          currentColour = new Color(204, 204, 255);
+          currentColour = LIGHT_BLUE;
         }
         else
         {