From: Jim Procter <jprocter@issues.jalview.org>
Date: Mon, 4 Apr 2016 14:48:42 +0000 (+0100)
Subject: JAL-2001 - note about possible side effect on JalviewLite.selectIn due to refactored... 
X-Git-Tag: Release_2_10_0~270^2^2~4
X-Git-Url: http://source.jalview.org/gitweb/?a=commitdiff_plain;h=cb250d2031d928304db5bed196f5bdf816ad79d6;p=jalview.git

JAL-2001 - note about possible side effect on JalviewLite.selectIn due to refactored columnSelection - maybe need a test here
---

diff --git a/src/jalview/bin/JalviewLite.java b/src/jalview/bin/JalviewLite.java
index ae84ba5..095cfd3 100644
--- a/src/jalview/bin/JalviewLite.java
+++ b/src/jalview/bin/JalviewLite.java
@@ -459,6 +459,9 @@ public class JalviewLite extends Applet implements
         if (csel != null)
         {
           List<Integer> cs = csel.getSelected();
+          // note - the following actually clears cs as well, since
+          // csel.getSelected returns a reference. Need to check if we need to
+          // have a concurrentModification exception thrown here
           csel.clear();
           for (Integer selectedCol : cs)
           {