svn commit: r561252 - in /mina: branches/1.0/core/src/main/java/org/apache/mina/transport/socket/nio/ branches/1.1/core/src/main/java/org/apache/mina/transport/socket/nio/ trunk/core/src/main/java/org/apache/mina/transport/socket/nio/

[trustin]

Author: trustin
Date: Tue Jul 31 00:23:25 2007
New Revision: 561252

URL: http://svn.apache.org/viewvc?view=rev&rev=561252
Log:
Fixed issue: DIRMINA-409 (Duplicate syncronization in 
SocketAcceptor.startupWorker()) - Removed unnecessary synchronization

Modified:
    
mina/branches/1.0/core/src/main/java/org/apache/mina/transport/socket/nio/SocketAcceptor.java
    
mina/branches/1.1/core/src/main/java/org/apache/mina/transport/socket/nio/SocketAcceptor.java
    
mina/trunk/core/src/main/java/org/apache/mina/transport/socket/nio/SocketAcceptor.java

Modified: 
mina/branches/1.0/core/src/main/java/org/apache/mina/transport/socket/nio/SocketAcceptor.java
URL: 
http://svn.apache.org/viewvc/mina/branches/1.0/core/src/main/java/org/apache/mina/transport/socket/nio/SocketAcceptor.java?view=diff&rev=561252&r1=561251&r2=561252
==============================================================================
--- 
mina/branches/1.0/core/src/main/java/org/apache/mina/transport/socket/nio/SocketAcceptor.java
 (original)
+++ 
mina/branches/1.0/core/src/main/java/org/apache/mina/transport/socket/nio/SocketAcceptor.java
 Tue Jul 31 00:23:25 2007
@@ -165,7 +165,7 @@
         }
     }
 
-    private synchronized void startupWorker() throws IOException {
+    private void startupWorker() throws IOException {
         synchronized (lock) {
             if (worker == null) {
                 selector = Selector.open();

Modified: 
mina/branches/1.1/core/src/main/java/org/apache/mina/transport/socket/nio/SocketAcceptor.java
URL: 
http://svn.apache.org/viewvc/mina/branches/1.1/core/src/main/java/org/apache/mina/transport/socket/nio/SocketAcceptor.java?view=diff&rev=561252&r1=561251&r2=561252
==============================================================================
--- 
mina/branches/1.1/core/src/main/java/org/apache/mina/transport/socket/nio/SocketAcceptor.java
 (original)
+++ 
mina/branches/1.1/core/src/main/java/org/apache/mina/transport/socket/nio/SocketAcceptor.java
 Tue Jul 31 00:23:25 2007
@@ -154,7 +154,7 @@
         }
     }
 
-    private synchronized void startupWorker() throws IOException {
+    private void startupWorker() throws IOException {
         synchronized (lock) {
             if (worker == null) {
                 selector = Selector.open();

Modified: 
mina/trunk/core/src/main/java/org/apache/mina/transport/socket/nio/SocketAcceptor.java
URL: 
http://svn.apache.org/viewvc/mina/trunk/core/src/main/java/org/apache/mina/transport/socket/nio/SocketAcceptor.java?view=diff&rev=561252&r1=561251&r2=561252
==============================================================================
--- 
mina/trunk/core/src/main/java/org/apache/mina/transport/socket/nio/SocketAcceptor.java
 (original)
+++ 
mina/trunk/core/src/main/java/org/apache/mina/transport/socket/nio/SocketAcceptor.java
 Tue Jul 31 00:23:25 2007
@@ -297,7 +297,7 @@
      * is now working, then nothing will happen and the method
      * will just return.
      */
-    private synchronized void startupWorker() {
+    private void startupWorker() {
         synchronized (lock) {
             if (worker == null) {
                 worker = new Worker();