diff options
| author | Arnaud Simon <arnaudsimon@apache.org> | 2007-12-04 09:54:28 +0000 |
|---|---|---|
| committer | Arnaud Simon <arnaudsimon@apache.org> | 2007-12-04 09:54:28 +0000 |
| commit | 1b7282b2df58f558fff9929fb2b6015f091b3b4c (patch) | |
| tree | 7ed9b0a0b9e62a920c86ffb67e52b1483cb75254 /java | |
| parent | e3303ea643833b1b72f6ac656c0e6ddca03d7258 (diff) | |
| download | qpid-python-1b7282b2df58f558fff9929fb2b6015f091b3b4c.tar.gz | |
removed useless lock
git-svn-id: https://svn.apache.org/repos/asf/incubator/qpid/trunk/qpid@600860 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'java')
| -rw-r--r-- | java/common/src/main/java/org/apache/qpidity/transport/network/mina/MinaSender.java | 20 |
1 files changed, 6 insertions, 14 deletions
diff --git a/java/common/src/main/java/org/apache/qpidity/transport/network/mina/MinaSender.java b/java/common/src/main/java/org/apache/qpidity/transport/network/mina/MinaSender.java index 6906c03cb6..75af9cfe20 100644 --- a/java/common/src/main/java/org/apache/qpidity/transport/network/mina/MinaSender.java +++ b/java/common/src/main/java/org/apache/qpidity/transport/network/mina/MinaSender.java @@ -30,15 +30,13 @@ import org.apache.qpidity.transport.Sender; /** * MinaSender - * */ public class MinaSender implements Sender<java.nio.ByteBuffer> { - private static final int TIMEOUT = 2*60*1000; + private static final int TIMEOUT = 2 * 60 * 1000; private final IoSession session; - private final Object lock = new Object(); private WriteFuture lastWrite = null; public MinaSender(IoSession session) @@ -48,26 +46,20 @@ public class MinaSender implements Sender<java.nio.ByteBuffer> public void send(java.nio.ByteBuffer buf) { - synchronized (lock) + if (session.isClosing()) { - if( session.isClosing()) - { - throw new RuntimeException("Trying to write on a closed socket"); - } - lastWrite = session.write(ByteBuffer.wrap(buf)); + throw new RuntimeException("Trying to write on a closed socket"); } + lastWrite = session.write(ByteBuffer.wrap(buf)); } public void close() { // MINA will sometimes throw away in-progress writes when you // ask it to close - synchronized (lock) + if (lastWrite != null) { - if (lastWrite != null) - { - lastWrite.join(); - } + lastWrite.join(); } CloseFuture closed = session.close(); closed.join(); |
