JAL-4090 JAL-1551 spotlessApply
[jalview.git] / src / jalview / bin / JalviewLite.java
index 36a7cff..b73fffc 100644 (file)
@@ -20,6 +20,8 @@
  */
 package jalview.bin;
 
+import java.util.Locale;
+
 import jalview.analysis.AlignmentUtils;
 import jalview.api.StructureSelectionManagerProvider;
 import jalview.appletgui.AlignFrame;
@@ -37,6 +39,9 @@ import jalview.datamodel.SequenceGroup;
 import jalview.datamodel.SequenceI;
 import jalview.io.AnnotationFile;
 import jalview.io.AppletFormatAdapter;
+import jalview.io.DataSourceType;
+import jalview.io.FileFormatI;
+import jalview.io.FileFormats;
 import jalview.io.FileParse;
 import jalview.io.IdentifyFile;
 import jalview.io.JPredFile;
@@ -48,6 +53,7 @@ import jalview.javascript.JsCallBack;
 import jalview.javascript.MouseOverStructureListener;
 import jalview.structure.SelectionListener;
 import jalview.structure.StructureSelectionManager;
+import jalview.util.ColorUtils;
 import jalview.util.HttpUtils;
 import jalview.util.MessageManager;
 
@@ -63,6 +69,7 @@ import java.awt.event.ActionEvent;
 import java.awt.event.WindowAdapter;
 import java.awt.event.WindowEvent;
 import java.io.BufferedReader;
+import java.io.IOException;
 import java.io.InputStreamReader;
 import java.net.URL;
 import java.util.ArrayList;
@@ -79,8 +86,8 @@ import netscape.javascript.JSObject;
  * @author $author$
  * @version $Revision: 1.92 $
  */
-public class JalviewLite extends Applet implements
-        StructureSelectionManagerProvider, JalviewLiteJsApi
+public class JalviewLite extends Applet
+        implements StructureSelectionManagerProvider, JalviewLiteJsApi
 {
 
   private static final String TRUE = "true";
@@ -100,6 +107,7 @@ public class JalviewLite extends Applet implements
    * 
    * @see jalview.bin.JalviewLiteJsApi#getSelectedSequences()
    */
+  @Override
   public String getSelectedSequences()
   {
     return getSelectedSequencesFrom(getDefaultTargetFrame());
@@ -110,6 +118,7 @@ public class JalviewLite extends Applet implements
    * 
    * @see jalview.bin.JalviewLiteJsApi#getSelectedSequences(java.lang.String)
    */
+  @Override
   public String getSelectedSequences(String sep)
   {
     return getSelectedSequencesFrom(getDefaultTargetFrame(), sep);
@@ -122,6 +131,7 @@ public class JalviewLite extends Applet implements
    * jalview.bin.JalviewLiteJsApi#getSelectedSequencesFrom(jalview.appletgui
    * .AlignFrame)
    */
+  @Override
   public String getSelectedSequencesFrom(AlignFrame alf)
   {
     return getSelectedSequencesFrom(alf, separator); // ""+0x00AC);
@@ -134,6 +144,7 @@ public class JalviewLite extends Applet implements
    * jalview.bin.JalviewLiteJsApi#getSelectedSequencesFrom(jalview.appletgui
    * .AlignFrame, java.lang.String)
    */
+  @Override
   public String getSelectedSequencesFrom(AlignFrame alf, String sep)
   {
     StringBuffer result = new StringBuffer("");
@@ -162,6 +173,7 @@ public class JalviewLite extends Applet implements
    * @see jalview.bin.JalviewLiteJsApi#highlight(java.lang.String,
    * java.lang.String, java.lang.String)
    */
+  @Override
   public void highlight(String sequenceId, String position,
           String alignedPosition)
   {
@@ -175,6 +187,7 @@ public class JalviewLite extends Applet implements
    * @see jalview.bin.JalviewLiteJsApi#highlightIn(jalview.appletgui.AlignFrame,
    * java.lang.String, java.lang.String, java.lang.String)
    */
+  @Override
   public void highlightIn(final AlignFrame alf, final String sequenceId,
           final String position, final String alignedPosition)
   {
@@ -187,7 +200,7 @@ public class JalviewLite extends Applet implements
       int apos = -1;
       try
       {
-        apos = new Integer(position).intValue();
+        apos = Integer.valueOf(position).intValue();
         apos--;
       } catch (NumberFormatException ex)
       {
@@ -198,7 +211,7 @@ public class JalviewLite extends Applet implements
       // use vamsas listener to broadcast to all listeners in scope
       if (alignedPosition != null
               && (alignedPosition.trim().length() == 0 || alignedPosition
-                      .toLowerCase().indexOf("false") > -1))
+                      .toLowerCase(Locale.ROOT).indexOf("false") > -1))
       {
         java.awt.EventQueue.invokeLater(new Runnable()
         {
@@ -231,6 +244,7 @@ public class JalviewLite extends Applet implements
    * @see jalview.bin.JalviewLiteJsApi#select(java.lang.String,
    * java.lang.String)
    */
+  @Override
   public void select(String sequenceIds, String columns)
   {
     selectIn(getDefaultTargetFrame(), sequenceIds, columns, separator);
@@ -242,6 +256,7 @@ public class JalviewLite extends Applet implements
    * @see jalview.bin.JalviewLiteJsApi#select(java.lang.String,
    * java.lang.String, java.lang.String)
    */
+  @Override
   public void select(String sequenceIds, String columns, String sep)
   {
     selectIn(getDefaultTargetFrame(), sequenceIds, columns, sep);
@@ -253,6 +268,7 @@ public class JalviewLite extends Applet implements
    * @see jalview.bin.JalviewLiteJsApi#selectIn(jalview.appletgui.AlignFrame,
    * java.lang.String, java.lang.String)
    */
+  @Override
   public void selectIn(AlignFrame alf, String sequenceIds, String columns)
   {
     selectIn(alf, sequenceIds, columns, separator);
@@ -264,6 +280,7 @@ public class JalviewLite extends Applet implements
    * @see jalview.bin.JalviewLiteJsApi#selectIn(jalview.appletgui.AlignFrame,
    * java.lang.String, java.lang.String, java.lang.String)
    */
+  @Override
   public void selectIn(final AlignFrame alf, String sequenceIds,
           String columns, String sep)
   {
@@ -275,8 +292,9 @@ public class JalviewLite extends Applet implements
     {
       if (debug)
       {
-        System.err.println("Selecting region using separator string '"
-                + separator + "'");
+        jalview.bin.Console
+                .errPrintln("Selecting region using separator string '"
+                        + separator + "'");
       }
     }
     // deparse fields
@@ -323,23 +341,23 @@ public class JalviewLite extends Applet implements
           int from = -1, to = -1;
           try
           {
-            from = new Integer(cl.substring(0, p)).intValue();
+            from = Integer.valueOf(cl.substring(0, p)).intValue();
             from--;
           } catch (NumberFormatException ex)
           {
-            System.err
-                    .println("ERROR: Couldn't parse first integer in range element column selection string '"
+            jalview.bin.Console.errPrintln(
+                    "ERROR: Couldn't parse first integer in range element column selection string '"
                             + cl + "' - format is 'from-to'");
             return;
           }
           try
           {
-            to = new Integer(cl.substring(p + 1)).intValue();
+            to = Integer.valueOf(cl.substring(p + 1)).intValue();
             to--;
           } catch (NumberFormatException ex)
           {
-            System.err
-                    .println("ERROR: Couldn't parse second integer in range element column selection string '"
+            jalview.bin.Console.errPrintln(
+                    "ERROR: Couldn't parse second integer in range element column selection string '"
                             + cl + "' - format is 'from-to'");
             return;
           }
@@ -379,13 +397,13 @@ public class JalviewLite extends Applet implements
             }
             if (debug)
             {
-              System.err.println("Range '" + cl + "' deparsed as [" + from
-                      + "," + to + "]");
+              jalview.bin.Console.errPrintln("Range '" + cl
+                      + "' deparsed as [" + from + "," + to + "]");
             }
           }
           else
           {
-            System.err.println("ERROR: Invalid Range '" + cl
+            jalview.bin.Console.errPrintln("ERROR: Invalid Range '" + cl
                     + "' deparsed as [" + from + "," + to + "]");
           }
         }
@@ -394,19 +412,19 @@ public class JalviewLite extends Applet implements
           int r = -1;
           try
           {
-            r = new Integer(cl).intValue();
+            r = Integer.valueOf(cl).intValue();
             r--;
           } catch (NumberFormatException ex)
           {
-            if (cl.toLowerCase().equals("sequence"))
+            if (cl.toLowerCase(Locale.ROOT).equals("sequence"))
             {
               // we are in the dataset sequence's coordinate frame.
               inseqpos = true;
             }
             else
             {
-              System.err
-                      .println("ERROR: Couldn't parse integer from point selection element of column selection string '"
+              jalview.bin.Console.errPrintln(
+                      "ERROR: Couldn't parse integer from point selection element of column selection string '"
                               + cl + "'");
               return;
             }
@@ -434,14 +452,15 @@ public class JalviewLite extends Applet implements
             csel.addElement(r);
             if (debug)
             {
-              System.err.println("Point selection '" + cl
+              jalview.bin.Console.errPrintln("Point selection '" + cl
                       + "' deparsed as [" + r + "]");
             }
           }
           else
           {
-            System.err.println("ERROR: Invalid Point selection '" + cl
-                    + "' deparsed as [" + r + "]");
+            jalview.bin.Console
+                    .errPrintln("ERROR: Invalid Point selection '" + cl
+                            + "' deparsed as [" + r + "]");
           }
         }
       }
@@ -456,14 +475,11 @@ public class JalviewLite extends Applet implements
         SequenceI rs = sel.getSequenceAt(0);
         start = rs.findIndex(start);
         end = rs.findIndex(end);
-        if (csel != null)
+        List<Integer> cs = new ArrayList<>(csel.getSelected());
+        csel.clear();
+        for (Integer selectedCol : cs)
         {
-          List<Integer> cs = csel.getSelected();
-          csel.clear();
-          for (Integer selectedCol : cs)
-          {
-            csel.addElement(rs.findIndex(selectedCol));
-          }
+          csel.addElement(rs.findIndex(selectedCol));
         }
       }
       sel.setStartRes(start);
@@ -473,7 +489,8 @@ public class JalviewLite extends Applet implements
         @Override
         public void run()
         {
-          alf.select(sel, csel);
+          alf.select(sel, csel,
+                  alf.getAlignViewport().getAlignment().getHiddenColumns());
         }
       });
     }
@@ -486,7 +503,9 @@ public class JalviewLite extends Applet implements
    * jalview.bin.JalviewLiteJsApi#getSelectedSequencesAsAlignment(java.lang.
    * String, java.lang.String)
    */
-  public String getSelectedSequencesAsAlignment(String format, String suffix)
+  @Override
+  public String getSelectedSequencesAsAlignment(String format,
+          String suffix)
   {
     return getSelectedSequencesAsAlignmentFrom(getDefaultTargetFrame(),
             format, suffix);
@@ -499,26 +518,29 @@ public class JalviewLite extends Applet implements
    * jalview.bin.JalviewLiteJsApi#getSelectedSequencesAsAlignmentFrom(jalview
    * .appletgui.AlignFrame, java.lang.String, java.lang.String)
    */
+  @Override
   public String getSelectedSequencesAsAlignmentFrom(AlignFrame alf,
           String format, String suffix)
   {
     try
     {
+      FileFormatI theFormat = FileFormats.getInstance().forName(format);
       boolean seqlimits = suffix.equalsIgnoreCase(TRUE);
       if (alf.viewport.getSelectionGroup() != null)
       {
         // JBPNote: getSelectionAsNewSequence behaviour has changed - this
         // method now returns a full copy of sequence data
         // TODO consider using getSequenceSelection instead here
-        String reply = new AppletFormatAdapter().formatSequences(format,
+        String reply = new AppletFormatAdapter().formatSequences(theFormat,
                 new Alignment(alf.viewport.getSelectionAsNewSequence()),
                 seqlimits);
         return reply;
       }
-    } catch (Exception ex)
+    } catch (IllegalArgumentException ex)
     {
       ex.printStackTrace();
-      return "Error retrieving alignment in " + format + " format. ";
+      return "Error retrieving alignment, possibly invalid format specifier: "
+              + format;
     }
     return "";
   }
@@ -528,6 +550,7 @@ public class JalviewLite extends Applet implements
    * 
    * @see jalview.bin.JalviewLiteJsApi#getAlignmentOrder()
    */
+  @Override
   public String getAlignmentOrder()
   {
     return getAlignmentOrderFrom(getDefaultTargetFrame());
@@ -540,6 +563,7 @@ public class JalviewLite extends Applet implements
    * jalview.bin.JalviewLiteJsApi#getAlignmentOrderFrom(jalview.appletgui.AlignFrame
    * )
    */
+  @Override
   public String getAlignmentOrderFrom(AlignFrame alf)
   {
     return getAlignmentOrderFrom(alf, separator);
@@ -552,6 +576,7 @@ public class JalviewLite extends Applet implements
    * jalview.bin.JalviewLiteJsApi#getAlignmentOrderFrom(jalview.appletgui.AlignFrame
    * , java.lang.String)
    */
+  @Override
   public String getAlignmentOrderFrom(AlignFrame alf, String sep)
   {
     AlignmentI alorder = alf.getAlignViewport().getAlignment();
@@ -569,6 +594,7 @@ public class JalviewLite extends Applet implements
    * @see jalview.bin.JalviewLiteJsApi#orderBy(java.lang.String,
    * java.lang.String)
    */
+  @Override
   public String orderBy(String order, String undoName)
   {
     return orderBy(order, undoName, separator);
@@ -580,6 +606,7 @@ public class JalviewLite extends Applet implements
    * @see jalview.bin.JalviewLiteJsApi#orderBy(java.lang.String,
    * java.lang.String, java.lang.String)
    */
+  @Override
   public String orderBy(String order, String undoName, String sep)
   {
     return orderAlignmentBy(getDefaultTargetFrame(), order, undoName, sep);
@@ -592,6 +619,7 @@ public class JalviewLite extends Applet implements
    * jalview.bin.JalviewLiteJsApi#orderAlignmentBy(jalview.appletgui.AlignFrame,
    * java.lang.String, java.lang.String, java.lang.String)
    */
+  @Override
   public String orderAlignmentBy(AlignFrame alf, String order,
           String undoName, String sep)
   {
@@ -648,6 +676,7 @@ public class JalviewLite extends Applet implements
    * 
    * @see jalview.bin.JalviewLiteJsApi#getAlignment(java.lang.String)
    */
+  @Override
   public String getAlignment(String format)
   {
     return getAlignmentFrom(getDefaultTargetFrame(), format, TRUE);
@@ -660,6 +689,7 @@ public class JalviewLite extends Applet implements
    * jalview.bin.JalviewLiteJsApi#getAlignmentFrom(jalview.appletgui.AlignFrame,
    * java.lang.String)
    */
+  @Override
   public String getAlignmentFrom(AlignFrame alf, String format)
   {
     return getAlignmentFrom(alf, format, TRUE);
@@ -671,6 +701,7 @@ public class JalviewLite extends Applet implements
    * @see jalview.bin.JalviewLiteJsApi#getAlignment(java.lang.String,
    * java.lang.String)
    */
+  @Override
   public String getAlignment(String format, String suffix)
   {
     return getAlignmentFrom(getDefaultTargetFrame(), format, suffix);
@@ -683,6 +714,7 @@ public class JalviewLite extends Applet implements
    * jalview.bin.JalviewLiteJsApi#getAlignmentFrom(jalview.appletgui.AlignFrame,
    * java.lang.String, java.lang.String)
    */
+  @Override
   public String getAlignmentFrom(AlignFrame alf, String format,
           String suffix)
   {
@@ -690,13 +722,15 @@ public class JalviewLite extends Applet implements
     {
       boolean seqlimits = suffix.equalsIgnoreCase(TRUE);
 
-      String reply = new AppletFormatAdapter().formatSequences(format,
+      FileFormatI theFormat = FileFormats.getInstance().forName(format);
+      String reply = new AppletFormatAdapter().formatSequences(theFormat,
               alf.viewport.getAlignment(), seqlimits);
       return reply;
-    } catch (Exception ex)
+    } catch (IllegalArgumentException ex)
     {
       ex.printStackTrace();
-      return "Error retrieving alignment in " + format + " format. ";
+      return "Error retrieving alignment, possibly invalid format specifier: "
+              + format;
     }
   }
 
@@ -705,6 +739,7 @@ public class JalviewLite extends Applet implements
    * 
    * @see jalview.bin.JalviewLiteJsApi#loadAnnotation(java.lang.String)
    */
+  @Override
   public void loadAnnotation(String annotation)
   {
     loadAnnotationFrom(getDefaultTargetFrame(), annotation);
@@ -717,17 +752,18 @@ public class JalviewLite extends Applet implements
    * jalview.bin.JalviewLiteJsApi#loadAnnotationFrom(jalview.appletgui.AlignFrame
    * , java.lang.String)
    */
+  @Override
   public void loadAnnotationFrom(AlignFrame alf, String annotation)
   {
     if (new AnnotationFile().annotateAlignmentView(alf.getAlignViewport(),
-            annotation, AppletFormatAdapter.PASTE))
+            annotation, DataSourceType.PASTE))
     {
       alf.alignPanel.fontChanged();
       alf.alignPanel.setScrollValues(0, 0);
     }
     else
     {
-      alf.parseFeaturesFile(annotation, AppletFormatAdapter.PASTE);
+      alf.parseFeaturesFile(annotation, DataSourceType.PASTE);
     }
   }
 
@@ -736,6 +772,7 @@ public class JalviewLite extends Applet implements
    * 
    * @see jalview.bin.JalviewLiteJsApi#loadAnnotation(java.lang.String)
    */
+  @Override
   public void loadFeatures(String features, boolean autoenabledisplay)
   {
     loadFeaturesFrom(getDefaultTargetFrame(), features, autoenabledisplay);
@@ -748,10 +785,11 @@ public class JalviewLite extends Applet implements
    * jalview.bin.JalviewLiteJsApi#loadAnnotationFrom(jalview.appletgui.AlignFrame
    * , java.lang.String)
    */
+  @Override
   public boolean loadFeaturesFrom(AlignFrame alf, String features,
           boolean autoenabledisplay)
   {
-    return alf.parseFeaturesFile(features, AppletFormatAdapter.PASTE,
+    return alf.parseFeaturesFile(features, DataSourceType.PASTE,
             autoenabledisplay);
   }
 
@@ -760,6 +798,7 @@ public class JalviewLite extends Applet implements
    * 
    * @see jalview.bin.JalviewLiteJsApi#getFeatures(java.lang.String)
    */
+  @Override
   public String getFeatures(String format)
   {
     return getFeaturesFrom(getDefaultTargetFrame(), format);
@@ -772,6 +811,7 @@ public class JalviewLite extends Applet implements
    * jalview.bin.JalviewLiteJsApi#getFeaturesFrom(jalview.appletgui.AlignFrame,
    * java.lang.String)
    */
+  @Override
   public String getFeaturesFrom(AlignFrame alf, String format)
   {
     return alf.outputFeatures(false, format);
@@ -782,6 +822,7 @@ public class JalviewLite extends Applet implements
    * 
    * @see jalview.bin.JalviewLiteJsApi#getAnnotation()
    */
+  @Override
   public String getAnnotation()
   {
     return getAnnotationFrom(getDefaultTargetFrame());
@@ -794,6 +835,7 @@ public class JalviewLite extends Applet implements
    * jalview.bin.JalviewLiteJsApi#getAnnotationFrom(jalview.appletgui.AlignFrame
    * )
    */
+  @Override
   public String getAnnotationFrom(AlignFrame alf)
   {
     return alf.outputAnnotations(false);
@@ -804,6 +846,7 @@ public class JalviewLite extends Applet implements
    * 
    * @see jalview.bin.JalviewLiteJsApi#newView()
    */
+  @Override
   public AlignFrame newView()
   {
     return newViewFrom(getDefaultTargetFrame());
@@ -814,6 +857,7 @@ public class JalviewLite extends Applet implements
    * 
    * @see jalview.bin.JalviewLiteJsApi#newView(java.lang.String)
    */
+  @Override
   public AlignFrame newView(String name)
   {
     return newViewFrom(getDefaultTargetFrame(), name);
@@ -824,6 +868,7 @@ public class JalviewLite extends Applet implements
    * 
    * @see jalview.bin.JalviewLiteJsApi#newViewFrom(jalview.appletgui.AlignFrame)
    */
+  @Override
   public AlignFrame newViewFrom(AlignFrame alf)
   {
     return alf.newView(null);
@@ -835,6 +880,7 @@ public class JalviewLite extends Applet implements
    * @see jalview.bin.JalviewLiteJsApi#newViewFrom(jalview.appletgui.AlignFrame,
    * java.lang.String)
    */
+  @Override
   public AlignFrame newViewFrom(AlignFrame alf, String name)
   {
     return alf.newView(name);
@@ -846,21 +892,22 @@ public class JalviewLite extends Applet implements
    * @see jalview.bin.JalviewLiteJsApi#loadAlignment(java.lang.String,
    * java.lang.String)
    */
+  @Override
   public AlignFrame loadAlignment(String text, String title)
   {
     AlignmentI al = null;
 
-    String format = new IdentifyFile().Identify(text,
-            AppletFormatAdapter.PASTE);
     try
     {
-      al = new AppletFormatAdapter().readFile(text,
-              AppletFormatAdapter.PASTE, format);
+      FileFormatI format = new IdentifyFile().identify(text,
+              DataSourceType.PASTE);
+      al = new AppletFormatAdapter().readFile(text, DataSourceType.PASTE,
+              format);
       if (al.getHeight() > 0)
       {
         return new AlignFrame(al, this, title, false);
       }
-    } catch (java.io.IOException ex)
+    } catch (IOException ex)
     {
       ex.printStackTrace();
     }
@@ -872,12 +919,13 @@ public class JalviewLite extends Applet implements
    * 
    * @see jalview.bin.JalviewLiteJsApi#setMouseoverListener(java.lang.String)
    */
+  @Override
   public void setMouseoverListener(String listener)
   {
     setMouseoverListener(currentAlignFrame, listener);
   }
 
-  private Vector<jalview.javascript.JSFunctionExec> javascriptListeners = new Vector<jalview.javascript.JSFunctionExec>();
+  private Vector<jalview.javascript.JSFunctionExec> javascriptListeners = new Vector<>();
 
   /*
    * (non-Javadoc)
@@ -886,6 +934,7 @@ public class JalviewLite extends Applet implements
    * jalview.bin.JalviewLiteJsApi#setMouseoverListener(jalview.appletgui.AlignFrame
    * , java.lang.String)
    */
+  @Override
   public void setMouseoverListener(AlignFrame af, String listener)
   {
     if (listener != null)
@@ -893,8 +942,8 @@ public class JalviewLite extends Applet implements
       listener = listener.trim();
       if (listener.length() == 0)
       {
-        System.err
-                .println("jalview Javascript error: Ignoring empty function for mouseover listener.");
+        jalview.bin.Console.errPrintln(
+                "jalview Javascript error: Ignoring empty function for mouseover listener.");
         return;
       }
     }
@@ -905,11 +954,12 @@ public class JalviewLite extends Applet implements
             .addStructureViewerListener(mol);
     if (debug)
     {
-      System.err.println("Added a mouseover listener for "
-              + ((af == null) ? "All frames" : "Just views for "
-                      + af.getAlignViewport().getSequenceSetId()));
-      System.err.println("There are now " + javascriptListeners.size()
-              + " listeners in total.");
+      jalview.bin.Console.errPrintln("Added a mouseover listener for "
+              + ((af == null) ? "All frames"
+                      : "Just views for "
+                              + af.getAlignViewport().getSequenceSetId()));
+      jalview.bin.Console.errPrintln("There are now "
+              + javascriptListeners.size() + " listeners in total.");
     }
   }
 
@@ -918,6 +968,7 @@ public class JalviewLite extends Applet implements
    * 
    * @see jalview.bin.JalviewLiteJsApi#setSelectionListener(java.lang.String)
    */
+  @Override
   public void setSelectionListener(String listener)
   {
     setSelectionListener(null, listener);
@@ -930,6 +981,7 @@ public class JalviewLite extends Applet implements
    * jalview.bin.JalviewLiteJsApi#setSelectionListener(jalview.appletgui.AlignFrame
    * , java.lang.String)
    */
+  @Override
   public void setSelectionListener(AlignFrame af, String listener)
   {
     if (listener != null)
@@ -937,8 +989,8 @@ public class JalviewLite extends Applet implements
       listener = listener.trim();
       if (listener.length() == 0)
       {
-        System.err
-                .println("jalview Javascript error: Ignoring empty function for selection listener.");
+        jalview.bin.Console.errPrintln(
+                "jalview Javascript error: Ignoring empty function for selection listener.");
         return;
       }
     }
@@ -949,11 +1001,12 @@ public class JalviewLite extends Applet implements
             .addSelectionListener(mol);
     if (debug)
     {
-      System.err.println("Added a selection listener for "
-              + ((af == null) ? "All frames" : "Just views for "
-                      + af.getAlignViewport().getSequenceSetId()));
-      System.err.println("There are now " + javascriptListeners.size()
-              + " listeners in total.");
+      jalview.bin.Console.errPrintln("Added a selection listener for "
+              + ((af == null) ? "All frames"
+                      : "Just views for "
+                              + af.getAlignViewport().getSequenceSetId()));
+      jalview.bin.Console.errPrintln("There are now "
+              + javascriptListeners.size() + " listeners in total.");
     }
   }
 
@@ -968,6 +1021,7 @@ public class JalviewLite extends Applet implements
    * @see jalview.bin.JalviewLiteJsApi#setStructureListener(java.lang.String,
    *      java.lang.String)
    */
+  @Override
   public void setStructureListener(String listener, String modelSet)
   {
     if (listener != null)
@@ -975,8 +1029,8 @@ public class JalviewLite extends Applet implements
       listener = listener.trim();
       if (listener.length() == 0)
       {
-        System.err
-                .println("jalview Javascript error: Ignoring empty function for selection listener.");
+        jalview.bin.Console.errPrintln(
+                "jalview Javascript error: Ignoring empty function for selection listener.");
         return;
       }
     }
@@ -987,10 +1041,11 @@ public class JalviewLite extends Applet implements
             .addStructureViewerListener(mol);
     if (debug)
     {
-      System.err.println("Added a javascript structure viewer listener '"
-              + listener + "'");
-      System.err.println("There are now " + javascriptListeners.size()
-              + " listeners in total.");
+      jalview.bin.Console
+              .errPrintln("Added a javascript structure viewer listener '"
+                      + listener + "'");
+      jalview.bin.Console.errPrintln("There are now "
+              + javascriptListeners.size() + " listeners in total.");
     }
   }
 
@@ -1001,6 +1056,7 @@ public class JalviewLite extends Applet implements
    * jalview.bin.JalviewLiteJsApi#removeJavascriptListener(jalview.appletgui
    * .AlignFrame, java.lang.String)
    */
+  @Override
   public void removeJavascriptListener(AlignFrame af, String listener)
   {
     if (listener != null)
@@ -1016,9 +1072,8 @@ public class JalviewLite extends Applet implements
     {
       Object lstn = javascriptListeners.elementAt(ms);
       JsCallBack lstner = (JsCallBack) lstn;
-      if ((af == null || lstner.getAlignFrame() == af)
-              && (listener == null || lstner.getListenerFunction().equals(
-                      listener)))
+      if ((af == null || lstner.getAlignFrame() == af) && (listener == null
+              || lstner.getListenerFunction().equals(listener)))
       {
         javascriptListeners.removeElement(lstner);
         msSize--;
@@ -1035,7 +1090,8 @@ public class JalviewLite extends Applet implements
         rprt = debug;
         if (debug)
         {
-          System.err.println("Removed listener '" + listener + "'");
+          jalview.bin.Console
+                  .errPrintln("Removed listener '" + listener + "'");
         }
       }
       else
@@ -1045,20 +1101,22 @@ public class JalviewLite extends Applet implements
     }
     if (rprt)
     {
-      System.err.println("There are now " + javascriptListeners.size()
-              + " listeners in total.");
+      jalview.bin.Console.errPrintln("There are now "
+              + javascriptListeners.size() + " listeners in total.");
     }
   }
 
+  @Override
   public void stop()
   {
-    System.err.println("Applet " + getName() + " stop().");
+    jalview.bin.Console.errPrintln("Applet " + getName() + " stop().");
     tidyUp();
   }
 
+  @Override
   public void destroy()
   {
-    System.err.println("Applet " + getName() + " destroy().");
+    jalview.bin.Console.errPrintln("Applet " + getName() + " destroy().");
     tidyUp();
   }
 
@@ -1112,8 +1170,9 @@ public class JalviewLite extends Applet implements
    * @see jalview.bin.JalviewLiteJsApi#mouseOverStructure(java.lang.String,
    * java.lang.String, java.lang.String)
    */
-  public void mouseOverStructure(final String pdbResNum,
-          final String chain, final String pdbfile)
+  @Override
+  public void mouseOverStructure(final String pdbResNum, final String chain,
+          final String pdbfile)
   {
     final StructureSelectionManagerProvider me = this;
     java.awt.EventQueue.invokeLater(new Runnable()
@@ -1124,18 +1183,19 @@ public class JalviewLite extends Applet implements
         try
         {
           StructureSelectionManager.getStructureSelectionManager(me)
-                  .mouseOverStructure(new Integer(pdbResNum).intValue(),
+                  .mouseOverStructure(Integer.valueOf(pdbResNum).intValue(),
                           chain, pdbfile);
           if (debug)
           {
-            System.err.println("mouseOver for '" + pdbResNum
-                    + "' in chain '" + chain + "' in structure '" + pdbfile
-                    + "'");
+            System.err
+                    .println("mouseOver for '" + pdbResNum + "' in chain '"
+                            + chain + "' in structure '" + pdbfile + "'");
           }
         } catch (NumberFormatException e)
         {
-          System.err.println("Ignoring invalid residue number string '"
-                  + pdbResNum + "'");
+          jalview.bin.Console
+                  .errPrintln("Ignoring invalid residue number string '"
+                          + pdbResNum + "'");
         }
 
       }
@@ -1149,6 +1209,7 @@ public class JalviewLite extends Applet implements
    * jalview.bin.JalviewLiteJsApi#scrollViewToIn(jalview.appletgui.AlignFrame,
    * java.lang.String, java.lang.String)
    */
+  @Override
   public void scrollViewToIn(final AlignFrame alf, final String topRow,
           final String leftHandColumn)
   {
@@ -1159,14 +1220,15 @@ public class JalviewLite extends Applet implements
       {
         try
         {
-          alf.scrollTo(new Integer(topRow).intValue(), new Integer(
-                  leftHandColumn).intValue());
+          alf.scrollTo(Integer.valueOf(topRow).intValue(),
+                  Integer.valueOf(leftHandColumn).intValue());
 
         } catch (Exception ex)
         {
-          System.err.println("Couldn't parse integer arguments (topRow='"
-                  + topRow + "' and leftHandColumn='" + leftHandColumn
-                  + "')");
+          jalview.bin.Console
+                  .errPrintln("Couldn't parse integer arguments (topRow='"
+                          + topRow + "' and leftHandColumn='"
+                          + leftHandColumn + "')");
           ex.printStackTrace();
         }
       }
@@ -1191,12 +1253,13 @@ public class JalviewLite extends Applet implements
       {
         try
         {
-          alf.scrollToRow(new Integer(topRow).intValue());
+          alf.scrollToRow(Integer.valueOf(topRow).intValue());
 
         } catch (Exception ex)
         {
-          System.err.println("Couldn't parse integer arguments (topRow='"
-                  + topRow + "')");
+          jalview.bin.Console
+                  .errPrintln("Couldn't parse integer arguments (topRow='"
+                          + topRow + "')");
           ex.printStackTrace();
         }
 
@@ -1223,12 +1286,12 @@ public class JalviewLite extends Applet implements
       {
         try
         {
-          alf.scrollToColumn(new Integer(leftHandColumn).intValue());
+          alf.scrollToColumn(Integer.valueOf(leftHandColumn).intValue());
 
         } catch (Exception ex)
         {
-          System.err
-                  .println("Couldn't parse integer arguments (leftHandColumn='"
+          jalview.bin.Console.errPrintln(
+                  "Couldn't parse integer arguments (leftHandColumn='"
                           + leftHandColumn + "')");
           ex.printStackTrace();
         }
@@ -1280,7 +1343,7 @@ public class JalviewLite extends Applet implements
   private boolean alignPdbStructures = false;
 
   /**
-   * use an external structure viewer exclusively (no jmols or MCViews will be
+   * use an external structure viewer exclusively (no jmols or mc_views will be
    * opened by JalviewLite itself)
    */
   public boolean useXtrnalSviewer = false;
@@ -1302,8 +1365,8 @@ public class JalviewLite extends Applet implements
       {
         try
         {
-          BufferedReader reader = new BufferedReader(new InputStreamReader(
-                  url.openStream()));
+          BufferedReader reader = new BufferedReader(
+                  new InputStreamReader(url.openStream()));
           String line;
           while ((line = reader.readLine()) != null)
           {
@@ -1359,19 +1422,20 @@ public class JalviewLite extends Applet implements
     {
       if (debug)
       {
-        System.err.println("Applet context is '"
+        jalview.bin.Console.errPrintln("Applet context is '"
                 + getAppletContext().getClass().toString() + "'");
       }
       JSObject scriptObject = JSObject.getWindow(this);
       if (debug && scriptObject != null)
       {
-        System.err.println("Applet has Javascript callback support.");
+        jalview.bin.Console
+                .errPrintln("Applet has Javascript callback support.");
       }
 
     } catch (Exception ex)
     {
-      System.err
-              .println("Warning: No JalviewLite javascript callbacks available.");
+      jalview.bin.Console.errPrintln(
+              "Warning: No JalviewLite javascript callbacks available.");
       if (debug)
       {
         ex.printStackTrace();
@@ -1380,14 +1444,15 @@ public class JalviewLite extends Applet implements
 
     if (debug)
     {
-      System.err.println("JalviewLite Version " + getVersion());
-      System.err.println("Build Date : " + getBuildDate());
-      System.err.println("Installation : " + getInstallation());
+      jalview.bin.Console.errPrintln("JalviewLite Version " + getVersion());
+      jalview.bin.Console.errPrintln("Build Date : " + getBuildDate());
+      jalview.bin.Console.errPrintln("Installation : " + getInstallation());
     }
     String externalsviewer = getParameter("externalstructureviewer");
     if (externalsviewer != null)
     {
-      useXtrnalSviewer = externalsviewer.trim().toLowerCase().equals(TRUE);
+      useXtrnalSviewer = externalsviewer.trim().toLowerCase(Locale.ROOT)
+              .equals(TRUE);
     }
     /**
      * if true disable the check for jmol
@@ -1408,14 +1473,14 @@ public class JalviewLite extends Applet implements
         separator = sep;
         if (debug)
         {
-          System.err.println("Separator set to '" + separator + "'");
+          jalview.bin.Console
+                  .errPrintln("Separator set to '" + separator + "'");
         }
       }
       else
       {
-        throw new Error(
-                MessageManager
-                        .getString("error.invalid_separator_parameter"));
+        throw new Error(MessageManager
+                .getString("error.invalid_separator_parameter"));
       }
     }
     int r = 255;
@@ -1488,6 +1553,7 @@ public class JalviewLite extends Applet implements
         add(launcher);
         launcher.addActionListener(new java.awt.event.ActionListener()
         {
+          @Override
           public void actionPerformed(ActionEvent e)
           {
             LoadingThread loader = new LoadingThread(file, file2,
@@ -1516,7 +1582,8 @@ public class JalviewLite extends Applet implements
     {
       if (tries > 0)
       {
-        System.err.println("LiveConnect request thread going to sleep.");
+        jalview.bin.Console
+                .errPrintln("LiveConnect request thread going to sleep.");
       }
       try
       {
@@ -1527,7 +1594,8 @@ public class JalviewLite extends Applet implements
       ;
       if (tries++ > 0)
       {
-        System.err.println("LiveConnect request thread woken up.");
+        jalview.bin.Console
+                .errPrintln("LiveConnect request thread woken up.");
       }
       try
       {
@@ -1538,7 +1606,7 @@ public class JalviewLite extends Applet implements
         }
       } catch (Exception jsex)
       {
-        System.err.println("Attempt " + tries
+        jalview.bin.Console.errPrintln("Attempt " + tries
                 + " to access LiveConnect javascript failed.");
       }
     }
@@ -1571,18 +1639,19 @@ public class JalviewLite extends Applet implements
         {
           // do onInit with the JS executor thread
           new JSFunctionExec(this).executeJavascriptFunction(true,
-                  initjscallback, null, "Calling oninit callback '"
-                          + initjscallback + "'.");
+                  initjscallback, null,
+                  "Calling oninit callback '" + initjscallback + "'.");
         } catch (Exception e)
         {
-          System.err.println("Exception when executing _oninit callback '"
-                  + initjscallback + "'.");
+          jalview.bin.Console
+                  .errPrintln("Exception when executing _oninit callback '"
+                          + initjscallback + "'.");
           e.printStackTrace();
         }
       }
       else
       {
-        System.err.println("Not executing _oninit callback '"
+        jalview.bin.Console.errPrintln("Not executing _oninit callback '"
                 + initjscallback + "' - no scripting allowed.");
       }
     }
@@ -1610,6 +1679,7 @@ public class JalviewLite extends Applet implements
     frame.setTitle(title);
     frame.addWindowListener(new WindowAdapter()
     {
+      @Override
       public void windowClosing(WindowEvent e)
       {
         if (frame instanceof AlignFrame)
@@ -1634,6 +1704,7 @@ public class JalviewLite extends Applet implements
         frame.dispose();
       }
 
+      @Override
       public void windowActivated(WindowEvent e)
       {
         if (frame instanceof AlignFrame)
@@ -1641,7 +1712,7 @@ public class JalviewLite extends Applet implements
           ((AlignFrame) frame).viewport.applet.currentAlignFrame = (AlignFrame) frame;
           if (debug)
           {
-            System.err.println("Activated window " + frame);
+            jalview.bin.Console.errPrintln("Activated window " + frame);
           }
         }
         // be good.
@@ -1656,7 +1727,7 @@ public class JalviewLite extends Applet implements
        * 
        * public void windowDeactivated(WindowEvent e) { if (currentAlignFrame ==
        * frame) { currentAlignFrame = null; if (debug) {
-       * System.err.println("Deactivated window "+frame); } }
+       * jalview.bin.Console.errPrintln("Deactivated window "+frame); } }
        * super.windowDeactivated(e); }
        */
     });
@@ -1671,6 +1742,7 @@ public class JalviewLite extends Applet implements
    * @param g
    *          graphics context
    */
+  @Override
   public void paint(Graphics g)
   {
     if (!fileFound)
@@ -1680,8 +1752,8 @@ public class JalviewLite extends Applet implements
       g.fillRect(0, 0, getSize().width, getSize().height);
       g.setColor(Color.red);
       g.drawString(
-              MessageManager.getString("label.jalview_cannot_open_file"),
-              5, 15);
+              MessageManager.getString("label.jalview_cannot_open_file"), 5,
+              15);
       g.drawString("\"" + file + "\"", 5, 30);
     }
     else if (embedded)
@@ -1722,6 +1794,7 @@ public class JalviewLite extends Applet implements
   {
     private boolean running = false;
 
+    @Override
     public void run()
     {
       if (running || checkedForJmol)
@@ -1740,8 +1813,8 @@ public class JalviewLite extends Applet implements
           }
           if (!jmolAvailable)
           {
-            System.out
-                    .println("Jmol not available - Using MCview for structures");
+            jalview.bin.Console.outPrintln(
+                    "Jmol not available - Using mc_view for structures");
           }
         } catch (java.lang.ClassNotFoundException ex)
         {
@@ -1752,8 +1825,8 @@ public class JalviewLite extends Applet implements
         jmolAvailable = false;
         if (debug)
         {
-          System.err
-                  .println("Skipping Jmol check. Will use MCView (probably)");
+          jalview.bin.Console.errPrintln(
+                  "Skipping Jmol check. Will use mc_view (probably)");
         }
       }
       checkedForJmol = true;
@@ -1771,7 +1844,7 @@ public class JalviewLite extends Applet implements
     /**
      * State variable: protocol for access to file source
      */
-    String protocol;
+    DataSourceType protocol;
 
     String _file; // alignment file or URL spec
 
@@ -1783,7 +1856,7 @@ public class JalviewLite extends Applet implements
     {
       if (JalviewLite.debug)
       {
-        System.err.println(msg);
+        jalview.bin.Console.errPrintln(msg);
       }
     }
 
@@ -1801,7 +1874,7 @@ public class JalviewLite extends Applet implements
        */
       if (path.startsWith("PASTE"))
       {
-        protocol = AppletFormatAdapter.PASTE;
+        protocol = DataSourceType.PASTE;
         return path.substring(5);
       }
 
@@ -1810,7 +1883,7 @@ public class JalviewLite extends Applet implements
        */
       if (path.indexOf("://") != -1)
       {
-        protocol = AppletFormatAdapter.URL;
+        protocol = DataSourceType.URL;
         return path;
       }
 
@@ -1823,10 +1896,10 @@ public class JalviewLite extends Applet implements
       {
         if (debug)
         {
-          System.err.println("Prepended document base '" + documentBase
-                  + "' to make: '" + withDocBase + "'");
+          jalview.bin.Console.errPrintln("Prepended document base '"
+                  + documentBase + "' to make: '" + withDocBase + "'");
         }
-        protocol = AppletFormatAdapter.URL;
+        protocol = DataSourceType.URL;
         return withDocBase;
       }
 
@@ -1839,10 +1912,10 @@ public class JalviewLite extends Applet implements
       if (!withCodeBase.equals(withDocBase)
               && HttpUtils.isValidUrl(withCodeBase))
       {
-        protocol = AppletFormatAdapter.URL;
+        protocol = DataSourceType.URL;
         if (debug)
         {
-          System.err.println("Prepended codebase '" + codeBase
+          jalview.bin.Console.errPrintln("Prepended codebase '" + codeBase
                   + "' to make: '" + withCodeBase + "'");
         }
         return withCodeBase;
@@ -1854,7 +1927,7 @@ public class JalviewLite extends Applet implements
        */
       if (inArchive(path))
       {
-        protocol = AppletFormatAdapter.CLASSLOADER;
+        protocol = DataSourceType.CLASSLOADER;
       }
       return path;
     }
@@ -1866,6 +1939,7 @@ public class JalviewLite extends Applet implements
       applet = _applet;
     }
 
+    @Override
     public void run()
     {
       LoadJmolThread jmolchecker = new LoadJmolThread();
@@ -1890,7 +1964,8 @@ public class JalviewLite extends Applet implements
      */
     private void startLoading()
     {
-      dbgMsg("Loading thread started with:\n>>file\n" + _file + ">>endfile");
+      dbgMsg("Loading thread started with:\n>>file\n" + _file
+              + ">>endfile");
 
       dbgMsg("Loading started.");
 
@@ -1947,7 +2022,7 @@ public class JalviewLite extends Applet implements
                   + " as "
                   + (al1.isNucleotide() ? "protein product" : "cDNA")
                   + " for " + af.getTitle();
-          System.err.println(msg);
+          jalview.bin.Console.errPrintln(msg);
         }
       }
 
@@ -1967,11 +2042,12 @@ public class JalviewLite extends Applet implements
         return null;
       }
       String resolvedFile = resolveFileProtocol(fileParam);
-      String format = new IdentifyFile().Identify(resolvedFile, protocol);
-      dbgMsg("File identified as '" + format + "'");
       AlignmentI al = null;
       try
       {
+        FileFormatI format = new IdentifyFile().identify(resolvedFile,
+                protocol);
+        dbgMsg("File identified as '" + format + "'");
         al = new AppletFormatAdapter().readFile(resolvedFile, protocol,
                 format);
         if ((al != null) && (al.getHeight() > 0))
@@ -1988,16 +2064,16 @@ public class JalviewLite extends Applet implements
           // update the focus.
           currentAlignFrame = newAlignFrame;
 
-          if (protocol == AppletFormatAdapter.PASTE)
+          if (protocol == DataSourceType.PASTE)
           {
-            newAlignFrame.setTitle(MessageManager.formatMessage(
-                    "label.sequences_from", new Object[] { applet
-                            .getDocumentBase().toString() }));
+            newAlignFrame.setTitle(MessageManager
+                    .formatMessage("label.sequences_from", new Object[]
+                    { applet.getDocumentBase().toString() }));
           }
 
           newAlignFrame.statusBar.setText(MessageManager.formatMessage(
-                  "label.successfully_loaded_file",
-                  new Object[] { resolvedFile }));
+                  "label.successfully_loaded_file", new Object[]
+                  { resolvedFile }));
 
           return newAlignFrame;
         }
@@ -2020,8 +2096,8 @@ public class JalviewLite extends Applet implements
             dbgMsg(">>>Dump finished.");
           } catch (Exception e)
           {
-            System.err
-                    .println("Exception when trying to dump the content of the file parameter.");
+            jalview.bin.Console.errPrintln(
+                    "Exception when trying to dump the content of the file parameter.");
             e.printStackTrace();
           }
         }
@@ -2044,8 +2120,8 @@ public class JalviewLite extends Applet implements
        * related to JAL-434
        */
 
-      applet.setAlignPdbStructures(getDefaultParameter("alignpdbfiles",
-              false));
+      applet.setAlignPdbStructures(
+              getDefaultParameter("alignpdbfiles", false));
       /*
        * <param name="PDBfile" value="1gaq.txt PDB|1GAQ|1GAQ|A PDB|1GAQ|1GAQ|B
        * PDB|1GAQ|1GAQ|C">
@@ -2061,9 +2137,11 @@ public class JalviewLite extends Applet implements
       Vector pdbs = new Vector();
       // create a lazy matcher if we're asked to
       jalview.analysis.SequenceIdMatcher matcher = (applet
-              .getDefaultParameter("relaxedidmatch", false)) ? new jalview.analysis.SequenceIdMatcher(
-              alignFrame.getAlignViewport().getAlignment()
-                      .getSequencesArray()) : null;
+              .getDefaultParameter("relaxedidmatch", false))
+                      ? new jalview.analysis.SequenceIdMatcher(
+                              alignFrame.getAlignViewport().getAlignment()
+                                      .getSequencesArray())
+                      : null;
 
       String param;
       do
@@ -2092,8 +2170,9 @@ public class JalviewLite extends Applet implements
             String sequence = applet.getParameter("PDBSEQ");
             if (sequence != null)
             {
-              seqs = new SequenceI[] { matcher == null ? (Sequence) alignFrame
-                      .getAlignViewport().getAlignment().findName(sequence)
+              seqs = new SequenceI[] { matcher == null
+                      ? (Sequence) alignFrame.getAlignViewport()
+                              .getAlignment().findName(sequence)
                       : matcher.findIdMatch(sequence) };
             }
 
@@ -2101,8 +2180,8 @@ public class JalviewLite extends Applet implements
           else
           {
             param = st.nextToken();
-            List<SequenceI> tmp = new ArrayList<SequenceI>();
-            List<String> tmp2 = new ArrayList<String>();
+            List<SequenceI> tmp = new ArrayList<>();
+            List<String> tmp2 = new ArrayList<>();
 
             while (st.hasMoreTokens())
             {
@@ -2114,9 +2193,10 @@ public class JalviewLite extends Applet implements
                 tmp2.add(st2.nextToken());
                 seqstring = st2.nextToken();
               }
-              tmp.add(matcher == null ? (Sequence) alignFrame
-                      .getAlignViewport().getAlignment()
-                      .findName(seqstring) : matcher.findIdMatch(seqstring));
+              tmp.add(matcher == null
+                      ? (Sequence) alignFrame.getAlignViewport()
+                              .getAlignment().findName(seqstring)
+                      : matcher.findIdMatch(seqstring));
             }
 
             seqs = tmp.toArray(new SequenceI[tmp.size()]);
@@ -2136,8 +2216,9 @@ public class JalviewLite extends Applet implements
               if (seqs[i] != null)
               {
                 ((Sequence) seqs[i]).addPDBId(pdb);
-                StructureSelectionManager.getStructureSelectionManager(
-                        applet).registerPDBEntry(pdb);
+                StructureSelectionManager
+                        .getStructureSelectionManager(applet)
+                        .registerPDBEntry(pdb);
               }
               else
               {
@@ -2145,8 +2226,8 @@ public class JalviewLite extends Applet implements
                 {
                   // this may not really be a problem but we give a warning
                   // anyway
-                  System.err
-                          .println("Warning: Possible input parsing error: Null sequence for attachment of PDB (sequence "
+                  jalview.bin.Console.errPrintln(
+                          "Warning: Possible input parsing error: Null sequence for attachment of PDB (sequence "
                                   + i + ")");
                 }
               }
@@ -2159,8 +2240,7 @@ public class JalviewLite extends Applet implements
             }
             else
             {
-              pdbs.addElement(new Object[] { pdb, seqs, chains,
-                  new String(protocol) });
+              pdbs.addElement(new Object[] { pdb, seqs, chains, protocol });
             }
           }
         }
@@ -2173,7 +2253,8 @@ public class JalviewLite extends Applet implements
         PDBEntry[] pdb = new PDBEntry[pdbs.size()];
         String[][] chains = new String[pdbs.size()][];
         String[] protocols = new String[pdbs.size()];
-        for (int pdbsi = 0, pdbsiSize = pdbs.size(); pdbsi < pdbsiSize; pdbsi++)
+        for (int pdbsi = 0, pdbsiSize = pdbs
+                .size(); pdbsi < pdbsiSize; pdbsi++)
         {
           Object[] o = (Object[]) pdbs.elementAt(pdbsi);
           pdb[pdbsi] = (PDBEntry) o[0];
@@ -2199,6 +2280,11 @@ public class JalviewLite extends Applet implements
     {
       boolean result = false;
       String param = applet.getParameter("jnetfile");
+      if (param == null)
+      {
+        // jnet became jpred around 2016
+        param = applet.getParameter("jpredfile");
+      }
       if (param != null)
       {
         try
@@ -2208,12 +2294,9 @@ public class JalviewLite extends Applet implements
           JnetAnnotationMaker.add_annotation(predictions,
                   alignFrame.viewport.getAlignment(), 0, false);
           // false == do not add sequence profile from concise output
-          SequenceI repseq = alignFrame.viewport.getAlignment()
-                  .getSequenceAt(0);
-          alignFrame.viewport.getAlignment().setSeqrep(repseq);
-          ColumnSelection cs = new ColumnSelection();
-          cs.hideInsertionsFor(repseq);
-          alignFrame.viewport.setColumnSelection(cs);
+
+          alignFrame.viewport.getAlignment().setupJPredAlignment();
+
           alignFrame.alignPanel.fontChanged();
           alignFrame.alignPanel.setScrollValues(0, 0);
           result = true;
@@ -2249,8 +2332,8 @@ public class JalviewLite extends Applet implements
         }
         else
         {
-          System.err
-                  .println("Annotations were not added from annotation file '"
+          jalview.bin.Console.errPrintln(
+                  "Annotations were not added from annotation file '"
                           + param + "'");
         }
       }
@@ -2321,14 +2404,14 @@ public class JalviewLite extends Applet implements
         {
           if (debug)
           {
-            System.err
-                    .println("Attempting to load T-COFFEE score file from the scoreFile parameter");
+            jalview.bin.Console.errPrintln(
+                    "Attempting to load T-COFFEE score file from the scoreFile parameter");
           }
           result = alignFrame.loadScoreFile(sScoreFile);
           if (!result)
           {
-            System.err
-                    .println("Failed to parse T-COFFEE parameter as a valid score file ('"
+            jalview.bin.Console.errPrintln(
+                    "Failed to parse T-COFFEE parameter as a valid score file ('"
                             + sScoreFile + "')");
           }
         } catch (Exception e)
@@ -2398,13 +2481,14 @@ public class JalviewLite extends Applet implements
         boolean rtn = (getClass().getResourceAsStream("/" + f) != null);
         if (debug)
         {
-          System.err.println("Resource '" + f + "' was "
+          jalview.bin.Console.errPrintln("Resource '" + f + "' was "
                   + (rtn ? "" : "not ") + "located by classloader.");
         }
         return rtn;
       } catch (Exception ex)
       {
-        System.out.println("Exception checking resources: " + f + " " + ex);
+        jalview.bin.Console.outPrintln(
+                "Exception checking resources: " + f + " " + ex);
         return false;
       }
     }
@@ -2425,8 +2509,8 @@ public class JalviewLite extends Applet implements
     {
       return initialAlignFrame;
     }
-    System.err
-            .println("Implementation error: Jalview Applet API cannot work out which AlignFrame to use.");
+    jalview.bin.Console.errPrintln(
+            "Implementation error: Jalview Applet API cannot work out which AlignFrame to use.");
     return null;
   }
 
@@ -2493,19 +2577,19 @@ public class JalviewLite extends Applet implements
       jv.removeAllElements();
       if (debug)
       {
-        System.err.println("Array from '" + separator
+        jalview.bin.Console.errPrintln("Array from '" + separator
                 + "' separated List:\n" + v.length);
         for (int i = 0; i < v.length; i++)
         {
-          System.err.println("item " + i + " '" + v[i] + "'");
+          jalview.bin.Console.errPrintln("item " + i + " '" + v[i] + "'");
         }
       }
       return v;
     }
     if (debug)
     {
-      System.err.println("Empty Array from '" + separator
-              + "' separated List");
+      jalview.bin.Console.errPrintln(
+              "Empty Array from '" + separator + "' separated List");
     }
     return null;
   }
@@ -2547,16 +2631,16 @@ public class JalviewLite extends Applet implements
       }
       if (debug)
       {
-        System.err.println("Returning '" + separator
-                + "' separated List:\n");
-        System.err.println(v);
+        System.err
+                .println("Returning '" + separator + "' separated List:\n");
+        jalview.bin.Console.errPrintln(v);
       }
       return v.toString();
     }
     if (debug)
     {
-      System.err.println("Returning empty '" + separator
-              + "' separated List\n");
+      jalview.bin.Console.errPrintln(
+              "Returning empty '" + separator + "' separated List\n");
     }
     return "" + separator;
   }
@@ -2566,10 +2650,11 @@ public class JalviewLite extends Applet implements
    * 
    * @see jalview.bin.JalviewLiteJsApi#getFeatureGroups()
    */
+  @Override
   public String getFeatureGroups()
   {
-    String lst = arrayToSeparatorList(getDefaultTargetFrame()
-            .getFeatureGroups());
+    String lst = arrayToSeparatorList(
+            getDefaultTargetFrame().getFeatureGroups());
     return lst;
   }
 
@@ -2580,6 +2665,7 @@ public class JalviewLite extends Applet implements
    * jalview.bin.JalviewLiteJsApi#getFeatureGroupsOn(jalview.appletgui.AlignFrame
    * )
    */
+  @Override
   public String getFeatureGroupsOn(AlignFrame alf)
   {
     String lst = arrayToSeparatorList(alf.getFeatureGroups());
@@ -2591,10 +2677,11 @@ public class JalviewLite extends Applet implements
    * 
    * @see jalview.bin.JalviewLiteJsApi#getFeatureGroupsOfState(boolean)
    */
+  @Override
   public String getFeatureGroupsOfState(boolean visible)
   {
-    return arrayToSeparatorList(getDefaultTargetFrame()
-            .getFeatureGroupsOfState(visible));
+    return arrayToSeparatorList(
+            getDefaultTargetFrame().getFeatureGroupsOfState(visible));
   }
 
   /*
@@ -2604,6 +2691,7 @@ public class JalviewLite extends Applet implements
    * jalview.bin.JalviewLiteJsApi#getFeatureGroupsOfStateOn(jalview.appletgui
    * .AlignFrame, boolean)
    */
+  @Override
   public String getFeatureGroupsOfStateOn(AlignFrame alf, boolean visible)
   {
     return arrayToSeparatorList(alf.getFeatureGroupsOfState(visible));
@@ -2615,11 +2703,12 @@ public class JalviewLite extends Applet implements
    * @see jalview.bin.JalviewLiteJsApi#setFeatureGroupStateOn(jalview.appletgui.
    * AlignFrame, java.lang.String, boolean)
    */
+  @Override
   public void setFeatureGroupStateOn(final AlignFrame alf,
           final String groups, boolean state)
   {
     final boolean st = state;// !(state==null || state.equals("") ||
-    // state.toLowerCase().equals("false"));
+    // state.toLowerCase(Locale.ROOT).equals("false"));
     java.awt.EventQueue.invokeLater(new Runnable()
     {
       @Override
@@ -2636,6 +2725,7 @@ public class JalviewLite extends Applet implements
    * @see jalview.bin.JalviewLiteJsApi#setFeatureGroupState(java.lang.String,
    * boolean)
    */
+  @Override
   public void setFeatureGroupState(String groups, boolean state)
   {
     setFeatureGroupStateOn(getDefaultTargetFrame(), groups, state);
@@ -2646,6 +2736,7 @@ public class JalviewLite extends Applet implements
    * 
    * @see jalview.bin.JalviewLiteJsApi#getSeparator()
    */
+  @Override
   public String getSeparator()
   {
     return separator;
@@ -2656,6 +2747,7 @@ public class JalviewLite extends Applet implements
    * 
    * @see jalview.bin.JalviewLiteJsApi#setSeparator(java.lang.String)
    */
+  @Override
   public void setSeparator(String separator)
   {
     if (separator == null || separator.length() < 1)
@@ -2666,7 +2758,8 @@ public class JalviewLite extends Applet implements
     this.separator = separator;
     if (debug)
     {
-      System.err.println("Default Separator now: '" + separator + "'");
+      jalview.bin.Console
+              .errPrintln("Default Separator now: '" + separator + "'");
     }
   }
 
@@ -2700,6 +2793,7 @@ public class JalviewLite extends Applet implements
    * @see jalview.bin.JalviewLiteJsApi#addPdbFile(jalview.appletgui.AlignFrame,
    * java.lang.String, java.lang.String, java.lang.String)
    */
+  @Override
   public boolean addPdbFile(AlignFrame alFrame, String sequenceId,
           String pdbEntryString, String pdbFile)
   {
@@ -2716,14 +2810,15 @@ public class JalviewLite extends Applet implements
     return alignPdbStructures;
   }
 
+  @Override
   public void start()
   {
     // callInitCallback();
   }
 
-  private Hashtable<String, long[]> jshashes = new Hashtable<String, long[]>();
+  private Hashtable<String, long[]> jshashes = new Hashtable<>();
 
-  private Hashtable<String, Hashtable<String, String[]>> jsmessages = new Hashtable<String, Hashtable<String, String[]>>();
+  private Hashtable<String, Hashtable<String, String[]>> jsmessages = new Hashtable<>();
 
   public void setJsMessageSet(String messageclass, String viewId,
           String[] colcommands)
@@ -2731,7 +2826,7 @@ public class JalviewLite extends Applet implements
     Hashtable<String, String[]> msgset = jsmessages.get(messageclass);
     if (msgset == null)
     {
-      msgset = new Hashtable<String, String[]>();
+      msgset = new Hashtable<>();
       jsmessages.put(messageclass, msgset);
     }
     msgset.put(viewId, colcommands);
@@ -2749,6 +2844,7 @@ public class JalviewLite extends Applet implements
    * @see jalview.bin.JalviewLiteJsApi#getJsMessage(java.lang.String,
    * java.lang.String)
    */
+  @Override
   public String getJsMessage(String messageclass, String viewId)
   {
     Hashtable<String, String[]> msgset = jsmessages.get(messageclass);
@@ -2816,22 +2912,13 @@ public class JalviewLite extends Applet implements
     {
       return defcolour;
     }
-    Color col = jalview.schemes.ColourSchemeProperty
-            .getAWTColorFromName(colprop);
+    Color col = ColorUtils.parseColourString(colprop);
     if (col == null)
     {
-      try
-      {
-        col = new jalview.schemes.UserColourScheme(colprop).findColour('A');
-      } catch (Exception ex)
-      {
-        System.err.println("Couldn't parse '" + colprop
-                + "' as a colour for " + colparam);
-        col = null;
-      }
+      jalview.bin.Console.errPrintln("Couldn't parse '" + colprop
+              + "' as a colour for " + colparam);
     }
     return (col == null) ? defcolour : col;
-
   }
 
   public void openJalviewHelpUrl()
@@ -2864,8 +2951,7 @@ public class JalviewLite extends Applet implements
       String codebase = localref.toString();
       String localfile = localref.getFile();
       resolvedPath = codebase.substring(0,
-              codebase.length() - localfile.length())
-              + targetPath;
+              codebase.length() - localfile.length()) + targetPath;
       return resolvedPath;
     }
 
@@ -2899,8 +2985,8 @@ public class JalviewLite extends Applet implements
     }
     if (debug)
     {
-      System.err.println("resolveUrlForLocalOrAbsolute returning "
-              + resolvedPath);
+      jalview.bin.Console.errPrintln(
+              "resolveUrlForLocalOrAbsolute returning " + resolvedPath);
     }
     return resolvedPath;
   }
@@ -2922,24 +3008,22 @@ public class JalviewLite extends Applet implements
         // form valid URL
         // Should really use docbase, not codebase.
         URL prepend;
-        url = resolveUrlForLocalOrAbsolute(
-                url,
-                prepend = getDefaultParameter("resolvetocodebase", false) ? getCodeBase()
+        url = resolveUrlForLocalOrAbsolute(url,
+                prepend = getDefaultParameter("resolvetocodebase", false)
+                        ? getCodeBase()
                         : getDocumentBase());
         if (debug)
         {
-          System.err
-                  .println("Show url (prepended "
-                          + prepend
-                          + " - toggle resolvetocodebase if code/docbase resolution is wrong): "
-                          + url);
+          jalview.bin.Console.errPrintln("Show url (prepended " + prepend
+                  + " - toggle resolvetocodebase if code/docbase resolution is wrong): "
+                  + url);
         }
       }
       else
       {
         if (debug)
         {
-          System.err.println("Show url: " + url);
+          jalview.bin.Console.errPrintln("Show url: " + url);
         }
       }
       if (url.indexOf("javascript:") == 0)