From f654474fc48815c7002b909045d72aca488abf1d Mon Sep 17 00:00:00 2001
From: scott snyder <scott.snyder@cern.ch>
Date: Tue, 6 Feb 2018 15:55:13 +0100
Subject: [PATCH] RootStorageSvc: Avoid a copy of auxid_set_t.

We can avoid a copy of auxid_set_t now that we've switched to using
ConcurrentBitset.
---
 Database/APR/RootStorageSvc/src/RootTreeContainer.cpp | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/Database/APR/RootStorageSvc/src/RootTreeContainer.cpp b/Database/APR/RootStorageSvc/src/RootTreeContainer.cpp
index 2436d841e57..bdd52e3e8a2 100755
--- a/Database/APR/RootStorageSvc/src/RootTreeContainer.cpp
+++ b/Database/APR/RootStorageSvc/src/RootTreeContainer.cpp
@@ -206,11 +206,7 @@ DbStatus RootTreeContainer::writeObject(TransactionStack::value_type& ent) {
                    // cout << "---    store object= " <<hex << store <<dec << " in " << dsc.branch->GetName()  <<endl;
                    // cout << "       obj=" << hex << dsc.object << dec << "  offset=" <<  dsc.aux_iostore_IFoffset << endl;
                    log << DbPrintLvl::Debug << "       Attributes= " << store->getSelectedAuxIDs().size() << DbPrint::endmsg;
-                   // Need to make a copy in case it changes.
-                   // FIXME: may be able to get rid of this copy
-                   // if we switch to a thread-safe container.
-                   const SG::auxid_set_t ids = store->getSelectedAuxIDs();
-                   for(SG::auxid_t id : ids ) {
+                   for(SG::auxid_t id : store->getSelectedAuxIDs()) {
                       BranchDesc&       newBrDsc( m_auxBranchMap[id] );
                       if( !newBrDsc.branch ) {
                          auto &reg = SG::AuxTypeRegistry::instance();
-- 
GitLab