You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by od...@apache.org on 2007/01/11 12:45:04 UTC
svn commit: r495201 - in /harmony/enhanced/classlib/trunk/modules/swing/src:
main/java/common/javax/swing/DefaultBoundedRangeModel.java
test/api/java/common/javax/swing/DefaultBoundedRangeModel2Test.java
Author: odeakin
Date: Thu Jan 11 03:45:03 2007
New Revision: 495201
URL: http://svn.apache.org/viewvc?view=rev&rev=495201
Log:
Apply patch HARMONY-2621 ([classlib][swing] DefaultBoundedRangeModel(Integer.MAX_VALUE,1,2,4) expected IAE)
Modified:
harmony/enhanced/classlib/trunk/modules/swing/src/main/java/common/javax/swing/DefaultBoundedRangeModel.java
harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/DefaultBoundedRangeModel2Test.java
Modified: harmony/enhanced/classlib/trunk/modules/swing/src/main/java/common/javax/swing/DefaultBoundedRangeModel.java
URL: http://svn.apache.org/viewvc/harmony/enhanced/classlib/trunk/modules/swing/src/main/java/common/javax/swing/DefaultBoundedRangeModel.java?view=diff&rev=495201&r1=495200&r2=495201
==============================================================================
--- harmony/enhanced/classlib/trunk/modules/swing/src/main/java/common/javax/swing/DefaultBoundedRangeModel.java (original)
+++ harmony/enhanced/classlib/trunk/modules/swing/src/main/java/common/javax/swing/DefaultBoundedRangeModel.java Thu Jan 11 03:45:03 2007
@@ -56,7 +56,7 @@
}
public DefaultBoundedRangeModel(int value, int extent, int min, int max) {
- if (extent < 0 || min > value || value + extent > max) {
+ if (min > value || value > value + extent || value + extent > max) {
throw new IllegalArgumentException("invalid range properties");
}
this.min = min;
Modified: harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/DefaultBoundedRangeModel2Test.java
URL: http://svn.apache.org/viewvc/harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/DefaultBoundedRangeModel2Test.java?view=diff&rev=495201&r1=495200&r2=495201
==============================================================================
--- harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/DefaultBoundedRangeModel2Test.java (original)
+++ harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/DefaultBoundedRangeModel2Test.java Thu Jan 11 03:45:03 2007
@@ -156,6 +156,12 @@
new DefaultBoundedRangeModel(1, -1, 1, 2);
}
});
+ testExceptionalCase(new IllegalArgumentCase() {
+ @Override // Regression test for HARMONY-2621
+ public void exceptionalAction() throws Exception {
+ new DefaultBoundedRangeModel(Integer.MAX_VALUE,1,2,4);
+ }
+ });
}
public void testFireStateChanged() {
@@ -329,4 +335,4 @@
ChangeListener listeners[] = brm1.getChangeListeners();
assertEquals(0, listeners.length);
}
-}
\ No newline at end of file
+}