svn commit: r499928 - /incubator/qpid/branches/qpid.0-9/java/broker/src/main/java/org/apache/qpid/server/queue/ConcurrentDeliveryManager.java

[rhs]

Author: rhs
Date: Thu Jan 25 11:22:24 2007
New Revision: 499928

URL: http://svn.apache.org/viewvc?view=rev&rev=499928
Log:
removed XXX exception

Modified:
    
incubator/qpid/branches/qpid.0-9/java/broker/src/main/java/org/apache/qpid/server/queue/ConcurrentDeliveryManager.java

Modified: 
incubator/qpid/branches/qpid.0-9/java/broker/src/main/java/org/apache/qpid/server/queue/ConcurrentDeliveryManager.java
URL: 
http://svn.apache.org/viewvc/incubator/qpid/branches/qpid.0-9/java/broker/src/main/java/org/apache/qpid/server/queue/ConcurrentDeliveryManager.java?view=diff&rev=499928&r1=499927&r2=499928
==============================================================================
--- 
incubator/qpid/branches/qpid.0-9/java/broker/src/main/java/org/apache/qpid/server/queue/ConcurrentDeliveryManager.java
 (original)
+++ 
incubator/qpid/branches/qpid.0-9/java/broker/src/main/java/org/apache/qpid/server/queue/ConcurrentDeliveryManager.java
 Thu Jan 25 11:22:24 2007
@@ -142,8 +142,13 @@
 
     private boolean addMessageToQueue(AMQMessage msg)
     {
+        // XXX: I don't think this will have an effect anymore because
+        // we don't allocate anymore space than we need. Also this
+        // seems like an odd place to do this. Why wouldn't we want to
+        // do this from a more central location on receiving the
+        // message?
+
         // Shrink the ContentBodies to their actual size to save memory.
-        if (true) throw new Error("XXX");
         /*if (compressBufferOnQueue)
         {
             Iterator it = msg.getContentBodies().iterator();