[jboss-cvs] JBoss Messaging SVN: r4822 - trunk/src/main/org/jboss/messaging/core/remoting/impl/netty.

[jboss-cvs-commits at lists.jboss.org]

Author: trustin
Date: 2008-08-18 06:41:20 -0400 (Mon, 18 Aug 2008)
New Revision: 4822

Modified:
   trunk/src/main/org/jboss/messaging/core/remoting/impl/netty/ChannelBufferWrapper.java
Log:
Fixed a bug where ChannelBufferWrapper is not created when the specified size is 0

Modified: trunk/src/main/org/jboss/messaging/core/remoting/impl/netty/ChannelBufferWrapper.java
===================================================================
--- trunk/src/main/org/jboss/messaging/core/remoting/impl/netty/ChannelBufferWrapper.java	2008-08-18 10:35:41 UTC (rev 4821)
+++ trunk/src/main/org/jboss/messaging/core/remoting/impl/netty/ChannelBufferWrapper.java	2008-08-18 10:41:20 UTC (rev 4822)
@@ -58,11 +58,17 @@
 
    public ChannelBufferWrapper(final int size)
    {
-      if (size < 0) {
-         throw new IllegalArgumentException("size: " + size);
+      if (size == 0) {
+         buffer = ChannelBuffer.EMPTY_BUFFER;
+      } else {
+         try {
+            buffer = dynamicBuffer(size);
+         } catch (IllegalArgumentException e) {
+            // FIXME: This block should go away once Netty 3.0.0.CR2 is
+            //        released.
+            throw new IllegalArgumentException("size: " + size);
+         }
       }
-
-      buffer = dynamicBuffer(size);
       buffer.writerIndex(buffer.capacity());
    }