You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pivot.apache.org by tv...@apache.org on 2009/11/16 17:06:27 UTC
svn commit: r880825 -
/incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/content/CalendarDateSpinnerData.java
Author: tvolkert
Date: Mon Nov 16 16:06:27 2009
New Revision: 880825
URL: http://svn.apache.org/viewvc?rev=880825&view=rev
Log:
Fixed corner-case bug in CalendarDateSpinnerData
Modified:
incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/content/CalendarDateSpinnerData.java
Modified: incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/content/CalendarDateSpinnerData.java
URL: http://svn.apache.org/viewvc/incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/content/CalendarDateSpinnerData.java?rev=880825&r1=880824&r2=880825&view=diff
==============================================================================
--- incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/content/CalendarDateSpinnerData.java (original)
+++ incubator/pivot/trunk/wtk/src/org/apache/pivot/wtk/content/CalendarDateSpinnerData.java Mon Nov 16 16:06:27 2009
@@ -103,8 +103,8 @@
throw new IllegalArgumentException("upperBound is null.");
}
- if (lowerBound.compareTo(upperBound) >= 0) {
- throw new IllegalArgumentException("lowerBound must be before upperBound.");
+ if (lowerBound.compareTo(upperBound) > 0) {
+ throw new IllegalArgumentException("lowerBound is after upperBound.");
}
calendar = new GregorianCalendar(lowerBound.year, lowerBound.month,
@@ -120,7 +120,7 @@
long lowerBoundMilliseconds = calendar.getTimeInMillis();
long upperBoundMilliseconds = upperBoundCalendar.getTimeInMillis();
long indexDiff = (upperBoundMilliseconds - lowerBoundMilliseconds) /
- (1000 * 60 * 60 * 24);
+ (1000l * 60 * 60 * 24);
length = (int)indexDiff + 1;
}
@@ -196,7 +196,7 @@
tmpCalendar.setTimeZone(TimeZone.getTimeZone("GMT"));
long itemMilliseconds = tmpCalendar.getTimeInMillis();
- long indexDiff = (itemMilliseconds - currentMilliseconds) / (1000 * 60 * 60 * 24);
+ long indexDiff = (itemMilliseconds - currentMilliseconds) / (1000l * 60 * 60 * 24);
int index = calendarIndex + (int)indexDiff;
return (index < 0 || index >= length) ? -1 : index;