Merge pull request #310 from boostorg/feature/va-start

Restore removed va_start calls; add missing va_end as reported in #264.

Author: robertramey

include/boost/serialization/extended_type_info_no_rtti.hpp

@@ -129,23 +129,26 @@ class extended_type_info_no_rtti :
     }
     void * construct(unsigned int count, ...) const BOOST_OVERRIDE {
         // count up the arguments
+        void * r = NULL;
         std::va_list ap;
+        va_start(ap, count);
         switch(count){
         case 0:
-            return factory<typename boost::remove_const< T >::type, 0>(ap);
+            r = factory<typename boost::remove_const< T >::type, 0>(ap);
         case 1:
-            return factory<typename boost::remove_const< T >::type, 1>(ap);
+            r = factory<typename boost::remove_const< T >::type, 1>(ap);
         case 2:
-            return factory<typename boost::remove_const< T >::type, 2>(ap);
+            r = factory<typename boost::remove_const< T >::type, 2>(ap);
         case 3:
-            return factory<typename boost::remove_const< T >::type, 3>(ap);
+            r = factory<typename boost::remove_const< T >::type, 3>(ap);
         case 4:
-            return factory<typename boost::remove_const< T >::type, 4>(ap);
+            r = factory<typename boost::remove_const< T >::type, 4>(ap);
         default:
             BOOST_ASSERT(false); // too many arguments
             // throw exception here?
-            return NULL;
         }
+        va_end(ap);
+        return r;
     }
     void destroy(void const * const p) const BOOST_OVERRIDE {
         boost::serialization::access::destroy(

include/boost/serialization/extended_type_info_typeid.hpp

@@ -112,23 +112,26 @@ class extended_type_info_typeid :
     }
     void * construct(unsigned int count, ...) const BOOST_OVERRIDE {
         // count up the arguments
+        void * r = NULL;
         std::va_list ap;
+        va_start(ap, count);
         switch(count){
         case 0:
-            return factory<typename boost::remove_const< T >::type, 0>(ap);
+            r = factory<typename boost::remove_const< T >::type, 0>(ap);
         case 1:
-            return factory<typename boost::remove_const< T >::type, 1>(ap);
+            r = factory<typename boost::remove_const< T >::type, 1>(ap);
         case 2:
-            return factory<typename boost::remove_const< T >::type, 2>(ap);
+            r = factory<typename boost::remove_const< T >::type, 2>(ap);
         case 3:
-            return factory<typename boost::remove_const< T >::type, 3>(ap);
+            r = factory<typename boost::remove_const< T >::type, 3>(ap);
         case 4:
-            return factory<typename boost::remove_const< T >::type, 4>(ap);
+            r = factory<typename boost::remove_const< T >::type, 4>(ap);
         default:
             BOOST_ASSERT(false); // too many arguments
             // throw exception here?
-            return NULL;
         }
+        va_end(ap);
+        return r;
     }
     void destroy(void const * const p) const BOOST_OVERRIDE {
         boost::serialization::access::destroy(