JAL-583 refactor todo about separating code to construct Jmol superposition from...
authorJim Procter <jprocter@compbio.dundee.ac.uk>
Sun, 7 Jul 2013 11:39:43 +0000 (12:39 +0100)
committerJim Procter <jprocter@compbio.dundee.ac.uk>
Sun, 7 Jul 2013 11:40:42 +0000 (12:40 +0100)
src/jalview/ext/jmol/JalviewJmolBinding.java

index 4ac247c..db1162e 100644 (file)
@@ -469,6 +469,13 @@ public abstract class JalviewJmolBinding implements StructureListener,
           }
         }
       }
+      
+      // TODO: consider bailing if nmatched less than 4 because superposition
+      // not
+      // well defined.
+      // TODO: refactor superposable position search (above) from jmol selection
+      // construction (below)
+
       String[] selcom = new String[files.length];
       int nmatched = 0;
       // generate select statements to select regions to superimpose structures
@@ -534,11 +541,6 @@ public abstract class JalviewJmolBinding implements StructureListener,
           }
         }
       }
-      // TODO: consider bailing if nmatched less than 4 because superposition
-      // not
-      // well defined.
-      // TODO: refactor superposable position search (above) from jmol selection
-      // construction (below)
       for (int pdbfnum = 0; pdbfnum < files.length; pdbfnum++)
       {
         if (pdbfnum == refStructure)