This fixes PR30216 by initialising the list of listeners early. Changelog:
2006-12-22 Andrew John Hughes <[EMAIL PROTECTED]>
PR classpath/30216:
* javax/management/MBeanServerDelegate.java:
Initialise list of listeners immediately.
(addListener(NotificationListener)): Remove list
construction.
--
Andrew :-)
Escape the Java Trap with GNU Classpath!
http://www.gnu.org/philosophy/java-trap.html
public class gcj extends Freedom implements Java { ... }
Index: javax/management/MBeanServerDelegate.java
===================================================================
RCS file:
/cvsroot/classpath/classpath/javax/management/MBeanServerDelegate.java,v
retrieving revision 1.1
diff -u -3 -p -u -r1.1 MBeanServerDelegate.java
--- javax/management/MBeanServerDelegate.java 2 Oct 2006 18:28:11 -0000
1.1
+++ javax/management/MBeanServerDelegate.java 19 Dec 2006 01:15:46 -0000
@@ -69,7 +69,7 @@ public class MBeanServerDelegate
/**
* The listeners registered with the delegate.
*/
- private List listeners;
+ private final List listeners = new ArrayList();
/**
* The sequence identifier used by the delegate.
@@ -120,8 +120,6 @@ public class MBeanServerDelegate
{
if (listener == null)
throw new IllegalArgumentException("A null listener was supplied.");
- if (listeners == null)
- listeners = new ArrayList();
listeners.add(new ListenerData(listener, filter, passback));
}
signature.asc
Description: Digital signature
