JAL-4134 support recovery of mapped positions for raw matrix column index in Mappable...

[jalview.git] / src / jalview / datamodel / MappedFeatures.java

diff --git a/src/jalview/datamodel/MappedFeatures.java b/src/jalview/datamodel/MappedFeatures.java
index 57c8c37..4637974 100644 (file)
--- a/src/jalview/datamodel/MappedFeatures.java
+++ b/src/jalview/datamodel/MappedFeatures.java
@@ -20,6 +20,8 @@
  */
 package jalview.datamodel;
 
+import java.util.Locale;
+
 import java.util.HashSet;
 import java.util.List;
 import java.util.Set;
@@ -110,7 +112,8 @@ public class MappedFeatures
      * determine codon positions and canonical codon
      * for a peptide-to-CDS mapping
      */
-    int[] codonIntervals = mapping.getMap().locateInFrom(toPosition, toPosition);
+    int[] codonIntervals = mapping.getMap().locateInFrom(toPosition,
+            toPosition);
     int[] codonPositions = codonIntervals == null ? null
             : MappingUtils.flattenRanges(codonIntervals);
     if (codonPositions != null && codonPositions.length == 3)
@@ -118,12 +121,12 @@ public class MappedFeatures
       codonPos = codonPositions;
       baseCodon = new char[3];
       int cdsStart = featureSequence.getStart();
-      baseCodon[0] = Character
-              .toUpperCase(featureSequence.getCharAt(codonPos[0] - cdsStart));
-      baseCodon[1] = Character
-              .toUpperCase(featureSequence.getCharAt(codonPos[1] - cdsStart));
-      baseCodon[2] = Character
-              .toUpperCase(featureSequence.getCharAt(codonPos[2] - cdsStart));
+      baseCodon[0] = Character.toUpperCase(
+              featureSequence.getCharAt(codonPos[0] - cdsStart));
+      baseCodon[1] = Character.toUpperCase(
+              featureSequence.getCharAt(codonPos[1] - cdsStart));
+      baseCodon[2] = Character.toUpperCase(
+              featureSequence.getCharAt(codonPos[2] - cdsStart));
     }
     else
     {
@@ -199,12 +202,12 @@ public class MappedFeatures
      * e.g. C,G,T gives variants G and T for base C
      */
     Set<String> variantPeptides = new HashSet<>();
-    String[] alleles = alls.toUpperCase().split(",");
+    String[] alleles = alls.toUpperCase(Locale.ROOT).split(",");
     StringBuilder vars = new StringBuilder();
 
     for (String allele : alleles)
     {
-      allele = allele.trim().toUpperCase();
+      allele = allele.trim().toUpperCase(Locale.ROOT);
       if (allele.length() > 1 || "-".equals(allele))
       {
         continue; // multi-locus variant
@@ -220,7 +223,7 @@ public class MappedFeatures
        */
       final int i = cdsPos == codonPos[0] ? 0
               : (cdsPos == codonPos[1] ? 1 : 2);
-      variantCodon[i] = allele.toUpperCase().charAt(0);
+      variantCodon[i] = allele.toUpperCase(Locale.ROOT).charAt(0);
       if (variantCodon[i] == baseCodon[i])
       {
         continue;
@@ -236,8 +239,7 @@ public class MappedFeatures
          */
         var.append("c.").append(String.valueOf(cdsPos))
                 .append(String.valueOf(baseCodon[i])).append(">")
-                .append(String.valueOf(variantCodon[i]))
-                .append("(p.=)");
+                .append(String.valueOf(variantCodon[i])).append("(p.=)");
       }
       else
       {