From: jprocter <jprocter@compbio.dundee.ac.uk>
Date: Thu, 27 Oct 2011 13:53:27 +0000 (+0100)
Subject: (JAL-969) removed TODOs
X-Git-Tag: Jalview_2_9~566^2~2
X-Git-Url: http://source.jalview.org/gitweb/?a=commitdiff_plain;h=b4877bc99e12cf4465e43e090e19277c3139ce77;p=jalview.git

(JAL-969) removed TODOs
---

diff --git a/src/jalview/appletgui/APopupMenu.java b/src/jalview/appletgui/APopupMenu.java
index 7a99a20..b45fa6b 100755
--- a/src/jalview/appletgui/APopupMenu.java
+++ b/src/jalview/appletgui/APopupMenu.java
@@ -529,7 +529,6 @@ public class APopupMenu extends java.awt.PopupMenu implements
       Vector regions = new Vector();
       if (sg != null)
       {
-        // TODO test change case behaves identically in JvLite
         int[][] startEnd = ap.av.getVisibleRegionBoundaries(sg.getStartRes(),
                 sg.getEndRes() + 1);
 
diff --git a/src/jalview/gui/PopupMenu.java b/src/jalview/gui/PopupMenu.java
index 45e976f..23a5008 100644
--- a/src/jalview/gui/PopupMenu.java
+++ b/src/jalview/gui/PopupMenu.java
@@ -1882,7 +1882,6 @@ public class PopupMenu extends JPopupMenu
 
   public void discoverPDB_actionPerformed()
   {
-    // TODO: test: verify PDB sequence discovery behaves correctly for group selections
 
     final SequenceI[] sequences = ((ap.av.getSelectionGroup() == null) ? new SequenceI[]
     { sequence }
diff --git a/src/jalview/viewmodel/AlignmentViewport.java b/src/jalview/viewmodel/AlignmentViewport.java
index 0cd23de..8e66188 100644
--- a/src/jalview/viewmodel/AlignmentViewport.java
+++ b/src/jalview/viewmodel/AlignmentViewport.java
@@ -207,7 +207,6 @@ public abstract class AlignmentViewport implements AlignViewportI
 
   public boolean isCalcInProgress()
   {
-    // TODO generalise to iterate over all calculators associated with av
     return calculator.isWorking();
   }
 
@@ -998,7 +997,6 @@ public abstract class AlignmentViewport implements AlignViewportI
    */
   public void alignmentChanged(AlignmentViewPanel ap)
   {
-    // TODO: test jvLite always pads, and that JVD has configurable padding.
     if (isPadGaps())
     {
       alignment.padGaps();