I'm checking this in.

Mario found that some prefs-using code was throwing illegal monitor
state exceptions.  There was a bug in the event dispatch code.

Tom

Index: ChangeLog
from  Tom Tromey  <[EMAIL PROTECTED]>

        * gnu/java/util/prefs/EventDispatcher.java (dispatch): Notify
        'queue'.
        (run): Wait on queue, not 'this'.

Index: gnu/java/util/prefs/EventDispatcher.java
===================================================================
RCS file: 
/cvsroot/classpath/classpath/gnu/java/util/prefs/EventDispatcher.java,v
retrieving revision 1.1
diff -u -r1.1 EventDispatcher.java
--- gnu/java/util/prefs/EventDispatcher.java 28 Feb 2006 16:53:16 -0000 1.1
+++ gnu/java/util/prefs/EventDispatcher.java 6 Jul 2006 18:12:51 -0000
@@ -74,7 +74,7 @@
               {
                 try
                   {
-                    wait();
+                    queue.wait();
                   }
                 catch (InterruptedException _)
                   {
@@ -107,6 +107,7 @@
     synchronized (queue)
       {
         queue.add(runner);
+       queue.notify();
       }
   }
 }

Reply via email to