X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fcommands%2FEditCommand.java;h=b39d1ddb69302035d183b18778a8c6939ee39b46;hb=9157d82cb9213d9071c15203dddf96ae9081dfbe;hp=61e2595a4c0e340aae6e02e4c80bfae8ba028954;hpb=d7e33557512a6e6c3c604aeb6ff67c023ee5c251;p=jalview.git

diff --git a/src/jalview/commands/EditCommand.java b/src/jalview/commands/EditCommand.java
index 61e2595..b39d1dd 100644
--- a/src/jalview/commands/EditCommand.java
+++ b/src/jalview/commands/EditCommand.java
@@ -1,104 +1,117 @@
 /*
- * Jalview - A Sequence Alignment Editor and Viewer
- * Copyright (C) 2007 AM Waterhouse, J Procter, G Barton, M Clamp, S Searle
- *
- * This program 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 2
+ * Jalview - A Sequence Alignment Editor and Viewer (Version 2.8.2)
+ * Copyright (C) 2014 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.
- *
- * This program 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.
- *
+ *  
+ * 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 this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA
+ * along with Jalview.  If not, see <http://www.gnu.org/licenses/>.
+ * The Jalview Authors are detailed in the 'AUTHORS' file.
  */
 package jalview.commands;
 
-import java.util.*;
+import jalview.datamodel.AlignmentAnnotation;
+import jalview.datamodel.AlignmentI;
+import jalview.datamodel.Annotation;
+import jalview.datamodel.Sequence;
+import jalview.datamodel.SequenceFeature;
+import jalview.datamodel.SequenceI;
 
-import jalview.datamodel.*;
+import java.util.Hashtable;
+import java.util.List;
 
 /**
- *
- * <p>Title: EditCommmand</p>
- *
- * <p>Description: Essential information for performing
- * undo and redo for cut/paste insert/delete gap
- * which can be stored in the HistoryList </p>
- *
- * <p>Copyright: Copyright (c) 2006</p>
- *
- * <p>Company: Dundee University</p>
- *
+ * 
+ * <p>
+ * Title: EditCommmand
+ * </p>
+ * 
+ * <p>
+ * Description: Essential information for performing undo and redo for cut/paste
+ * insert/delete gap which can be stored in the HistoryList
+ * </p>
+ * 
+ * <p>
+ * Copyright: Copyright (c) 2006
+ * </p>
+ * 
+ * <p>
+ * Company: Dundee University
+ * </p>
+ * 
  * @author not attributable
  * @version 1.0
  */
-public class EditCommand
-    implements CommandI
+public class EditCommand implements CommandI
 {
   public static final int INSERT_GAP = 0;
+
   public static final int DELETE_GAP = 1;
+
   public static final int CUT = 2;
+
   public static final int PASTE = 3;
+
   public static final int REPLACE = 4;
 
+  public static final int INSERT_NUC = 5;
+
   Edit[] edits;
 
   String description;
 
   public EditCommand()
-  {}
+  {
+  }
 
   public EditCommand(String description)
   {
     this.description = description;
   }
 
-  public EditCommand(String description,
-                     int command,
-                     SequenceI[] seqs,
-                     int position,
-                     int number,
-                     AlignmentI al)
+  public EditCommand(String description, int command, SequenceI[] seqs,
+          int position, int number, AlignmentI al)
   {
     this.description = description;
     if (command == CUT || command == PASTE)
     {
       edits = new Edit[]
-          {
-          new Edit(command, seqs, position, number, al)};
+      { new Edit(command, seqs, position, number, al) };
     }
 
     performEdit(0, null);
   }
 
-  public EditCommand(String description,
-                     int command,
-                     String replace,
-                     SequenceI[] seqs,
-                     int position,
-                     int number,
-                     AlignmentI al)
+  public EditCommand(String description, int command, String replace,
+          SequenceI[] seqs, int position, int number, AlignmentI al)
   {
     this.description = description;
     if (command == REPLACE)
     {
       edits = new Edit[]
-          { new Edit(command, seqs, position, number, al, replace)};
+      { new Edit(command, seqs, position, number, al, replace) };
     }
 
     performEdit(0, null);
   }
 
+  @Override
   final public String getDescription()
   {
     return description;
   }
 
+  @Override
   public int getSize()
   {
     return edits == null ? 0 : edits.length;
@@ -110,9 +123,11 @@ public class EditCommand
   }
 
   /**
-   * append a new editCommand
-   * Note. this shouldn't be called if the edit is an operation affects more alignment objects than the one referenced
-   * in al (for example, cut or pasting whole sequences). Use the form with an additional AlignmentI[] views parameter.
+   * append a new editCommand Note. this shouldn't be called if the edit is an
+   * operation affects more alignment objects than the one referenced in al (for
+   * example, cut or pasting whole sequences). Use the form with an additional
+   * AlignmentI[] views parameter.
+   * 
    * @param command
    * @param seqs
    * @param position
@@ -120,17 +135,16 @@ public class EditCommand
    * @param al
    * @param performEdit
    */
-  final public void appendEdit(int command,
-          SequenceI[] seqs,
-          int position,
-          int number,
-          AlignmentI al,
-          boolean performEdit)
+  final public void appendEdit(int command, SequenceI[] seqs, int position,
+          int number, AlignmentI al, boolean performEdit)
   {
     appendEdit(command, seqs, position, number, al, performEdit, null);
   }
+
   /**
-   * append a new edit command with a set of alignment views that may be operated on
+   * append a new edit command with a set of alignment views that may be
+   * operated on
+   * 
    * @param command
    * @param seqs
    * @param position
@@ -139,14 +153,11 @@ public class EditCommand
    * @param performEdit
    * @param views
    */
-  final public void appendEdit(int command,
-                               SequenceI[] seqs,
-                               int position,
-                               int number,
-                               AlignmentI al,
-                               boolean performEdit, AlignmentI[] views)
+  final public void appendEdit(int command, SequenceI[] seqs, int position,
+          int number, AlignmentI al, boolean performEdit, AlignmentI[] views)
   {
-    Edit edit = new Edit(command, seqs, position, number, al.getGapCharacter());
+    Edit edit = new Edit(command, seqs, position, number,
+            al.getGapCharacter());
     if (al.getHeight() == seqs.length)
     {
       edit.al = al;
@@ -163,8 +174,7 @@ public class EditCommand
     else
     {
       edits = new Edit[]
-          {
-          edit};
+      { edit };
     }
 
     if (performEdit)
@@ -178,55 +188,59 @@ public class EditCommand
     int eSize = edits.length;
     for (int e = commandIndex; e < eSize; e++)
     {
-      switch(edits[e].command)
+      switch (edits[e].command)
       {
-        case INSERT_GAP:
+      case INSERT_GAP:
         insertGap(edits[e]);
-          break;
-        case DELETE_GAP:
+        break;
+      case DELETE_GAP:
         deleteGap(edits[e]);
-          break;
-        case CUT:
+        break;
+      case CUT:
         cut(edits[e], views);
-          break;
-        case PASTE:
+        break;
+      case PASTE:
         paste(edits[e], views);
-          break;
-        case REPLACE:
-          replace(edits[e]);
-          break;
+        break;
+      case REPLACE:
+        replace(edits[e]);
+        break;
+      // TODO:add deleteNuc for UNDO
+      // case INSERT_NUC:
+      // insertNuc(edits[e]);
+      // break;
       }
     }
   }
 
+  @Override
   final public void doCommand(AlignmentI[] views)
   {
-    performEdit(0,views);
+    performEdit(0, views);
   }
 
+  @Override
   final public void undoCommand(AlignmentI[] views)
-  {
-    int e = 0, eSize = edits.length;
-    for (e = eSize - 1; e > -1; e--)
-    {
-      switch (edits[e].command)
-      {
+  { 
+    for(Edit e : edits){
+    	switch (e.command)
+        {
         case INSERT_GAP:
-        deleteGap(edits[e]);
+          deleteGap(e);
           break;
         case DELETE_GAP:
-        insertGap(edits[e]);
+          insertGap(e);
           break;
         case CUT:
-        paste(edits[e], views);
+          paste(e, views);
           break;
         case PASTE:
-        cut(edits[e], views);
+          cut(e, views);
           break;
         case REPLACE:
-          replace(edits[e]);
+          replace(e);
           break;
-      }
+        }
     }
   }
 
@@ -235,20 +249,33 @@ public class EditCommand
 
     for (int s = 0; s < command.seqs.length; s++)
     {
-      command.seqs[s].insertCharAt(command.position,
-                                   command.number,
-                                   command.gapChar);
+      command.seqs[s].insertCharAt(command.position, command.number,
+              command.gapChar);
+      // System.out.println("pos: "+command.position+" number: "+command.number);
     }
 
     adjustAnnotations(command, true, false, null);
   }
 
+  //
+  // final void insertNuc(Edit command)
+  // {
+  //
+  // for (int s = 0; s < command.seqs.length; s++)
+  // {
+  // System.out.println("pos: "+command.position+" number: "+command.number);
+  // command.seqs[s].insertCharAt(command.position, command.number,'A');
+  // }
+  //
+  // adjustAnnotations(command, true, false, null);
+  // }
+
   final void deleteGap(Edit command)
   {
     for (int s = 0; s < command.seqs.length; s++)
     {
-      command.seqs[s].deleteChars(command.position,
-                                  command.position + command.number);
+      command.seqs[s].deleteChars(command.position, command.position
+              + command.number);
     }
 
     adjustAnnotations(command, false, false, null);
@@ -256,7 +283,7 @@ public class EditCommand
 
   void cut(Edit command, AlignmentI[] views)
   {
-    boolean seqDeleted=false;
+    boolean seqDeleted = false;
     command.string = new char[command.seqs.length][];
 
     for (int i = 0; i < command.seqs.length; i++)
@@ -264,34 +291,46 @@ public class EditCommand
       if (command.seqs[i].getLength() > command.position)
       {
         command.string[i] = command.seqs[i].getSequence(command.position,
-            command.position + command.number);
-
-        if (command.seqs[i].getDatasetSequence() != null
-            || command.seqs[i].getSequenceFeatures() != null)
+                command.position + command.number);
+        SequenceI oldds = command.seqs[i].getDatasetSequence();
+        if (command.oldds != null && command.oldds[i] != null)
+        {
+          // we are redoing an undone cut.
+          command.seqs[i].setDatasetSequence(null);
+        }
+        command.seqs[i].deleteChars(command.position, command.position
+                + command.number);
+        if (command.oldds != null && command.oldds[i] != null)
         {
-          for (int s = command.position; s < command.position + command.number;
-               s++)
+          // oldds entry contains the cut dataset sequence.
+          command.seqs[i].setDatasetSequence(command.oldds[i]);
+          command.oldds[i] = oldds;
+        }
+        else
+        {
+          // modify the oldds if necessary
+          if (oldds != command.seqs[i].getDatasetSequence()
+                  || command.seqs[i].getSequenceFeatures() != null)
           {
-            if (jalview.schemes.ResidueProperties
-                .aaIndex[command.seqs[i].getCharAt(s)] != 23)
+            if (command.oldds == null)
             {
-              adjustFeatures(command, i,
-                             command.seqs[i].findPosition(command.position),
-                             command.seqs[i].findPosition(command.position +
-                  command.number),
-                             false);
-              break;
+              command.oldds = new SequenceI[command.seqs.length];
             }
+            command.oldds[i] = oldds;
+            adjustFeatures(
+                    command,
+                    i,
+                    command.seqs[i].findPosition(command.position),
+                    command.seqs[i].findPosition(command.position
+                            + command.number), false);
           }
         }
-        command.seqs[i].deleteChars(command.position,
-                                    command.position + command.number);
       }
 
       if (command.seqs[i].getLength() < 1)
       {
         command.al.deleteSequence(command.seqs[i]);
-        seqDeleted=true;
+        seqDeleted = true;
       }
     }
 
@@ -302,36 +341,49 @@ public class EditCommand
   {
     StringBuffer tmp;
     boolean newDSNeeded;
-    boolean seqWasDeleted=false;
+    boolean newDSWasNeeded;
+    int newstart, newend;
+    boolean seqWasDeleted = false;
     int start = 0, end = 0;
 
     for (int i = 0; i < command.seqs.length; i++)
     {
       newDSNeeded = false;
+      newDSWasNeeded = command.oldds != null && command.oldds[i] != null;
       if (command.seqs[i].getLength() < 1)
       {
         // ie this sequence was deleted, we need to
         // read it to the alignment
         if (command.alIndex[i] < command.al.getHeight())
         {
-          command.al.getSequences().insertElementAt(command.seqs[i],
-              command.alIndex[i]);
+          List<SequenceI> sequences;
+          synchronized (sequences = command.al.getSequences())
+          {
+            // int index = command.al.findIndex(command.seqs[i]);
+            // sequences.add(index, command.seqs[i]);
+            sequences.add(command.alIndex[i] < 0 ? 0 : command.alIndex[i], command.seqs[i]);
+          }
         }
         else
         {
           command.al.addSequence(command.seqs[i]);
         }
-        seqWasDeleted=true;
+        seqWasDeleted = true;
       }
+      newstart = command.seqs[i].getStart();
+      newend = command.seqs[i].getEnd();
+
       tmp = new StringBuffer();
       tmp.append(command.seqs[i].getSequence());
+      // Undo of a delete does not replace original dataset sequence on to
+      // alignment sequence.
 
       if (command.string != null && command.string[i] != null)
       {
         if (command.position >= tmp.length())
         {
-          //This occurs if padding is on, and residues
-          //are removed from end of alignment
+          // This occurs if padding is on, and residues
+          // are removed from end of alignment
           int length = command.position - tmp.length();
           while (length > 0)
           {
@@ -340,39 +392,60 @@ public class EditCommand
           }
         }
         tmp.insert(command.position, command.string[i]);
-
         for (int s = 0; s < command.string[i].length; s++)
         {
-          if (jalview.schemes.ResidueProperties.aaIndex[command.string[i][s]] !=
-              23)
+          if (jalview.schemes.ResidueProperties.aaIndex[command.string[i][s]] != 23)
           {
-            newDSNeeded = true;
-            start = command.seqs[i].findPosition(command.position);
-            end = command.seqs[i].findPosition(command.position +
-                                               command.number);
-            break;
+            if (!newDSNeeded)
+            {
+              newDSNeeded = true;
+              start = command.seqs[i].findPosition(command.position);
+              end = command.seqs[i].findPosition(command.position
+                      + command.number);
+            }
+            if (command.seqs[i].getStart() == start)
+            {
+              newstart--;
+            }
+            else
+            {
+              newend++;
+            }
           }
         }
         command.string[i] = null;
       }
 
       command.seqs[i].setSequence(tmp.toString());
-
+      command.seqs[i].setStart(newstart);
+      command.seqs[i].setEnd(newend);
       if (newDSNeeded)
       {
         if (command.seqs[i].getDatasetSequence() != null)
-        { // use new ds mechanism here
-          Sequence ds = new Sequence(command.seqs[i].getName(),
-                                     jalview.analysis.AlignSeq.extractGaps(
-                                         jalview.util.Comparison.GapChars,
-                                         command.seqs[i].getSequenceAsString()
-                                     ),
-                                     command.seqs[i].getStart(),
-                                     command.seqs[i].getEnd());
-          ds.setDescription(command.seqs[i].getDescription());
+        {
+          SequenceI ds;
+          if (newDSWasNeeded)
+          {
+            ds = command.oldds[i];
+          }
+          else
+          {
+            // make a new DS sequence
+            // use new ds mechanism here
+            ds = new Sequence(command.seqs[i].getName(),
+                    jalview.analysis.AlignSeq.extractGaps(
+                            jalview.util.Comparison.GapChars,
+                            command.seqs[i].getSequenceAsString()),
+                    command.seqs[i].getStart(), command.seqs[i].getEnd());
+            ds.setDescription(command.seqs[i].getDescription());
+          }
+          if (command.oldds == null)
+          {
+            command.oldds = new SequenceI[command.seqs.length];
+          }
+          command.oldds[i] = command.seqs[i].getDatasetSequence();
           command.seqs[i].setDatasetSequence(ds);
         }
-
         adjustFeatures(command, i, start, end, true);
       }
     }
@@ -387,22 +460,79 @@ public class EditCommand
     String oldstring;
     int start = command.position;
     int end = command.number;
-
+    // TODO TUTORIAL - Fix for replacement with different length of sequence (or
+    // whole sequence)
+    // TODO Jalview 2.4 bugfix change to an aggregate command - original
+    // sequence string is cut, new string is pasted in.
     command.number = start + command.string[0].length;
     for (int i = 0; i < command.seqs.length; i++)
     {
+      boolean newDSWasNeeded = command.oldds != null
+              && command.oldds[i] != null;
+
+      /**
+       * cut addHistoryItem(new EditCommand("Cut Sequences", EditCommand.CUT,
+       * cut, sg.getStartRes(), sg.getEndRes()-sg.getStartRes()+1,
+       * viewport.alignment));
+       * 
+       */
+      /**
+       * then addHistoryItem(new EditCommand( "Add sequences",
+       * EditCommand.PASTE, sequences, 0, alignment.getWidth(), alignment) );
+       * 
+       */
       oldstring = command.seqs[i].getSequenceAsString();
       tmp = new StringBuffer(oldstring.substring(0, start));
       tmp.append(command.string[i]);
+      String nogaprep = jalview.analysis.AlignSeq.extractGaps(
+              jalview.util.Comparison.GapChars, new String(
+                      command.string[i]));
+      int ipos = command.seqs[i].findPosition(start)
+              - command.seqs[i].getStart();
       tmp.append(oldstring.substring(end));
       command.seqs[i].setSequence(tmp.toString());
       command.string[i] = oldstring.substring(start, end).toCharArray();
+      String nogapold = jalview.analysis.AlignSeq.extractGaps(
+              jalview.util.Comparison.GapChars, new String(
+                      command.string[i]));
+      if (!nogaprep.toLowerCase().equals(nogapold.toLowerCase()))
+      {
+        if (newDSWasNeeded)
+        {
+          SequenceI oldds = command.seqs[i].getDatasetSequence();
+          command.seqs[i].setDatasetSequence(command.oldds[i]);
+          command.oldds[i] = oldds;
+        }
+        else
+        {
+          if (command.oldds == null)
+          {
+            command.oldds = new SequenceI[command.seqs.length];
+          }
+          command.oldds[i] = command.seqs[i].getDatasetSequence();
+          SequenceI newds = new Sequence(
+                  command.seqs[i].getDatasetSequence());
+          String fullseq, osp = newds.getSequenceAsString();
+          fullseq = osp.substring(0, ipos) + nogaprep
+                  + osp.substring(ipos + nogaprep.length());
+          newds.setSequence(fullseq.toUpperCase());
+          // TODO: JAL-1131 ensure newly created dataset sequence is added to
+          // the set of
+          // dataset sequences associated with the alignment.
+          // TODO: JAL-1131 fix up any annotation associated with new dataset
+          // sequence to ensure that original sequence/annotation relationships
+          // are preserved.
+          command.seqs[i].setDatasetSequence(newds);
+
+        }
+      }
       tmp = null;
       oldstring = null;
     }
   }
 
-  final void adjustAnnotations(Edit command, boolean insert, boolean modifyVisibility, AlignmentI[] views)
+  final void adjustAnnotations(Edit command, boolean insert,
+          boolean modifyVisibility, AlignmentI[] views)
   {
     AlignmentAnnotation[] annotations = null;
 
@@ -424,28 +554,31 @@ public class EditCommand
         if (modifyVisibility)
         {
           // Rows are only removed or added to sequence object.
-          if (!insert) {
+          if (!insert)
+          {
             // remove rows
             tmp = command.seqs[s].getAnnotation();
-            if (tmp!=null) {
-              int alen=tmp.length;
-              for (int aa =0; aa<tmp.length; aa++)
+            if (tmp != null)
+            {
+              int alen = tmp.length;
+              for (int aa = 0; aa < tmp.length; aa++)
               {
                 if (!command.al.deleteAnnotation(tmp[aa]))
                 {
-                  // strip out annotation not in the current al (will be put back on insert in all views)
+                  // strip out annotation not in the current al (will be put
+                  // back on insert in all views)
                   tmp[aa] = null;
                   alen--;
                 }
               }
               command.seqs[s].setAlignmentAnnotation(null);
-              if (alen!=tmp.length)
+              if (alen != tmp.length)
               {
                 // save the non-null annotation references only
                 AlignmentAnnotation[] saved = new AlignmentAnnotation[alen];
-                for (int aa=0,aapos=0;aa<tmp.length;aa++)
+                for (int aa = 0, aapos = 0; aa < tmp.length; aa++)
                 {
-                  if (tmp[aa]!=null)
+                  if (tmp[aa] != null)
                   {
                     saved[aapos++] = tmp[aa];
                     tmp[aa] = null;
@@ -454,56 +587,68 @@ public class EditCommand
                 tmp = saved;
                 command.deletedAnnotationRows.put(command.seqs[s], saved);
                 // and then remove any annotation in the other views
-                for (int alview=0; views!=null && alview<views.length; alview++)
+                for (int alview = 0; views != null && alview < views.length; alview++)
                 {
-                  if (views[alview]!=command.al)
+                  if (views[alview] != command.al)
                   {
-                    AlignmentAnnotation[] toremove = views[alview].getAlignmentAnnotation();
-                    if (toremove==null || toremove.length==0)
+                    AlignmentAnnotation[] toremove = views[alview]
+                            .getAlignmentAnnotation();
+                    if (toremove == null || toremove.length == 0)
                     {
                       continue;
                     }
-                    // remove any alignment annotation on this sequence that's on that alignment view.
-                    for (int aa = 0; aa<toremove.length; aa++)
+                    // remove any alignment annotation on this sequence that's
+                    // on that alignment view.
+                    for (int aa = 0; aa < toremove.length; aa++)
                     {
-                      if (toremove[aa].sequenceRef==command.seqs[s])
+                      if (toremove[aa].sequenceRef == command.seqs[s])
                       {
                         views[alview].deleteAnnotation(toremove[aa]);
                       }
                     }
                   }
                 }
-              } else {
+              }
+              else
+              {
                 // save all the annotation
                 command.deletedAnnotationRows.put(command.seqs[s], tmp);
               }
             }
-          } else {
+          }
+          else
+          {
             // recover rows
-            if (command.deletedAnnotationRows!=null && command.deletedAnnotationRows.containsKey(command.seqs[s]))
+            if (command.deletedAnnotationRows != null
+                    && command.deletedAnnotationRows
+                            .containsKey(command.seqs[s]))
             {
-              AlignmentAnnotation[] revealed = (AlignmentAnnotation[]) command.deletedAnnotationRows.get(command.seqs[s]);
+              AlignmentAnnotation[] revealed = (AlignmentAnnotation[]) command.deletedAnnotationRows
+                      .get(command.seqs[s]);
               command.seqs[s].setAlignmentAnnotation(revealed);
-              if (revealed!=null) {
-                for (int aa =0; aa<revealed.length; aa++)
+              if (revealed != null)
+              {
+                for (int aa = 0; aa < revealed.length; aa++)
                 {
                   // iterate through al adding original annotation
                   command.al.addAnnotation(revealed[aa]);
                 }
-                for (int aa =0; aa<revealed.length; aa++)
+                for (int aa = 0; aa < revealed.length; aa++)
                 {
                   command.al.setAnnotationIndex(revealed[aa], aa);
                 }
-                // and then duplicate added annotation on every other alignment view
-                for (int vnum=0; views!=null && vnum<views.length; vnum++)
+                // and then duplicate added annotation on every other alignment
+                // view
+                for (int vnum = 0; views != null && vnum < views.length; vnum++)
                 {
-                  if (views[vnum]!=command.al)
+                  if (views[vnum] != command.al)
                   {
-                    int avwidth = views[vnum].getWidth()+1;
+                    int avwidth = views[vnum].getWidth() + 1;
                     // duplicate in this view
-                    for (int a=0; a<revealed.length; a++)
+                    for (int a = 0; a < revealed.length; a++)
                     {
-                      AlignmentAnnotation newann = new AlignmentAnnotation(revealed[a]);
+                      AlignmentAnnotation newann = new AlignmentAnnotation(
+                              revealed[a]);
                       command.seqs[s].addAlignmentAnnotation(newann);
                       newann.padAnnotation(avwidth);
                       views[vnum].addAnnotation(newann);
@@ -528,14 +673,13 @@ public class EditCommand
         }
         else
         {
-          tmp = new AlignmentAnnotation
-              [aSize + command.seqs[s].getAnnotation().length];
+          tmp = new AlignmentAnnotation[aSize
+                  + command.seqs[s].getAnnotation().length];
 
           System.arraycopy(annotations, 0, tmp, 0, aSize);
 
-          System.arraycopy(command.seqs[s].getAnnotation(),
-                           0, tmp, aSize,
-                           command.seqs[s].getAnnotation().length);
+          System.arraycopy(command.seqs[s].getAnnotation(), 0, tmp, aSize,
+                  command.seqs[s].getAnnotation().length);
 
           annotations = tmp;
         }
@@ -557,7 +701,8 @@ public class EditCommand
     Annotation[] temp;
     for (int a = 0; a < annotations.length; a++)
     {
-      if (annotations[a].autoCalculated || annotations[a].annotations == null)
+      if (annotations[a].autoCalculated
+              || annotations[a].annotations == null)
       {
         continue;
       }
@@ -568,13 +713,13 @@ public class EditCommand
       if (insert)
       {
         temp = new Annotation[aSize + command.number];
-        if(annotations[a].padGaps)
+        if (annotations[a].padGaps)
+        {
           for (int aa = 0; aa < temp.length; aa++)
           {
-            temp[aa] = new Annotation(
-                command.gapChar+"",
-                null, ' ', 0);
+            temp[aa] = new Annotation(command.gapChar + "", null, ' ', 0);
           }
+        }
       }
       else
       {
@@ -601,52 +746,44 @@ public class EditCommand
         temp = new Annotation[tSize];
       }
 
-
       if (insert)
       {
         if (command.position < annotations[a].annotations.length)
         {
-          System.arraycopy(annotations[a].annotations,
-                           0, temp, 0, command.position);
+          System.arraycopy(annotations[a].annotations, 0, temp, 0,
+                  command.position);
 
           if (command.deletedAnnotations != null
-              &&
-              command.deletedAnnotations.containsKey(annotations[a].
-              annotationId))
+                  && command.deletedAnnotations
+                          .containsKey(annotations[a].annotationId))
           {
-            Annotation[] restore = (Annotation[])
-                command.deletedAnnotations.get(annotations[a].annotationId);
+            Annotation[] restore = (Annotation[]) command.deletedAnnotations
+                    .get(annotations[a].annotationId);
 
-            System.arraycopy(restore,
-                             0,
-                             temp,
-                             command.position,
-                             command.number);
+            System.arraycopy(restore, 0, temp, command.position,
+                    command.number);
 
           }
 
-          System.arraycopy(annotations[a].annotations,
-                           command.position, temp,
-                           command.position + command.number,
-                           aSize - command.position);
+          System.arraycopy(annotations[a].annotations, command.position,
+                  temp, command.position + command.number, aSize
+                          - command.position);
         }
         else
         {
           if (command.deletedAnnotations != null
-              &&
-              command.deletedAnnotations.containsKey(annotations[a].
-              annotationId))
+                  && command.deletedAnnotations
+                          .containsKey(annotations[a].annotationId))
           {
-            Annotation[] restore = (Annotation[])
-                command.deletedAnnotations.get(annotations[a].annotationId);
-
-            temp = new Annotation[annotations[a].annotations.length +
-                restore.length];
-            System.arraycopy(annotations[a].annotations,
-                             0, temp, 0,
-                             annotations[a].annotations.length);
+            Annotation[] restore = (Annotation[]) command.deletedAnnotations
+                    .get(annotations[a].annotationId);
+
+            temp = new Annotation[annotations[a].annotations.length
+                    + restore.length];
+            System.arraycopy(annotations[a].annotations, 0, temp, 0,
+                    annotations[a].annotations.length);
             System.arraycopy(restore, 0, temp,
-                             annotations[a].annotations.length, restore.length);
+                    annotations[a].annotations.length, restore.length);
           }
           else
           {
@@ -658,15 +795,20 @@ public class EditCommand
       {
         if (tSize != aSize || command.position < 2)
         {
-          int copylen = Math.min(command.position, annotations[a].annotations.length);
-          if (copylen>0)
-            System.arraycopy(annotations[a].annotations,
-                           0, temp, 0, copylen); //command.position);
+          int copylen = Math.min(command.position,
+                  annotations[a].annotations.length);
+          if (copylen > 0)
+           {
+            System.arraycopy(annotations[a].annotations, 0, temp, 0,
+                    copylen); // command.position);
+          }
 
           Annotation[] deleted = new Annotation[command.number];
-          if (copylen>=command.position) {
-            copylen = Math.min(command.number, annotations[a].annotations.length-command.position);
-            if (copylen>0)
+          if (copylen >= command.position)
+          {
+            copylen = Math.min(command.number,
+                    annotations[a].annotations.length - command.position);
+            if (copylen > 0)
             {
               System.arraycopy(annotations[a].annotations,
                       command.position, deleted, 0, copylen); // command.number);
@@ -674,12 +816,14 @@ public class EditCommand
           }
 
           command.deletedAnnotations.put(annotations[a].annotationId,
-                                         deleted);
-          if (annotations[a].annotations.length>command.position+command.number) {
-            System.arraycopy(annotations[a].annotations,
-                           command.position + command.number,
-                           temp, command.position,
-                           annotations[a].annotations.length - command.position - command.number); // aSize
+                  deleted);
+          if (annotations[a].annotations.length > command.position
+                  + command.number)
+          {
+            System.arraycopy(annotations[a].annotations, command.position
+                    + command.number, temp, command.position,
+                    annotations[a].annotations.length - command.position
+                            - command.number); // aSize
           }
         }
         else
@@ -689,17 +833,16 @@ public class EditCommand
           if (dSize > 0)
           {
             Annotation[] deleted = new Annotation[command.number];
-            System.arraycopy(annotations[a].annotations,
-                             command.position, deleted, 0, dSize);
+            System.arraycopy(annotations[a].annotations, command.position,
+                    deleted, 0, dSize);
 
             command.deletedAnnotations.put(annotations[a].annotationId,
-                                           deleted);
+                    deleted);
 
             tSize = Math.min(annotations[a].annotations.length,
-                             command.position);
+                    command.position);
             temp = new Annotation[tSize];
-            System.arraycopy(annotations[a].annotations,
-                             0, temp, 0, tSize);
+            System.arraycopy(annotations[a].annotations, 0, temp, 0, tSize);
           }
           else
           {
@@ -713,7 +856,7 @@ public class EditCommand
   }
 
   final void adjustFeatures(Edit command, int index, int i, int j,
-                            boolean insert)
+          boolean insert)
   {
     SequenceI seq = command.seqs[index];
     SequenceI sequence = seq.getDatasetSequence();
@@ -725,11 +868,10 @@ public class EditCommand
     if (insert)
     {
       if (command.editedFeatures != null
-          && command.editedFeatures.containsKey(seq))
+              && command.editedFeatures.containsKey(seq))
       {
-        sequence.setSequenceFeatures(
-            (SequenceFeature[]) command.editedFeatures.get(seq)
-            );
+        sequence.setSequenceFeatures((SequenceFeature[]) command.editedFeatures
+                .get(seq));
       }
 
       return;
@@ -793,23 +935,32 @@ public class EditCommand
 
   class Edit
   {
+    public SequenceI[] oldds;
+
     boolean fullAlignmentHeight = false;
+
     Hashtable deletedAnnotationRows;
+
     Hashtable deletedAnnotations;
+
     Hashtable editedFeatures;
+
     AlignmentI al;
+
     int command;
+
     char[][] string;
+
     SequenceI[] seqs;
+
     int[] alIndex;
+
     int position, number;
+
     char gapChar;
 
-    Edit(int command,
-         SequenceI[] seqs,
-         int position,
-         int number,
-         char gapChar)
+    Edit(int command, SequenceI[] seqs, int position, int number,
+            char gapChar)
     {
       this.command = command;
       this.seqs = seqs;
@@ -818,11 +969,8 @@ public class EditCommand
       this.gapChar = gapChar;
     }
 
-    Edit(int command,
-         SequenceI[] seqs,
-         int position,
-         int number,
-         AlignmentI al)
+    Edit(int command, SequenceI[] seqs, int position, int number,
+            AlignmentI al)
     {
       this.gapChar = al.getGapCharacter();
       this.command = command;
@@ -840,12 +988,8 @@ public class EditCommand
       fullAlignmentHeight = (al.getHeight() == seqs.length);
     }
 
-    Edit(int command,
-         SequenceI[] seqs,
-         int position,
-         int number,
-         AlignmentI al,
-         String replace)
+    Edit(int command, SequenceI[] seqs, int position, int number,
+            AlignmentI al, String replace)
     {
       this.command = command;
       this.seqs = seqs;