X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fworkers%2FConservationThread.java;h=236bccfb59dee09d7869f8e61504f0f4873a4260;hb=ad15cff29620f960119f80176f1fd443da9f6763;hp=47ce6eb5b1fe3cb361432c9c3ba5038124e10a80;hpb=d668b0fe1a9eefb95c935bc6c71a88e67642e712;p=jalview.git

diff --git a/src/jalview/workers/ConservationThread.java b/src/jalview/workers/ConservationThread.java
index 47ce6eb..236bccf 100644
--- a/src/jalview/workers/ConservationThread.java
+++ b/src/jalview/workers/ConservationThread.java
@@ -1,46 +1,55 @@
 /*
- * Jalview - A Sequence Alignment Editor and Viewer (Version 2.7)
- * Copyright (C) 2011 J Procter, AM Waterhouse, J Engelhardt, LM Lui, G Barton, M Clamp, S Searle
- *
+ * Jalview - A Sequence Alignment Editor and Viewer ($$Version-Rel$$)
+ * Copyright (C) $$Year-Rel$$ The Jalview Authors
+ * 
  * 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
- * 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
+ * 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 
  * 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/>.
+ * 
+ * You should have received a copy of the GNU General Public License
+ * along with Jalview.  If not, see <http://www.gnu.org/licenses/>.
+ * The Jalview Authors are detailed in the 'AUTHORS' file.
  */
 package jalview.workers;
 
-import java.util.ArrayList;
-import java.util.List;
-
 import jalview.analysis.Conservation;
 import jalview.api.AlignCalcWorkerI;
-import jalview.api.AlignmentViewPanel;
 import jalview.api.AlignViewportI;
+import jalview.api.AlignmentViewPanel;
 import jalview.datamodel.AlignmentAnnotation;
 import jalview.datamodel.AlignmentI;
 
-public class ConservationThread extends AlignCalcWorker implements AlignCalcWorkerI
+import java.util.ArrayList;
+import java.util.List;
+
+public class ConservationThread extends AlignCalcWorker implements
+        AlignCalcWorkerI
 {
 
-  private int ConsPercGaps = 25; // JBPNote : This should be a configurable property!
+  private int ConsPercGaps = 25; // JBPNote : This should be a configurable
+                                 // property!
 
-  public ConservationThread(AlignViewportI alignViewport, AlignmentViewPanel alignPanel)
+  public ConservationThread(AlignViewportI alignViewport,
+          AlignmentViewPanel alignPanel)
   {
     super(alignViewport, alignPanel);
     ConsPercGaps = alignViewport.getConsPercGaps();
   }
 
   private Conservation cons;
-  AlignmentAnnotation conservation,quality;
+
+  AlignmentAnnotation conservation, quality;
+
   int alWidth;
+
   @Override
   public void run()
   {
@@ -54,7 +63,7 @@ public class ConservationThread extends AlignCalcWorker implements AlignCalcWork
         {
           if (ap != null)
           {
-            ap.paintAlignment(false);
+            // ap.paintAlignment(false);
           }
           Thread.sleep(200);
         } catch (Exception ex)
@@ -62,31 +71,41 @@ public class ConservationThread extends AlignCalcWorker implements AlignCalcWork
           ex.printStackTrace();
         }
       }
-      if (alignViewport.isClosed()) {
+      if (alignViewport.isClosed())
+      {
         abortAndDestroy();
+        return;
       }
-      List<AlignmentAnnotation>ourAnnot = new ArrayList<AlignmentAnnotation>();
-      AlignmentI alignment=alignViewport.getAlignment();
-      conservation=alignViewport.getAlignmentConservationAnnotation();
-      quality=alignViewport.getAlignmentQualityAnnot();
+      List<AlignmentAnnotation> ourAnnot = new ArrayList<AlignmentAnnotation>();
+      AlignmentI alignment = alignViewport.getAlignment();
+      conservation = alignViewport.getAlignmentConservationAnnotation();
+      quality = alignViewport.getAlignmentQualityAnnot();
       ourAnnot.add(conservation);
       ourAnnot.add(quality);
       ourAnnots = ourAnnot;
-
+      ConsPercGaps = alignViewport.getConsPercGaps();
       // AlignViewport.UPDATING_CONSERVATION = true;
 
-      if (alignment==null || (alWidth=alignment.getWidth())< 0)
+      if (alignment == null || (alWidth = alignment.getWidth()) < 0)
       {
         calcMan.workerComplete(this);
-        //.updatingConservation = false;
-        //AlignViewport.UPDATING_CONSERVATION = false;
+        // .updatingConservation = false;
+        // AlignViewport.UPDATING_CONSERVATION = false;
 
         return;
       }
-
-      cons = Conservation.calculateConservation("All",
-              jalview.schemes.ResidueProperties.propHash, 3,
-              alignment.getSequences(), 0, alWidth - 1, false, ConsPercGaps, quality!=null);
+      try
+      {
+        cons = Conservation.calculateConservation("All",
+                jalview.schemes.ResidueProperties.propHash, 3,
+                alignment.getSequences(), 0, alWidth - 1, false,
+                ConsPercGaps, quality != null);
+      } catch (IndexOutOfBoundsException x)
+      {
+        // probable race condition. just finish and return without any fuss.
+        calcMan.workerComplete(this);
+        return;
+      }
       updateResultAnnotation(true);
     } catch (OutOfMemoryError error)
     {
@@ -107,10 +126,14 @@ public class ConservationThread extends AlignCalcWorker implements AlignCalcWork
 
   private void updateResultAnnotation(boolean b)
   {
-    if (b || !calcMan.isWorking(this) && cons!=null && conservation!=null && quality!=null)
-    cons.completeAnnotations(conservation,
-            quality, 0, alWidth);
+    if (b || !calcMan.isWorking(this) && cons != null
+            && conservation != null && quality != null)
+    {
+      alignViewport.setConservation(cons);
+      cons.completeAnnotations(conservation, quality, 0, alWidth);
+    }
   }
+
   @Override
   public void updateAnnotation()
   {