Author: nextgens
Date: 2007-03-17 14:47:47 +0000 (Sat, 17 Mar 2007)
New Revision: 12183

Modified:
   trunk/freenet/src/freenet/client/StandardOnionFECCodec.java
   trunk/freenet/src/freenet/config/FreenetFilePersistentConfig.java
   trunk/freenet/src/freenet/crypt/DSAGroup.java
   trunk/freenet/src/freenet/io/NetworkInterface.java
   trunk/freenet/src/freenet/io/comm/UdpSocketManager.java
   trunk/freenet/src/freenet/io/xfer/BlockTransmitter.java
   trunk/freenet/src/freenet/node/Node.java
   trunk/freenet/src/freenet/node/fcp/FCPServer.java
   trunk/freenet/src/freenet/support/FileLoggerHook.java
Log:
Change notify() to notifyAll() in many places : most are sensible, some might 
not be.

review would be welcome.

Modified: trunk/freenet/src/freenet/client/StandardOnionFECCodec.java
===================================================================
--- trunk/freenet/src/freenet/client/StandardOnionFECCodec.java 2007-03-17 
14:10:24 UTC (rev 12182)
+++ trunk/freenet/src/freenet/client/StandardOnionFECCodec.java 2007-03-17 
14:47:47 UTC (rev 12183)
@@ -60,7 +60,7 @@
                                Logger.minor(this, "Splitfile encode: k="+k+", 
n="+n+" encode took "+(endTime-startTime)+"ms");
                        finished = true;
                        synchronized(this) {
-                               notify();
+                               notifyAll();
                        }
                }

@@ -187,7 +187,7 @@
                } finally {
                        synchronized(runningDecodesSync) {
                                runningDecodes--;
-                               runningDecodesSync.notify();
+                               runningDecodesSync.notifyAll();
                        }
                }
        }
@@ -359,7 +359,7 @@
                } finally {
                        synchronized(runningDecodesSync) {
                                runningDecodes--;
-                               runningDecodesSync.notify();
+                               runningDecodesSync.notifyAll();
                        }
                }
        }

Modified: trunk/freenet/src/freenet/config/FreenetFilePersistentConfig.java
===================================================================
--- trunk/freenet/src/freenet/config/FreenetFilePersistentConfig.java   
2007-03-17 14:10:24 UTC (rev 12182)
+++ trunk/freenet/src/freenet/config/FreenetFilePersistentConfig.java   
2007-03-17 14:47:47 UTC (rev 12183)
@@ -72,7 +72,7 @@
                synchronized (this) {
                        if(hasNodeStarted) Logger.error(this, "It has already 
been called! that shouldn't happen!");
                        this.hasNodeStarted = true;
-                       notify();
+                       notifyAll();
                }
        }
 }

Modified: trunk/freenet/src/freenet/crypt/DSAGroup.java
===================================================================
--- trunk/freenet/src/freenet/crypt/DSAGroup.java       2007-03-17 14:10:24 UTC 
(rev 12182)
+++ trunk/freenet/src/freenet/crypt/DSAGroup.java       2007-03-17 14:47:47 UTC 
(rev 12183)
@@ -203,7 +203,7 @@
                 qg.qs.removeElementAt(0);

                 synchronized (qg) {
-                    qg.notify();
+                    qg.notifyAll();
                 }
             } else
                 q = makePrime(DSAGroup.Q_BIT_LENGTH, 80, r);

Modified: trunk/freenet/src/freenet/io/NetworkInterface.java
===================================================================
--- trunk/freenet/src/freenet/io/NetworkInterface.java  2007-03-17 14:10:24 UTC 
(rev 12182)
+++ trunk/freenet/src/freenet/io/NetworkInterface.java  2007-03-17 14:47:47 UTC 
(rev 12183)
@@ -333,7 +333,7 @@
                                        if (addressMatched) {
                                                synchronized (syncObject) {
                                                        
acceptedSockets.add(clientSocket);
-                                                       syncObject.notify();
+                                                       syncObject.notifyAll();
                                                }
                                        } else {
                                                try {

Modified: trunk/freenet/src/freenet/io/comm/UdpSocketManager.java
===================================================================
--- trunk/freenet/src/freenet/io/comm/UdpSocketManager.java     2007-03-17 
14:10:24 UTC (rev 12182)
+++ trunk/freenet/src/freenet/io/comm/UdpSocketManager.java     2007-03-17 
14:47:47 UTC (rev 12183)
@@ -313,7 +313,7 @@
                                        f.setMessage(null);
                                        synchronized (f) {
                                                i.remove();
-                                               f.notify();
+                                               f.notifyAll();
                                        }
                                } else { // Because _filters are in order of 
timeout, we
                                        // can abort the iteration as soon as 
we find one that
@@ -361,7 +361,7 @@
                                        f.setMessage(m);
                                        synchronized (f) {
                                                i.remove();
-                                               f.notify();
+                                               f.notifyAll();
                                        }
                                        if(logMINOR) Logger.minor(this, 
"Matched: "+f);
                                        break; // Only one match permitted per 
message
@@ -410,7 +410,7 @@
                                                f.setMessage(m);
                                                synchronized (f) {
                                                        i.remove();
-                                                       f.notify();
+                                                       f.notifyAll();
                                                }
                                                if(logMINOR) Logger.minor(this, 
"Matched: "+f);
                                                break; // Only one match 
permitted per message

Modified: trunk/freenet/src/freenet/io/xfer/BlockTransmitter.java
===================================================================
--- trunk/freenet/src/freenet/io/xfer/BlockTransmitter.java     2007-03-17 
14:10:24 UTC (rev 12182)
+++ trunk/freenet/src/freenet/io/xfer/BlockTransmitter.java     2007-03-17 
14:47:47 UTC (rev 12183)
@@ -192,7 +192,7 @@
                                                synchronized(_senderThread) {
                                                        _unsent.addLast(new 
Integer(packetNo));
                                                        
_sentPackets.setBit(packetNo, false);
-                                                       _senderThread.notify();
+                                                       
_senderThread.notifyAll();
                                                }
                                        }


Modified: trunk/freenet/src/freenet/node/Node.java
===================================================================
--- trunk/freenet/src/freenet/node/Node.java    2007-03-17 14:10:24 UTC (rev 
12182)
+++ trunk/freenet/src/freenet/node/Node.java    2007-03-17 14:47:47 UTC (rev 
12183)
@@ -3493,7 +3493,7 @@

                void interrupt() {
                        synchronized(this) {
-                               notify();
+                               notifyAll();
                        }
                }


Modified: trunk/freenet/src/freenet/node/fcp/FCPServer.java
===================================================================
--- trunk/freenet/src/freenet/node/fcp/FCPServer.java   2007-03-17 14:10:24 UTC 
(rev 12182)
+++ trunk/freenet/src/freenet/node/fcp/FCPServer.java   2007-03-17 14:47:47 UTC 
(rev 12183)
@@ -297,7 +297,7 @@
                        FCPServerPersister p = server.persister;
                        if(p != null) {
                                synchronized(p) {
-                                       p.notify();
+                                       p.notifyAll();
                                }
                        }
                }

Modified: trunk/freenet/src/freenet/support/FileLoggerHook.java
===================================================================
--- trunk/freenet/src/freenet/support/FileLoggerHook.java       2007-03-17 
14:10:24 UTC (rev 12182)
+++ trunk/freenet/src/freenet/support/FileLoggerHook.java       2007-03-17 
14:47:47 UTC (rev 12183)
@@ -862,7 +862,7 @@
                                listBytes += (buf.length + LINE_OVERHEAD);
                        }
                        if (sz == 0)
-                               list.notify();
+                               list.notifyAll();
                }
        }



Reply via email to