Added framing::BodyHolder:

 - Uniform holder for all body types, replaces MethodHolder.
 - Uses in_place constructors to avoid avoid body copy.

framing::AMQFrame:
 - Holds body in heap-allocated intrusive_ptr<BodyHolder>
 - Uses in_place constructors to avoid avoid body copy.

Removed/downgraded to TODO many redundant FIXME comments.


git-svn-id: https://svn.apache.org/repos/asf/incubator/qpid/trunk/qpid@597513 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 3 insertions, 2 deletions

diff --git a/cpp/src/tests/TxAckTest.cpp b/cpp/src/tests/TxAckTest.cpp
index 495b8b550e..bcf422e706 100644
--- a/cpp/src/tests/TxAckTest.cpp
+++ b/cpp/src/tests/TxAckTest.cpp
@@ -70,8 +70,9 @@ public:
     {
         for(int i = 0; i < 10; i++){
             intrusive_ptr<Message> msg(new Message());
-            AMQFrame method(0, MessageTransferBody(ProtocolVersion(), 0, "exchange", 0, 0));
-            AMQFrame header(0, AMQHeaderBody());
+            AMQFrame method(in_place<MessageTransferBody>(
+                                ProtocolVersion(), 0, "exchange", 0, 0));
+            AMQFrame header(in_place<AMQHeaderBody>());
             msg->getFrames().append(method);
             msg->getFrames().append(header);
             msg->getProperties<DeliveryProperties>()->setDeliveryMode(PERSISTENT);