From 883b67a84b22ae174e56f237be51e9034648bbd2 Mon Sep 17 00:00:00 2001 From: Enrico Guiraud <enrico.guiraud@cern.ch> Date: Tue, 26 Mar 2019 19:58:32 +0100 Subject: [PATCH] [TTree] Avoid getting the number of branches twice The operation is relatively costly as TObjArray::GetEntries also creates a TReadLockGuard. --- tree/tree/src/TTree.cxx | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/tree/tree/src/TTree.cxx b/tree/tree/src/TTree.cxx index 77a2fee964e..ec202a7ecdb 100644 --- a/tree/tree/src/TTree.cxx +++ b/tree/tree/src/TTree.cxx @@ -5429,8 +5429,7 @@ Int_t TTree::GetEntry(Long64_t entry, Int_t getall) }; #ifdef R__USE_IMT - const auto nBranches = GetListOfBranches()->GetEntries(); - if (nBranches > 1 && ROOT::IsImplicitMTEnabled() && fIMTEnabled && !TTreeCacheUnzip::IsParallelUnzip()) { + if (nbranches > 1 && ROOT::IsImplicitMTEnabled() && fIMTEnabled && !TTreeCacheUnzip::IsParallelUnzip()) { if (fSortedBranches.empty()) InitializeBranchLists(true); -- GitLab