X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fanalysis%2FAlignmentSorter.java;h=60052088892beded922573870b3b1b3060c3e5a9;hb=4a3def9f59cefe629c9a33d87483283aee085928;hp=058cc72ce6978e3dfc8948733eb76d54b9542510;hpb=ef887ebe817cd6595b6275f5f51001677c377214;p=jalview.git

diff --git a/src/jalview/analysis/AlignmentSorter.java b/src/jalview/analysis/AlignmentSorter.java
index 058cc72..6005208 100755
--- a/src/jalview/analysis/AlignmentSorter.java
+++ b/src/jalview/analysis/AlignmentSorter.java
@@ -29,8 +29,8 @@ import jalview.datamodel.SequenceFeature;
 import jalview.datamodel.SequenceGroup;
 import jalview.datamodel.SequenceI;
 import jalview.datamodel.SequenceNode;
-import jalview.ext.treeviewer.ExternalTreeI;
-import jalview.ext.treeviewer.ExternalTreeNodeI;
+import jalview.ext.treeviewer.TreeI;
+import jalview.ext.treeviewer.TreeNodeI;
 import jalview.util.QuickSort;
 
 import java.util.ArrayList;
@@ -72,7 +72,7 @@ public class AlignmentSorter
 
   static TreeModel lastTree = null;
 
-  static ExternalTreeI lastExternalTree = null;
+  static TreeI lastExternalTree = null;
 
   static boolean sortTreeAscending = true;
 
@@ -148,8 +148,8 @@ public class AlignmentSorter
     }
 
     // NOTE: DO NOT USE align.setSequenceAt() here - it will NOT work
-    List<SequenceI> asq;
-    synchronized (asq = align.getSequences())
+    List<SequenceI> asq = align.getSequences();
+    synchronized (asq)
     {
       for (int i = 0; i < len; i++)
       {
@@ -184,8 +184,8 @@ public class AlignmentSorter
   public static void setOrder(AlignmentI align, SequenceI[] seqs)
   {
     // NOTE: DO NOT USE align.setSequenceAt() here - it will NOT work
-    List<SequenceI> algn;
-    synchronized (algn = align.getSequences())
+    List<SequenceI> algn = align.getSequences();
+    synchronized (algn)
     {
       List<SequenceI> tmp = new ArrayList<>();
 
@@ -468,16 +468,16 @@ public class AlignmentSorter
 
 
 
-  private static List<SequenceI> getOrderByTree(ExternalTreeI aptxTree,
-          Map<ExternalTreeNodeI, SequenceI> nodesWithBoundSeqs)
+  private static List<SequenceI> getOrderByTree(TreeI aptxTree,
+          Map<TreeNodeI, SequenceI> nodesWithBoundSeqs)
   {
     List<SequenceI> seqsByTreeOrder = new ArrayList<>();
     if (!aptxTree.isEmpty())
     {
-      for (final Iterator<ExternalTreeNodeI> iter = aptxTree
+      for (final Iterator<TreeNodeI> iter = aptxTree
               .iterateInPreOrder(); iter.hasNext();)
       {
-        ExternalTreeNodeI treeNode = iter.next();
+        TreeNodeI treeNode = iter.next();
         seqsByTreeOrder.add(nodesWithBoundSeqs.get(treeNode));
       }
 
@@ -530,17 +530,17 @@ public class AlignmentSorter
    *          tree which has
    */
   public static void sortByTree(AlignmentI align,
-          Map<ExternalTreeNodeI, SequenceI> nodesBoundToSequences,
-          ExternalTreeI externalTreeI) throws IllegalArgumentException
+          Map<TreeNodeI, SequenceI> nodesBoundToSequences,
+          TreeI treeI) throws IllegalArgumentException
   {
-    List<SequenceI> tmp = getOrderByTree(externalTreeI, nodesBoundToSequences);
+    List<SequenceI> tmp = getOrderByTree(treeI, nodesBoundToSequences);
 
     if (!tmp.isEmpty())
     {
-      if (lastExternalTree != externalTreeI)
+      if (lastExternalTree != treeI)
       {
         sortTreeAscending = true;
-        lastExternalTree = externalTreeI;
+        lastExternalTree = treeI;
       }
       else
       {
@@ -656,7 +656,7 @@ public class AlignmentSorter
 
     for (int i = 0; i < alignment.length; i++)
     {
-      ids[i] = (new Float(alignment[i].getName().substring(8)))
+      ids[i] = (Float.valueOf(alignment[i].getName().substring(8)))
               .floatValue();
     }
 
@@ -778,14 +778,15 @@ public class AlignmentSorter
     if (method != FEATURE_SCORE && method != FEATURE_LABEL
             && method != FEATURE_DENSITY)
     {
-      String msg = String
-              .format("Implementation Error - sortByFeature method must be either '%s' or '%s'",
-                      FEATURE_SCORE, FEATURE_DENSITY);
+      String msg = String.format(
+              "Implementation Error - sortByFeature method must be either '%s' or '%s'",
+              FEATURE_SCORE, FEATURE_DENSITY);
       System.err.println(msg);
       return;
     }
 
-    flipFeatureSortIfUnchanged(method, featureTypes, groups, startCol, endCol);
+    flipFeatureSortIfUnchanged(method, featureTypes, groups, startCol,
+            endCol);
 
     SequenceI[] seqs = alignment.getSequencesArray();
 
@@ -804,8 +805,8 @@ public class AlignmentSorter
        * get sequence residues overlapping column region
        * and features for residue positions and specified types
        */
-      String[] types = featureTypes == null ? null : featureTypes
-              .toArray(new String[featureTypes.size()]);
+      String[] types = featureTypes == null ? null
+              : featureTypes.toArray(new String[featureTypes.size()]);
       List<SequenceFeature> sfs = seqs[i].findFeatures(startCol + 1,
               endCol + 1, types);