You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by gs...@apache.org on 2010/05/17 11:38:31 UTC

svn commit: r945048 - /qpid/trunk/qpid/cpp/src/qpid/framing/SendContent.cpp

Author: gsim
Date: Mon May 17 09:38:31 2010
New Revision: 945048

URL: http://svn.apache.org/viewvc?rev=945048&view=rev
Log:
QPID-2605: Fix to fragmentation logic for delivered messages

Modified:
    qpid/trunk/qpid/cpp/src/qpid/framing/SendContent.cpp

Modified: qpid/trunk/qpid/cpp/src/qpid/framing/SendContent.cpp
URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/cpp/src/qpid/framing/SendContent.cpp?rev=945048&r1=945047&r2=945048&view=diff
==============================================================================
--- qpid/trunk/qpid/cpp/src/qpid/framing/SendContent.cpp (original)
+++ qpid/trunk/qpid/cpp/src/qpid/framing/SendContent.cpp Mon May 17 09:38:31 2010
@@ -30,9 +30,7 @@ void qpid::framing::SendContent::operato
     bool first = frameCount == 0;
     bool last = ++frameCount == expectedFrameCount;
 
-     /*end of frame marker is included in frameOverhead() but not in
-       real frame size, hence substract -1 from frameOverhead()*/
-    uint16_t maxContentSize = maxFrameSize - (AMQFrame::frameOverhead() - 1);
+    uint16_t maxContentSize = maxFrameSize - AMQFrame::frameOverhead();
     const AMQContentBody* body(f.castBody<AMQContentBody>()); 
     if (body->encodedSize() > maxContentSize) {
         uint32_t offset = 0;



---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:commits-subscribe@qpid.apache.org