JAL-2439 Out-by-one in setting endRes/endSeq
authorkiramt <k.mourao@dundee.ac.uk>
Thu, 16 Mar 2017 12:12:31 +0000 (12:12 +0000)
committerkiramt <k.mourao@dundee.ac.uk>
Thu, 16 Mar 2017 12:12:31 +0000 (12:12 +0000)
src/jalview/viewmodel/ViewportPositionProps.java

index d143130..61dd18c 100644 (file)
@@ -108,9 +108,9 @@ public class ViewportPositionProps extends ViewportProperties
    */
   public void setEndRes(int res)
   {
-    if (res > al.getWidth())
+    if (res >= al.getWidth())
     {
-      res = al.getWidth();
+      res = al.getWidth() - 1;
     }
     else if (res < 1)
     {
@@ -146,9 +146,9 @@ public class ViewportPositionProps extends ViewportProperties
    */
   public void setEndSeq(int seq)
   {
-    if (seq > al.getHeight())
+    if (seq >= al.getHeight())
     {
-      seq = al.getHeight();
+      seq = al.getHeight() - 1;
     }
     else if (seq < 1)
     {