Switch to tuple for consistency

AIDASoft · Jun 11, 2024 · fa71a06 · fa71a06
1 parent 7d2167d
commit fa71a06
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/include/podio/utilities/RootHelpers.h b/include/podio/utilities/RootHelpers.h
@@ -20,7 +20,7 @@ namespace root_utils {
   using CollectionInfoWithoutSchemaT = std::tuple<int, std::string, bool>;
 
   /// A collection name and a base pointer grouped together for writing
-  using StoreCollection = std::pair<const std::string&, podio::CollectionBase*>;
+  using StoreCollection = std::tuple<const std::string&, podio::CollectionBase*>;
 
   /// Small helper struct to collect all branches that are necessary to read or
   /// write a collection. Needed to cache the branch pointers and avoid having to

diff --git a/src/ROOTWriter.cc b/src/ROOTWriter.cc
@@ -132,9 +132,9 @@ void ROOTWriter::resetBranches(std::vector<root_utils::CollectionBranches>& bran
                                const std::vector<root_utils::StoreCollection>& collections,
                                /*const*/ podio::GenericParameters* parameters) {
   size_t iColl = 0;
-  for (auto& coll : collections) {
+  for (auto& [_, coll] : collections) {
     const auto& collBranches = branches[iColl];
-    root_utils::setCollectionAddresses(coll.second->getBuffers(), collBranches);
+    root_utils::setCollectionAddresses(coll->getBuffers(), collBranches);
     iColl++;
   }