The patch suggested in the bug report
(http://gcc.gnu.org/bugzilla/show_bug.cgi?id=24235) was essentially
correct, plus the specs require an explicit check for a null parameter.
This is committed.


2005-10-20  Anthony Balkissoon  <[EMAIL PROTECTED]>

        * java/awt/ScrollPaneAdjustable.java:
        (addAdjustmentListener): If parameter is null take no action and throw
        no exception.  Store the resulting listener chain from call to 
        AWTEventMulticaster.add.
        (removeAdjustmentListener): If parameter is null take no action and
        throw no exception.  Store the resulting listener chain from call to
        AWTEventMulticaster.remove.

--Tony
Index: java/awt/ScrollPaneAdjustable.java
===================================================================
RCS file: /cvsroot/classpath/classpath/java/awt/ScrollPaneAdjustable.java,v
retrieving revision 1.9
diff -u -r1.9 ScrollPaneAdjustable.java
--- java/awt/ScrollPaneAdjustable.java	2 Jul 2005 20:32:25 -0000	1.9
+++ java/awt/ScrollPaneAdjustable.java	20 Oct 2005 19:31:20 -0000
@@ -87,12 +87,16 @@
   
   public void addAdjustmentListener (AdjustmentListener listener)
   {
-    AWTEventMulticaster.add (adjustmentListener, listener);
+    if (listener == null)
+      return;
+    adjustmentListener = AWTEventMulticaster.add (adjustmentListener, listener);
   }
   
   public void removeAdjustmentListener (AdjustmentListener listener)
   {
-    AWTEventMulticaster.remove (adjustmentListener, listener);
+    if (listener == null)
+      return;
+    adjustmentListener = AWTEventMulticaster.remove (adjustmentListener, listener);
   }
   
   public AdjustmentListener[] getAdjustmentListeners ()
_______________________________________________
Classpath-patches mailing list
Classpath-patches@gnu.org
http://lists.gnu.org/mailman/listinfo/classpath-patches

Reply via email to