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