Fix pointer error checking

Author: endurodave

main.cpp

@@ -166,6 +166,10 @@ class AllData : public serialize::I
         ms.write(os, dataSetPtr);
         ms.write(os, dataSetValue);
         ms.write(os, dataSetInt);
+
+        // static_assert; pointers not allowed
+        //ms.write(os, pdate);
+        //ms.write(os, pint);
         return os;
     }
 
@@ -200,6 +204,10 @@ class AllData : public serialize::I
         ms.read(is, dataSetPtr);
         ms.read(is, dataSetValue);
         ms.read(is, dataSetInt);
+
+        // static_assert; pointers not allowed
+        //ms.read(is, pdate);
+        //ms.read(is, pint);
         return is;
     }
 
@@ -232,6 +240,10 @@ class AllData : public serialize::I
     set<Date*> dataSetPtr;
     set<Date> dataSetValue;
     set<int> dataSetInt;
+
+    // Pointer to data not allowed; static_assert triggered
+    Date* pdate = nullptr;
+    int* pint = nullptr;
 };
 
 // DataV1 is a version 1 data structure

serialize.h

@@ -483,8 +483,12 @@ class serialize
     template<typename T>
     std::istream& read(std::istream& is, T &t_, bool readPrependedType = true)
     {   
-        static_assert(!(std::is_pointer<T>::value && std::is_arithmetic<typename std::remove_pointer<T>::type>::value),
-            "T cannot be a pointer to a built-in data type");
+        static_assert(!is_unsupported_container<T>::value, "Unsupported C++ container type");
+
+        static_assert(!(std::is_pointer<T>::value &&
+            (std::is_arithmetic<typename std::remove_pointer<T>::type>::value ||
+                std::is_class<typename std::remove_pointer<T>::type>::value)),
+            "T cannot be a pointer to a built-in or custom data type");
 
         if (check_stop_parse(is))
             return is;
@@ -534,8 +538,10 @@ class serialize
     {
         static_assert(!is_unsupported_container<T>::value, "Unsupported C++ container type");
 
-        static_assert(!(std::is_pointer<T>::value && std::is_arithmetic<typename std::remove_pointer<T>::type>::value),
-            "T cannot be a pointer to a built-in data type");
+        static_assert(!(std::is_pointer<T>::value &&
+            (std::is_arithmetic<typename std::remove_pointer<T>::type>::value ||
+                std::is_class<typename std::remove_pointer<T>::type>::value)),
+            "T cannot be a pointer to a built-in or custom data type");
 
         // Is T type a built-in data type (e.g. float, int, ...)?
         if (std::is_class<T>::value == false)