From ddea0f030b4a1dc0cdb249345973ce59da9a0072 Mon Sep 17 00:00:00 2001
From: jprocter <Jim Procter>
Date: Sun, 15 Oct 2006 12:44:45 +0000
Subject: [PATCH] fixed array out of bounds exception in copy constructor

---
 src/jalview/datamodel/ColumnSelection.java |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/jalview/datamodel/ColumnSelection.java b/src/jalview/datamodel/ColumnSelection.java
index c35ff46..60d7aea 100644
--- a/src/jalview/datamodel/ColumnSelection.java
+++ b/src/jalview/datamodel/ColumnSelection.java
@@ -644,18 +644,18 @@ public class ColumnSelection
       if (copy.selected!=null) {
         selected = new Vector();
         for (int i=0,j=copy.selected.size(); i<j; i++) {
-          selected.setElementAt( ((Integer) copy.selected.elementAt(i)), i);
+          selected.addElement(copy.selected.elementAt(i));
         }
       }
       if (copy.hiddenColumns!=null) {
-        hiddenColumns=new Vector();
+        hiddenColumns=new Vector(copy.hiddenColumns.size());
         for (int i=0,j=copy.hiddenColumns.size(); i<j; i++) {
           int[] rh,cp;
           rh = (int[])copy.hiddenColumns.elementAt(i);
           if (rh!=null) {
             cp = new int[rh.length];
             System.arraycopy(rh, 0, cp, 0, rh.length);
-            hiddenColumns.setElementAt(cp, i);
+            hiddenColumns.addElement(cp);
           }
         }
       }
-- 
1.7.10.2