Revert "JAL-2110 use shared alignment dataset when copying alignment for split frame"

[jalview.git] / src / jalview / analysis / Conservation.java

diff --git a/src/jalview/analysis/Conservation.java b/src/jalview/analysis/Conservation.java
index b8f1d92..7b3ce25 100755 (executable)
--- a/src/jalview/analysis/Conservation.java
+++ b/src/jalview/analysis/Conservation.java
@@ -235,10 +235,7 @@ public class Conservation
               c = '-';
             }
 
-            if (!canonicaliseAa && 'a' <= c && c <= 'z')
-            {
-              c -= (32); // 32 = 'a' - 'A'
-            }
+            c = toUpperCase(c);
           }
           values[c]++;
         }
@@ -280,8 +277,7 @@ public class Conservation
                 resultHash.put(type, ht.get("-"));
               }
             }
-            else if (((Integer) resultHash.get(type)).equals(ht
-                    .get(res)) == false)
+            else if (((Integer) resultHash.get(type)).equals(ht.get(res)) == false)
             {
               resultHash.put(type, new Integer(-1));
             }
@@ -327,6 +323,7 @@ public class Conservation
       }
       else
       {
+        c = toUpperCase(c);
         nres++;
 
         if (nres == 1)
@@ -348,6 +345,22 @@ public class Conservation
   }
 
   /**
+   * Returns the upper-cased character if between 'a' and 'z', else the
+   * unchanged value
+   * 
+   * @param c
+   * @return
+   */
+  char toUpperCase(char c)
+  {
+    if ('a' <= c && c <= 'z')
+    {
+      c -= (32); // 32 = 'a' - 'A'
+    }
+    return c;
+  }
+
+  /**
    * Calculates the conservation sequence
    * 
    * @param consflag
@@ -374,14 +387,14 @@ public class Conservation
     {
       consString.append('-');
     }
-    consSymbs = new String[end-start+1];
+    consSymbs = new String[end - start + 1];
     for (int i = start; i <= end; i++)
     {
       gapcons = countConsNGaps(i);
       totGaps = gapcons[1];
       pgaps = ((float) totGaps * 100) / sequences.length;
-      consSymbs[i-start]=new String();
-      
+      consSymbs[i - start] = new String();
+
       if (percentageGaps > pgaps)
       {
         resultHash = total[i - start];
@@ -398,7 +411,7 @@ public class Conservation
           {
             if (result.intValue() == 1)
             {
-              consSymbs[i-start] = type+" "+consSymbs[i-start];
+              consSymbs[i - start] = type + " " + consSymbs[i - start];
               count++;
             }
           }
@@ -406,14 +419,17 @@ public class Conservation
           {
             if (result.intValue() != -1)
             {
-              { 
-                 if (result.intValue()==0) {
-                   consSymbs[i-start] = consSymbs[i-start]+ " !"+type;
-                 } else {
-                   consSymbs[i-start] = type+" "+consSymbs[i-start];
-                 }
+              {
+                if (result.intValue() == 0)
+                {
+                  consSymbs[i - start] = consSymbs[i - start] + " !" + type;
+                }
+                else
+                {
+                  consSymbs[i - start] = type + " " + consSymbs[i - start];
+                }
               }
-              
+
               count++;
             }
           }
@@ -702,9 +718,8 @@ public class Conservation
       String conssym = (value > 0 && consp > -1 && consp < consSymbs.length) ? consSymbs[consp]
               : "";
       conservation.annotations[i] = new Annotation(String.valueOf(c),
-              conssym, ' ', value, new Color(minR
-                      + (maxR * vprop), minG + (maxG * vprop), minB
-                      + (maxB * vprop)));
+              conssym, ' ', value, new Color(minR + (maxR * vprop), minG
+                      + (maxG * vprop), minB + (maxB * vprop)));
 
       // Quality calc
       if (quality2 != null)