Make tests work by hiding forward declaration from CLING

AIDASoft · Jun 11, 2024 · 771d995 · 771d995
1 parent 14042f4
commit 771d995
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/include/podio/GenericParameters.h b/include/podio/GenericParameters.h
@@ -30,10 +30,10 @@ class RNTupleWriter;
 #endif
 
 namespace podio {
-class ROOTReader;
-}
 
-namespace podio {
+#if !defined(__CLING__)
+class ROOTReader;
+#endif
 
 /// The types which are supported in the GenericParameters
 using SupportedGenericDataTypes = std::tuple<int, float, std::string, double>;
@@ -133,12 +133,15 @@ class GenericParameters {
   friend void readGenericParameters(sio::read_device& device, GenericParameters& parameters, sio::version_type version);
 #endif
 
-  friend ROOTReader;
 #if PODIO_ENABLE_RNTUPLE
   friend RNTupleReader;
   friend RNTupleWriter;
 #endif
 
+#if !defined(__CLING__)
+  friend ROOTReader;
+#endif
+
   /// Get a reference to the internal map for a given type
   template <typename T>
   const MapType<detail::GetVectorType<T>>& getMap() const {

diff --git a/src/ROOTReader.cc b/src/ROOTReader.cc
@@ -26,6 +26,8 @@ std::tuple<std::vector<root_utils::CollectionBranches>, std::vector<std::pair<st
 createCollectionBranchesIndexBased(TChain* chain, const podio::CollectionIDTable& idTable,
                                    const std::vector<root_utils::CollectionWriteInfoT>& collInfo);
 
+/// Helper struct to get the negative offsets from the end of the branches
+/// vector for the different types of generic parameters.
 template <typename T>
 struct TypeToBranchIndexOffset;