From: Jim Procter 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 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) {