From ea9bfd096c5a037ac83f8924027a16651b191a35 Mon Sep 17 00:00:00 2001 From: Vincenzo Eduardo Padulano Date: Wed, 29 Oct 2025 17:49:32 +0100 Subject: [PATCH] [tree] Remove unused method TChain::Loop The equivalent TTree method was removed 22 years ago https://github.com/root-project/root/commit/5621165b06fd8a2bbffbe22009b9e66dd522d81b . --- tree/tree/inc/TChain.h | 3 +-- tree/tree/src/TChain.cxx | 44 ---------------------------------------- 2 files changed, 1 insertion(+), 46 deletions(-) diff --git a/tree/tree/inc/TChain.h b/tree/tree/inc/TChain.h index 31e9b2a02516e..8d362124ec261 100644 --- a/tree/tree/inc/TChain.h +++ b/tree/tree/inc/TChain.h @@ -127,8 +127,7 @@ class TChain : public TTree { bool InPlaceClone(TDirectory *newdirectory, const char *options = "") override; Int_t LoadBaskets(Long64_t maxmemory) override; Long64_t LoadTree(Long64_t entry) override; - void Lookup(bool force = false); - virtual void Loop(Option_t *option="", Long64_t nentries=kMaxEntries, Long64_t firstentry=0); // *MENU* + void Lookup(bool force = false); void ls(Option_t *option="") const override; virtual Long64_t Merge(const char *name, Option_t *option = ""); Long64_t Merge(TCollection *list, Option_t *option = "") override; diff --git a/tree/tree/src/TChain.cxx b/tree/tree/src/TChain.cxx index 2ef501117ffe1..fc536792ac7c2 100644 --- a/tree/tree/src/TChain.cxx +++ b/tree/tree/src/TChain.cxx @@ -1804,50 +1804,6 @@ void TChain::Lookup(bool force) SafeDelete(stg); } -//////////////////////////////////////////////////////////////////////////////// -/// Loop on nentries of this chain starting at firstentry. (NOT IMPLEMENTED) - -void TChain::Loop(Option_t* option, Long64_t nentries, Long64_t firstentry) -{ - Error("Loop", "Function not yet implemented"); - - if (option || nentries || firstentry) { } // keep warnings away - -#if 0 - if (LoadTree(firstentry) < 0) return; - - if (firstentry < 0) firstentry = 0; - Long64_t lastentry = firstentry + nentries -1; - if (lastentry > fEntries-1) { - lastentry = fEntries -1; - } - - GetPlayer(); - GetSelector(); - fSelector->Start(option); - - Long64_t entry = firstentry; - Int_t tree,e0,en; - for (tree=0;tree lastentry) en = lastentry; - if (entry > en) continue; - - LoadTree(entry); - fSelector->BeginFile(); - - while (entry <= en) { - fSelector->Execute(fTree, entry - e0); - entry++; - } - fSelector->EndFile(); - } - - fSelector->Finish(option); -#endif -} - //////////////////////////////////////////////////////////////////////////////// /// List the chain.