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, 2 insertions, 2 deletions

diff --git a/cpp/src/tests/HeaderTest.cpp b/cpp/src/tests/HeaderTest.cpp
index 21374e30e1..9e2bddb4de 100644
--- a/cpp/src/tests/HeaderTest.cpp
+++ b/cpp/src/tests/HeaderTest.cpp
@@ -56,7 +56,7 @@ class HeaderTest : public CppUnit::TestCase
     }
 
     void testMessageProperties() {
-        AMQFrame out(0, AMQHeaderBody());
+        AMQFrame out(in_place<AMQHeaderBody>());
         MessageProperties* props1 = 
             out.castBody<AMQHeaderBody>()->get<MessageProperties>(true);
 
@@ -96,7 +96,7 @@ class HeaderTest : public CppUnit::TestCase
     }
 
     void testDeliveryProperies() {
-        AMQFrame out(0, AMQHeaderBody());
+        AMQFrame out(in_place<AMQHeaderBody>());
         DeliveryProperties* props1 = 
             out.castBody<AMQHeaderBody>()->get<DeliveryProperties>(true);