formatting
[jalview.git] / src / jalview / viewmodel / AlignmentViewport.java
index 3cac289..fcb03d8 100644 (file)
@@ -1,18 +1,18 @@
 /*
  * Jalview - A Sequence Alignment Editor and Viewer (Version 2.8)
  * Copyright (C) 2011 J Procter, AM Waterhouse, J Engelhardt, LM Lui, G Barton, M Clamp, S Searle
- * 
+ *
  * This file is part of Jalview.
- * 
+ *
  * Jalview is free software: you can redistribute it and/or
- * modify it under the terms of the GNU General Public License 
+ * modify it under the terms of the GNU General Public License
  * as published by the Free Software Foundation, either version 3 of the License, or (at your option) any later version.
- * 
- * Jalview is distributed in the hope that it will be useful, but 
- * WITHOUT ANY WARRANTY; without even the implied warranty 
- * of MERCHANTABILITY or FITNESS FOR A PARTICULAR 
+ *
+ * Jalview is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty
+ * of MERCHANTABILITY or FITNESS FOR A PARTICULAR
  * PURPOSE.  See the GNU General Public License for more details.
- * 
+ *
  * You should have received a copy of the GNU General Public License along with Jalview.  If not, see <http://www.gnu.org/licenses/>.
  */
 package jalview.viewmodel;
@@ -41,7 +41,9 @@ import java.util.Hashtable;
 import java.util.Vector;
 
 /**
- * base class holding visualization and analysis attributes and common logic for an active alignment view displayed in the GUI
+ * base class holding visualization and analysis attributes and common logic for
+ * an active alignment view displayed in the GUI
+ *
  * @author jimp
  *
  */
@@ -55,7 +57,8 @@ public abstract class AlignmentViewport implements AlignViewportI
   protected String sequenceSetID;
 
   /**
-   * probably unused indicator that view is of a dataset rather than an alignment
+   * probably unused indicator that view is of a dataset rather than an
+   * alignment
    */
   protected boolean isDataset = false;
 
@@ -63,13 +66,11 @@ public abstract class AlignmentViewport implements AlignViewportI
 
   protected ColumnSelection colSel = new ColumnSelection();
 
-
   public boolean autoCalculateConsensus = true;
 
   protected boolean autoCalculateStrucConsensus = true;
 
   protected boolean ignoreGapsInConsensusCalculation = false;
-  
 
   protected ColourSchemeI globalColourScheme = null;
 
@@ -79,12 +80,12 @@ public abstract class AlignmentViewport implements AlignViewportI
     globalColourScheme = cs;
   }
 
+  @Override
   public ColourSchemeI getGlobalColourScheme()
   {
     return globalColourScheme;
   }
 
-
   protected AlignmentAnnotation consensus;
 
   protected AlignmentAnnotation strucConsensus;
@@ -97,31 +98,34 @@ public abstract class AlignmentViewport implements AlignViewportI
 
   protected AlignmentAnnotation[] groupConservation;
 
-  /** 
-   * results of alignment consensus analysis for visible portion of view 
+  /**
+   * results of alignment consensus analysis for visible portion of view
    */
-  protected Hashtable[] hconsensus=null;
+  protected Hashtable[] hconsensus = null;
 
   /**
-   * results of secondary structure base pair consensus for visible portion of view
+   * results of secondary structure base pair consensus for visible portion of
+   * view
    */
-  protected Hashtable[] hStrucConsensus=null;
+  protected Hashtable[] hStrucConsensus = null;
 
   /**
-   * percentage gaps allowed in a column before all amino acid properties should be considered unconserved
+   * percentage gaps allowed in a column before all amino acid properties should
+   * be considered unconserved
    */
   int ConsPercGaps = 25; // JBPNote : This should be a scalable property!
 
-
+  @Override
   public int getConsPercGaps()
   {
     return ConsPercGaps;
   }
+
   @Override
   public void setSequenceConsensusHash(Hashtable[] hconsensus)
   {
-    this.hconsensus=hconsensus;
-    
+    this.hconsensus = hconsensus;
+
   }
 
   @Override
@@ -135,12 +139,14 @@ public abstract class AlignmentViewport implements AlignViewportI
   {
     return hStrucConsensus;
   }
+
   @Override
   public void setRnaStructureConsensusHash(Hashtable[] hStrucConsensus)
   {
-    this.hStrucConsensus=hStrucConsensus;
-    
+    this.hStrucConsensus = hStrucConsensus;
+
   }
+
   @Override
   public AlignmentAnnotation getAlignmentQualityAnnot()
   {
@@ -152,25 +158,20 @@ public abstract class AlignmentViewport implements AlignViewportI
   {
     return conservation;
   }
+
   @Override
   public AlignmentAnnotation getAlignmentConsensusAnnotation()
   {
     return consensus;
   }
+
   @Override
   public AlignmentAnnotation getAlignmentStrucConsensusAnnotation()
   {
     return strucConsensus;
   }
-  
-  protected AlignCalcManagerI calculator=new AlignCalcManager();
-
-  jalview.workers.ConsensusThread consensusThread;
 
-  StrucConsensusThread strucConsensusThread;
-
-
-  private ConservationThread conservationThread;
+  protected AlignCalcManagerI calculator = new AlignCalcManager();
 
   /**
    * trigger update of conservation annotation
@@ -183,8 +184,12 @@ public abstract class AlignmentViewport implements AlignViewportI
     {
       return;
     }
-    
-    calculator.startWorker(conservationThread=new jalview.workers.ConservationThread(this, ap));
+    if (!calculator
+            .startRegisteredWorkersOfClass(jalview.workers.ConservationThread.class))
+    {
+      calculator.registerWorker(new jalview.workers.ConservationThread(
+              this, ap));
+    }
   }
 
   /**
@@ -197,15 +202,19 @@ public abstract class AlignmentViewport implements AlignViewportI
     {
       return;
     }
-    calculator.startWorker(consensusThread = new ConsensusThread(this, ap));
+    if (!calculator.startRegisteredWorkersOfClass(ConsensusThread.class))
+    {
+      calculator.registerWorker(new ConsensusThread(this, ap));
+    }
   }
 
   // --------START Structure Conservation
   public void updateStrucConsensus(final AlignmentViewPanel ap)
   {
-    if (autoCalculateStrucConsensus && strucConsensus==null && alignment.isNucleotide() && alignment.hasRNAStructure())
+    if (autoCalculateStrucConsensus && strucConsensus == null
+            && alignment.isNucleotide() && alignment.hasRNAStructure())
     {
-      
+
     }
 
     // see note in mantis : issue number 8585
@@ -213,7 +222,11 @@ public abstract class AlignmentViewport implements AlignViewportI
     {
       return;
     }
-    calculator.startWorker(strucConsensusThread = new StrucConsensusThread(this,ap));
+    if (!calculator
+            .startRegisteredWorkersOfClass(StrucConsensusThread.class))
+    {
+      calculator.registerWorker(new StrucConsensusThread(this, ap));
+    }
   }
 
   public boolean isCalcInProgress()
@@ -221,25 +234,26 @@ public abstract class AlignmentViewport implements AlignViewportI
     return calculator.isWorking();
   }
 
+  @Override
   public boolean isCalculationInProgress(
           AlignmentAnnotation alignmentAnnotation)
   {
     if (!alignmentAnnotation.autoCalculated)
       return false;
-    if ((calculator.isWorking(consensusThread) && consensus==alignmentAnnotation)
-            || (calculator.isWorking(conservationThread) &&  (conservation==alignmentAnnotation || quality==alignmentAnnotation))
-            || (calculator.isWorking(strucConsensusThread) && strucConsensus==alignmentAnnotation)
-            )
+    if (calculator.workingInvolvedWith(alignmentAnnotation))
     {
+      // System.err.println("grey out ("+alignmentAnnotation.label+")");
       return true;
     }
     return false;
   }
+
   @Override
   public boolean isClosed()
   {
-    // TODO: check that this isClosed is only true after panel is closed, not before it is fully constructed.
-    return alignment==null;
+    // TODO: check that this isClosed is only true after panel is closed, not
+    // before it is fully constructed.
+    return alignment == null;
   }
 
   @Override
@@ -262,10 +276,12 @@ public abstract class AlignmentViewport implements AlignViewportI
    * should consensus profile be rendered by default
    */
   protected boolean showSequenceLogo = false;
+
   /**
    * should consensus profile be rendered normalised to row height
    */
   protected boolean normaliseSequenceLogo = false;
+
   /**
    * should consensus histograms be rendered by default
    */
@@ -274,6 +290,7 @@ public abstract class AlignmentViewport implements AlignViewportI
   /**
    * @return the showConsensusProfile
    */
+  @Override
   public boolean isShowSequenceLogo()
   {
     return showSequenceLogo;
@@ -290,14 +307,8 @@ public abstract class AlignmentViewport implements AlignViewportI
       // TODO: decouple settings setting from calculation when refactoring
       // annotation update method from alignframe to viewport
       this.showSequenceLogo = showSequenceLogo;
-      if (consensusThread != null)
-      {
-        consensusThread.updateAnnotation();
-      }
-      if (strucConsensusThread != null)
-      {
-        strucConsensusThread.updateAnnotation();
-      }
+      calculator.updateAnnotationFor(ConsensusThread.class);
+      calculator.updateAnnotationFor(StrucConsensusThread.class);
     }
     this.showSequenceLogo = showSequenceLogo;
   }
@@ -346,10 +357,11 @@ public abstract class AlignmentViewport implements AlignViewportI
   }
 
   /**
-   * 
+   *
    * @return flag to indicate if the consensus histogram should be rendered by
    *         default
    */
+  @Override
   public boolean isShowConsensusHistogram()
   {
     return this.showConsensusHistogram;
@@ -360,7 +372,6 @@ public abstract class AlignmentViewport implements AlignViewportI
    */
   protected boolean showUnconserved = false;
 
-
   /**
    * when set, updateAlignment will always ensure sequences are of equal length
    */
@@ -391,8 +402,8 @@ public abstract class AlignmentViewport implements AlignViewportI
   }
 
   /**
-   * 
-   * 
+   *
+   *
    * @return null or the currently selected sequence region
    */
   public SequenceGroup getSelectionGroup()
@@ -402,10 +413,10 @@ public abstract class AlignmentViewport implements AlignViewportI
 
   /**
    * Set the selection group for this window.
-   * 
+   *
    * @param sg
    *          - group holding references to sequences in this alignment view
-   * 
+   *
    */
   public void setSelectionGroup(SequenceGroup sg)
   {
@@ -421,13 +432,15 @@ public abstract class AlignmentViewport implements AlignViewportI
     }
   }
 
+  @Override
   public ColumnSelection getColumnSelection()
   {
     return colSel;
   }
+
   public void setColumnSelection(ColumnSelection colSel)
   {
-    this.colSel=colSel;
+    this.colSel = colSel;
   }
   public Hashtable getHiddenRepSequences()
   {
@@ -441,12 +454,13 @@ public abstract class AlignmentViewport implements AlignViewportI
 
   public void updateHiddenColumns()
   {
-    hasHiddenColumns = colSel.getHiddenColumns() != null;  
+    hasHiddenColumns = colSel.getHiddenColumns() != null;
   }
-  
+
   protected boolean hasHiddenRows = false;
-  
-  public boolean hasHiddenRows() {
+
+  public boolean hasHiddenRows()
+  {
     return hasHiddenRows;
   }
 
@@ -454,12 +468,14 @@ public abstract class AlignmentViewport implements AlignViewportI
 
   public void setSequenceSetId(String newid)
   {
-    if (sequenceSetID!=null)
+    if (sequenceSetID != null)
     {
-      System.err.println("Warning - overwriting a sequenceSetId for a viewport!");
+      System.err
+              .println("Warning - overwriting a sequenceSetId for a viewport!");
     }
-    sequenceSetID=new String(newid);
+    sequenceSetID = new String(newid);
   }
+
   public String getSequenceSetId()
   {
     if (sequenceSetID == null)
@@ -469,9 +485,10 @@ public abstract class AlignmentViewport implements AlignViewportI
 
     return sequenceSetID;
   }
+
   /**
    * unique viewId for synchronizing state (e.g. with stored Jalview Project)
-   * 
+   *
    */
   protected String viewId = null;
 
@@ -483,26 +500,31 @@ public abstract class AlignmentViewport implements AlignViewportI
     }
     return viewId;
   }
+
   public void setIgnoreGapsConsensus(boolean b, AlignmentViewPanel ap)
   {
     ignoreGapsInConsensusCalculation = b;
-    if (ap!=null) {updateConsensus(ap);
-    if (globalColourScheme != null)
+    if (ap != null)
     {
-      globalColourScheme.setThreshold(globalColourScheme.getThreshold(),
-              ignoreGapsInConsensusCalculation);
-    }}
-    
+      updateConsensus(ap);
+      if (globalColourScheme != null)
+      {
+        globalColourScheme.setThreshold(globalColourScheme.getThreshold(),
+                ignoreGapsInConsensusCalculation);
+      }
+    }
+
   }
+
   private long sgrouphash = -1, colselhash = -1;
 
   /**
    * checks current SelectionGroup against record of last hash value, and
    * updates record.
-   * 
+   *
    * @param b
    *          update the record of last hash value
-   * 
+   *
    * @return true if SelectionGroup changed since last call (when b is true)
    */
   public boolean isSelectionGroupChanged(boolean b)
@@ -523,7 +545,7 @@ public abstract class AlignmentViewport implements AlignViewportI
   /**
    * checks current colsel against record of last hash value, and optionally
    * updates record.
-   * 
+   *
    * @param b
    *          update the record of last hash value
    * @return true if colsel changed since last call (when b is true)
@@ -543,15 +565,16 @@ public abstract class AlignmentViewport implements AlignViewportI
     return false;
   }
 
+  @Override
   public boolean getIgnoreGapsConsensus()
   {
     return ignoreGapsInConsensusCalculation;
   }
 
-  /// property change stuff
+  // / property change stuff
 
   // JBPNote Prolly only need this in the applet version.
-  private java.beans.PropertyChangeSupport changeSupport = new java.beans.PropertyChangeSupport(
+  private final java.beans.PropertyChangeSupport changeSupport = new java.beans.PropertyChangeSupport(
           this);
 
   protected boolean showConservation = true;
@@ -560,10 +583,9 @@ public abstract class AlignmentViewport implements AlignViewportI
 
   protected boolean showConsensus = true;
 
-
   /**
    * Property change listener for changes in alignment
-   * 
+   *
    * @param listener
    *          DOCUMENT ME!
    */
@@ -575,7 +597,7 @@ public abstract class AlignmentViewport implements AlignViewportI
 
   /**
    * DOCUMENT ME!
-   * 
+   *
    * @param listener
    *          DOCUMENT ME!
    */
@@ -587,7 +609,7 @@ public abstract class AlignmentViewport implements AlignViewportI
 
   /**
    * Property change listener for changes in alignment
-   * 
+   *
    * @param prop
    *          DOCUMENT ME!
    * @param oldvalue
@@ -602,7 +624,6 @@ public abstract class AlignmentViewport implements AlignViewportI
   }
 
   // common hide/show column stuff
-  
 
   public void hideSelectedColumns()
   {
@@ -646,7 +667,6 @@ public abstract class AlignmentViewport implements AlignViewportI
     hasHiddenColumns = false;
   }
 
-  
   // common hide/show seq stuff
   public void showAllHiddenSeqs()
   {
@@ -668,7 +688,8 @@ public abstract class AlignmentViewport implements AlignViewportI
       hiddenRepSequences = null;
 
       firePropertyChange("alignment", null, alignment.getSequences());
-      // used to set hasHiddenRows/hiddenRepSequences here, after the property changed event
+      // used to set hasHiddenRows/hiddenRepSequences here, after the property
+      // changed event
       sendSelection();
     }
   }
@@ -689,7 +710,8 @@ public abstract class AlignmentViewport implements AlignViewportI
       {
         selectionGroup.addSequence((SequenceI) tmp.elementAt(t), false);
       }
-      // JBPNote: refactor: only update flag if we modified visiblity (used to do this regardless) 
+      // JBPNote: refactor: only update flag if we modified visiblity (used to
+      // do this regardless)
       if (alignment.getHiddenSequences().getSize() < 1)
       {
         hasHiddenRows = false;
@@ -699,8 +721,6 @@ public abstract class AlignmentViewport implements AlignViewportI
     }
   }
 
-
-  
   public void hideAllSelectedSeqs()
   {
     if (selectionGroup == null || selectionGroup.getSize() < 1)
@@ -714,7 +734,6 @@ public abstract class AlignmentViewport implements AlignViewportI
 
     setSelectionGroup(null);
   }
-  
 
   public void hideSequence(SequenceI[] seq)
   {
@@ -759,7 +778,7 @@ public abstract class AlignmentViewport implements AlignViewportI
         seqs[index++] = sg.getSequenceAt(i);
       }
     }
-    sg.setSeqrep(repSequence); // note: not done in 2.7applet 
+    sg.setSeqrep(repSequence); // note: not done in 2.7applet
     sg.setHidereps(true); // note: not done in 2.7applet
     hideSequence(seqs);
 
@@ -768,11 +787,13 @@ public abstract class AlignmentViewport implements AlignViewportI
   public boolean isHiddenRepSequence(SequenceI seq)
   {
     return hiddenRepSequences != null
-          && hiddenRepSequences.containsKey(seq);
+            && hiddenRepSequences.containsKey(seq);
   }
+
   public SequenceGroup getRepresentedSequences(SequenceI seq)
   {
-    return (SequenceGroup) (hiddenRepSequences == null ? null : hiddenRepSequences.get(seq));
+    return (SequenceGroup) (hiddenRepSequences == null ? null
+            : hiddenRepSequences.get(seq));
   }
 
   public int adjustForHiddenSeqs(int alignmentIndex)
@@ -786,19 +807,17 @@ public abstract class AlignmentViewport implements AlignViewportI
    * broadcast selection to any interested parties
    */
   public abstract void sendSelection();
-  
 
   public void invertColumnSelection()
   {
     colSel.invertColumnSelection(0, alignment.getWidth());
   }
 
-
   /**
    * This method returns an array of new SequenceI objects derived from the
    * whole alignment or just the current selection with start and end points
    * adjusted
-   * 
+   *
    * @note if you need references to the actual SequenceI objects in the
    *       alignment or currently selected then use getSequenceSelection()
    * @return selection as new sequenceI objects
@@ -806,8 +825,11 @@ public abstract class AlignmentViewport implements AlignViewportI
   public SequenceI[] getSelectionAsNewSequence()
   {
     SequenceI[] sequences;
-    // JBPNote: Need to test jalviewLite.getSelectedSequencesAsAlignmentFrom - this was the only caller in the applet for this method
-    // JBPNote: in applet, this method returned references to the alignment sequences, and it did not honour the presence/absence of annotation attached to the alignment (probably!)
+    // JBPNote: Need to test jalviewLite.getSelectedSequencesAsAlignmentFrom -
+    // this was the only caller in the applet for this method
+    // JBPNote: in applet, this method returned references to the alignment
+    // sequences, and it did not honour the presence/absence of annotation
+    // attached to the alignment (probably!)
     if (selectionGroup == null)
     {
       sequences = alignment.getSequencesArray();
@@ -828,11 +850,10 @@ public abstract class AlignmentViewport implements AlignViewportI
     return sequences;
   }
 
-
   /**
    * get the currently selected sequence objects or all the sequences in the
    * alignment.
-   * 
+   *
    * @return array of references to sequence objects
    */
   public SequenceI[] getSequenceSelection()
@@ -849,13 +870,12 @@ public abstract class AlignmentViewport implements AlignViewportI
     return sequences;
   }
 
-
   /**
    * This method returns the visible alignment as text, as seen on the GUI, ie
    * if columns are hidden they will not be returned in the result. Use this for
    * calculating trees, PCA, redundancy etc on views which contain hidden
    * columns.
-   * 
+   *
    * @return String[]
    */
   public jalview.datamodel.CigarArray getViewAsCigars(
@@ -869,7 +889,7 @@ public abstract class AlignmentViewport implements AlignViewportI
   /**
    * return a compact representation of the current alignment selection to pass
    * to an analysis function
-   * 
+   *
    * @param selectedOnly
    *          boolean true to just return the selected view
    * @return AlignmentView
@@ -883,7 +903,7 @@ public abstract class AlignmentViewport implements AlignViewportI
   /**
    * return a compact representation of the current alignment selection to pass
    * to an analysis function
-   * 
+   *
    * @param selectedOnly
    *          boolean true to just return the selected view
    * @param markGroups
@@ -899,13 +919,12 @@ public abstract class AlignmentViewport implements AlignViewportI
             hasHiddenColumns, selectedOnly, markGroups);
   }
 
-
   /**
    * This method returns the visible alignment as text, as seen on the GUI, ie
    * if columns are hidden they will not be returned in the result. Use this for
    * calculating trees, PCA, redundancy etc on views which contain hidden
    * columns.
-   * 
+   *
    * @return String[]
    */
   public String[] getViewAsString(boolean selectedRegionOnly)
@@ -944,11 +963,13 @@ public abstract class AlignmentViewport implements AlignViewportI
     return selection;
   }
 
-
   /**
    * return visible region boundaries within given column range
-   * @param min first column (inclusive, from 0)
-   * @param max last column (exclusive)
+   *
+   * @param min
+   *          first column (inclusive, from 0)
+   * @param max
+   *          last column (exclusive)
    * @return int[][] range of {start,end} visible positions
    */
   public int[][] getVisibleRegionBoundaries(int min, int max)
@@ -994,6 +1015,7 @@ public abstract class AlignmentViewport implements AlignViewportI
     return startEnd;
 
   }
+
   /**
    * @return the padGaps
    */
@@ -1001,15 +1023,20 @@ public abstract class AlignmentViewport implements AlignViewportI
   {
     return padGaps;
   }
+
   /**
-   * @param padGaps the padGaps to set
+   * @param padGaps
+   *          the padGaps to set
    */
   public void setPadGaps(boolean padGaps)
   {
     this.padGaps = padGaps;
   }
+
   /**
-   * apply any post-edit constraints and trigger any calculations needed after an edit has been performed on the alignment 
+   * apply any post-edit constraints and trigger any calculations needed after
+   * an edit has been performed on the alignment
+   *
    * @param ap
    */
   public void alignmentChanged(AlignmentViewPanel ap)
@@ -1052,7 +1079,7 @@ public abstract class AlignmentViewport implements AlignViewportI
     }
 
     resetAllColourSchemes();
-
+    calculator.restartWorkers();
     // alignment.adjustSequenceAnnotations();
   }
 
@@ -1065,6 +1092,8 @@ public abstract class AlignmentViewport implements AlignViewportI
     ColourSchemeI cs = globalColourScheme;
     if (cs != null)
     {
+      cs.alignmentChanged(alignment);
+      // TODO: fold all recalc events for clustalX into alignmentChanged
       if (cs instanceof ClustalxColourScheme)
       {
         ((ClustalxColourScheme) cs).resetClustalX(alignment.getSequences(),
@@ -1105,48 +1134,53 @@ public abstract class AlignmentViewport implements AlignViewportI
       {
         if (showConservation)
         {
-          if (conservation==null)
-        {
-        conservation = new AlignmentAnnotation("Conservation",
-                "Conservation of total alignment less than " + getConsPercGaps()
-                        + "% gaps", new Annotation[1], 0f, 11f,
-                AlignmentAnnotation.BAR_GRAPH);
-        conservation.hasText = true;
-        conservation.autoCalculated = true;
-          alignment.addAnnotation(conservation);
-        }
+          if (conservation == null)
+          {
+            conservation = new AlignmentAnnotation("Conservation",
+                    "Conservation of total alignment less than "
+                            + getConsPercGaps() + "% gaps",
+                    new Annotation[1], 0f, 11f,
+                    AlignmentAnnotation.BAR_GRAPH);
+            conservation.hasText = true;
+            conservation.autoCalculated = true;
+            alignment.addAnnotation(conservation);
+          }
         }
         if (showQuality)
         {
-          if (quality==null)
+          if (quality == null)
           {
-          quality = new AlignmentAnnotation("Quality",
-                  "Alignment Quality based on Blosum62 scores",
-                  new Annotation[1], 0f, 11f, AlignmentAnnotation.BAR_GRAPH);
-          quality.hasText = true;
-          quality.autoCalculated = true;
-          alignment.addAnnotation(quality);
-        }
+            quality = new AlignmentAnnotation("Quality",
+                    "Alignment Quality based on Blosum62 scores",
+                    new Annotation[1], 0f, 11f,
+                    AlignmentAnnotation.BAR_GRAPH);
+            quality.hasText = true;
+            quality.autoCalculated = true;
+            alignment.addAnnotation(quality);
+          }
         }
-      } else {
+      }
+      else
+      {
         if (alignment.hasRNAStructure())
         {
           strucConsensus = new AlignmentAnnotation("StrucConsensus", "PID",
-                  new Annotation[1], 0f, 100f, AlignmentAnnotation.BAR_GRAPH);
+                  new Annotation[1], 0f, 100f,
+                  AlignmentAnnotation.BAR_GRAPH);
           strucConsensus.hasText = true;
           strucConsensus.autoCalculated = true;
         }
       }
-        
+
       consensus = new AlignmentAnnotation("Consensus", "PID",
               new Annotation[1], 0f, 100f, AlignmentAnnotation.BAR_GRAPH);
       consensus.hasText = true;
       consensus.autoCalculated = true;
-  
+
       if (showConsensus)
       {
         alignment.addAnnotation(consensus);
-        if (strucConsensus!=null)
+        if (strucConsensus != null)
         {
           alignment.addAnnotation(strucConsensus);
         }