JAL-3518 more pull up / test coverage of structure command generation

[jalview.git] / src / jalview / ext / jmol / JmolCommands.java

diff --git a/src/jalview/ext/jmol/JmolCommands.java b/src/jalview/ext/jmol/JmolCommands.java
index 7dd5c0b..6f682be 100644 (file)
--- a/src/jalview/ext/jmol/JmolCommands.java
+++ b/src/jalview/ext/jmol/JmolCommands.java
@@ -71,7 +71,13 @@ public class JmolCommands extends StructureCommandsBase
     return 1;
   }
 
-  @Override
+  /**
+   * Returns a string representation of the given colour suitable for inclusion
+   * in Jmol commands
+   * 
+   * @param c
+   * @return
+   */
   protected String getColourString(Color c)
   {
     return c == null ? null
@@ -79,27 +85,13 @@ public class JmolCommands extends StructureCommandsBase
                     c.getBlue());
   }
 
-  /**
-   * Returns commands (one per colour key in the map) like
-   * 
-   * <pre>
-   *   select 2:A/1.1|3-27:B/1.1|9-12:A/2.1;color[173,0,82]
-   * </pre>
-   */
-  @Override
-  public String[] colourBySequence(Map<Object, AtomSpecModel> colourMap)
-  {
-    List<String> colourCommands = buildColourCommands(colourMap);
-
-    return colourCommands.toArray(new String[colourCommands.size()]);
-  }
 
   public String[] colourBySequence(StructureSelectionManager ssm,
           String[] files,
           SequenceI[][] sequence, SequenceRenderer sr,
           AlignmentViewPanel viewPanel)
   {
-    // TODO refactor to call buildColoursMap() first...
+    // TODO delete method
 
     FeatureRenderer fr = viewPanel.getFeatureRenderer();
     FeatureColourFinder finder = new FeatureColourFinder(fr);