JAL-3383 minor code tidying and commenting

[jalview.git] / src / jalview / gui / IdCanvas.java

diff --git a/src/jalview/gui/IdCanvas.java b/src/jalview/gui/IdCanvas.java
index e5a5946..0731bf3 100755 (executable)
--- a/src/jalview/gui/IdCanvas.java
+++ b/src/jalview/gui/IdCanvas.java
@@ -59,7 +59,7 @@ public class IdCanvas extends JPanel implements ViewportListenerI
 
   int imgHeight = 0;
 
-  boolean fastPaint = false;
+  private boolean fastPaint = false;
 
   List<SequenceI> searchResults;
 
@@ -579,45 +579,35 @@ public class IdCanvas extends JPanel implements ViewportListenerI
   @Override
   public void propertyChange(PropertyChangeEvent evt)
   {
-    // BH just clarifying logic
     String propertyName = evt.getPropertyName();
-    switch (propertyName) {
+    switch (propertyName)
+    {
     case ViewportRanges.STARTSEQ:
       fastPaint((int) evt.getNewValue() - (int) evt.getOldValue());
-      return;
+      break;
     case ViewportRanges.STARTRES:
       if (av.getWrapAlignment())
       {
         fastPaint((int) evt.getNewValue() - (int) evt.getOldValue());
       }
-      return;
+      break;
     case ViewportRanges.STARTRESANDSEQ:
       fastPaint(((int[]) evt.getNewValue())[1]
               - ((int[]) evt.getOldValue())[1]);
-      return;
+      break;
     case ViewportRanges.MOVE_VIEWPORT:
       repaint();
-      return;
-    case ViewportRanges.ENDRES:
-    case ViewportRanges.ENDSEQ:
-      // ignore ??
-      return;
+      break;
+    default:
     }
-// BH 2019.07.27 was:
-//    if (propertyName.equals(ViewportRanges.STARTSEQ)
-//            || (av.getWrapAlignment()
-//                    && propertyName.equals(ViewportRanges.STARTRES)))
-//    {
-//      fastPaint((int) evt.getNewValue() - (int) evt.getOldValue());
-//    }
-//    else if (propertyName.equals(ViewportRanges.STARTRESANDSEQ))
-//    {
-//      fastPaint(((int[]) evt.getNewValue())[1]
-//              - ((int[]) evt.getOldValue())[1]);
-//    }
-//    else if (propertyName.equals(ViewportRanges.MOVE_VIEWPORT))
-//    {
-//      repaint();
-    // }
+  }
+
+  /**
+   * Clears the flag that allows a 'fast paint' on the next repaint, so
+   * requiring a full repaint
+   */
+  public void setNoFastPaint()
+  {
+    fastPaint = false;
   }
 }