This patch fixes a bug that is showing up when I try to launch FindBugs (I've also added Mauve tests to back up the change):

2005-11-17  David Gilbert  <[EMAIL PROTECTED]>

        * javax/swing/JComponent.java
        (setMaximumSize): Check for null argument before making a copy.

Regards,

Dave
Index: javax/swing/JComponent.java
===================================================================
RCS file: /cvsroot/classpath/classpath/javax/swing/JComponent.java,v
retrieving revision 1.86
diff -u -r1.86 JComponent.java
--- javax/swing/JComponent.java 15 Nov 2005 23:07:23 -0000      1.86
+++ javax/swing/JComponent.java 17 Nov 2005 23:18:00 -0000
@@ -2530,7 +2530,10 @@
   public void setMaximumSize(Dimension max)
   {
     Dimension oldMaximumSize = maximumSize;
-    maximumSize = new Dimension(max);
+    if (max != null) 
+      maximumSize = new Dimension(max);
+    else
+      maximumSize = null;
     firePropertyChange("maximumSize", oldMaximumSize, maximumSize);
   }
 
_______________________________________________
Classpath-patches mailing list
Classpath-patches@gnu.org
http://lists.gnu.org/mailman/listinfo/classpath-patches

Reply via email to