From: jprocter <Jim Procter>
Date: Sun, 15 Oct 2006 12:44:45 +0000 (+0000)
Subject: fixed array out of bounds exception in copy constructor
X-Git-Tag: Release_2_2~272
X-Git-Url: http://source.jalview.org/gitweb/?a=commitdiff_plain;h=ddea0f030b4a1dc0cdb249345973ce59da9a0072;p=jalview.git

fixed array out of bounds exception in copy constructor
---

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);
           }
         }
       }