Move implementation to .tpp

franzpoeschel · Oct 20, 2023 · 90d02c1 · 90d02c1
1 parent 42fd638
commit 90d02c1
Show file tree

Hide file tree

Showing 2 changed files with 34 additions and 30 deletions.
diff --git a/include/openPMD/RecordComponent.hpp b/include/openPMD/RecordComponent.hpp
@@ -98,29 +98,6 @@ namespace internal
     };
 } // namespace internal
 
-class RecordComponent;
-
-namespace detail
-{
-    template <typename Functor, typename Res>
-    struct VisitRecordComponent
-    {
-        template <typename T, typename... Args>
-        static Res call(RecordComponent &rc, Args &&...args)
-        {
-            return Functor::template call<T>(rc, std::forward<Args>(args)...);
-        }
-
-        template <int = 0, typename... Args>
-        static Res call(Args &&...)
-        {
-            throw std::runtime_error(
-                "[RecordComponent::visit()] Unknown datatype in "
-                "RecordComponent");
-        }
-    };
-} // namespace detail
-
 class RecordComponent : public BaseRecordComponent
 {
     template <typename T, typename T_key, typename T_container>
@@ -461,13 +438,7 @@ class RecordComponent : public BaseRecordComponent
     template <typename Visitor, typename... Args>
     constexpr auto visit(Args &&...args)
         -> decltype(Visitor::template call<char>(
-            std::declval<RecordComponent &>(), std::forward<Args>(args)...))
-    {
-        using Res = decltype(Visitor::template call<char>(
             std::declval<RecordComponent &>(), std::forward<Args>(args)...));
-        return switchNonVectorType<detail::VisitRecordComponent<Visitor, Res>>(
-            getDatatype(), *this, std::forward<Args>(args)...);
-    }
 
     static constexpr char const *const SCALAR = "\vScalar";
 

diff --git a/include/openPMD/RecordComponent.tpp b/include/openPMD/RecordComponent.tpp
@@ -250,7 +250,7 @@ void RecordComponent::storeChunkRaw(T *ptr, Offset offset, Extent extent)
 
 template <typename T_ContiguousContainer>
 inline typename std::enable_if_t<
-    auxiliary::IsContiguousContainer_v<T_ContiguousContainer> >
+    auxiliary::IsContiguousContainer_v<T_ContiguousContainer>>
 RecordComponent::storeChunk(T_ContiguousContainer &data, Offset o, Extent e)
 {
     uint8_t dim = getDimensionality();
@@ -373,4 +373,37 @@ RecordComponent::storeChunk(Offset offset, Extent extent)
 #endif
     });
 }
+
+namespace detail
+{
+    template <typename Functor, typename Res>
+    struct VisitRecordComponent
+    {
+        template <typename T, typename... Args>
+        static Res call(RecordComponent &rc, Args &&...args)
+        {
+            return Functor::template call<T>(rc, std::forward<Args>(args)...);
+        }
+
+        template <int = 0, typename... Args>
+        static Res call(Args &&...)
+        {
+            throw std::runtime_error(
+                "[RecordComponent::visit()] Unknown datatype in "
+                "RecordComponent");
+        }
+    };
+} // namespace detail
+
+template <typename Visitor, typename... Args>
+constexpr auto RecordComponent::visit(Args &&...args)
+    -> decltype(Visitor::template call<char>(
+        std::declval<RecordComponent &>(), std::forward<Args>(args)...))
+{
+    using Res = decltype(Visitor::template call<char>(
+        std::declval<RecordComponent &>(), std::forward<Args>(args)...));
+    return switchNonVectorType<detail::VisitRecordComponent<Visitor, Res>>(
+        getDatatype(), *this, std::forward<Args>(args)...);
+}
+
 } // namespace openPMD