svn commit: r525727 - in /mina/trunk/core/src/main/java/org/apache/mina/transport/vmpipe: VmPipeAcceptor.java VmPipeAddress.java

[trustin]

Author: trustin
Date: Wed Apr  4 23:40:10 2007
New Revision: 525727

URL: http://svn.apache.org/viewvc?view=rev&rev=525727
Log:
* Changed VmPipeAcceptor to throw an IOException with an explicit message when 
negative port number is specified.
* Changed VmPipeAddress.toString() to return better string representation

Modified:
    
mina/trunk/core/src/main/java/org/apache/mina/transport/vmpipe/VmPipeAcceptor.java
    
mina/trunk/core/src/main/java/org/apache/mina/transport/vmpipe/VmPipeAddress.java

Modified: 
mina/trunk/core/src/main/java/org/apache/mina/transport/vmpipe/VmPipeAcceptor.java
URL: 
http://svn.apache.org/viewvc/mina/trunk/core/src/main/java/org/apache/mina/transport/vmpipe/VmPipeAcceptor.java?view=diff&rev=525727&r1=525726&r2=525727
==============================================================================
--- 
mina/trunk/core/src/main/java/org/apache/mina/transport/vmpipe/VmPipeAcceptor.java
 (original)
+++ 
mina/trunk/core/src/main/java/org/apache/mina/transport/vmpipe/VmPipeAcceptor.java
 Wed Apr  4 23:40:10 2007
@@ -79,7 +79,7 @@
         
         synchronized( boundHandlers )
         {
-            if( localAddress == null || localAddress.getPort() <= 0 )
+            if( localAddress == null || localAddress.getPort() == 0 )
             {
                 localAddress = null;
                 for( int i = 1; i < Integer.MAX_VALUE; i++ )
@@ -96,8 +96,9 @@
                     throw new IOException("No port available.");
                 }
             }
-            else if( boundHandlers.containsKey( localAddress ) )
-            {
+            else if( localAddress.getPort() < 0) {
+                throw new IOException("Bind port number must be 0 or above.");
+            } else if( boundHandlers.containsKey( localAddress ) ) {
                 throw new IOException( "Address already bound: " + 
localAddress );
             }
 

Modified: 
mina/trunk/core/src/main/java/org/apache/mina/transport/vmpipe/VmPipeAddress.java
URL: 
http://svn.apache.org/viewvc/mina/trunk/core/src/main/java/org/apache/mina/transport/vmpipe/VmPipeAddress.java?view=diff&rev=525727&r1=525726&r2=525727
==============================================================================
--- 
mina/trunk/core/src/main/java/org/apache/mina/transport/vmpipe/VmPipeAddress.java
 (original)
+++ 
mina/trunk/core/src/main/java/org/apache/mina/transport/vmpipe/VmPipeAddress.java
 Wed Apr  4 23:40:10 2007
@@ -81,6 +81,10 @@
     @Override
     public String toString()
     {
-        return "vm:" + port;
+        if (port >= 0) {
+            return "vm:server:" + port;
+        } else {
+            return "vm:client:" + (-port);
+        }
     }
 }