Skip to content

Conversation

@vepadulano
Copy link
Member

Following up on this comment #20222 (comment)

@vepadulano vepadulano self-assigned this Oct 29, 2025
@vepadulano vepadulano requested a review from pcanal as a code owner October 29, 2025 16:50
@github-actions
Copy link

github-actions bot commented Oct 29, 2025

Test Results

    22 files      22 suites   3d 14h 42m 33s ⏱️
 3 706 tests  3 706 ✅ 0 💤 0 ❌
79 581 runs  79 581 ✅ 0 💤 0 ❌

Results for commit ea9bfd0.

♻️ This comment has been updated with latest results.

Copy link
Member

@pcanal pcanal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's remove the whole function.

@vepadulano vepadulano force-pushed the tree-remove-dead-code branch 2 times, most recently from 3029e3d to 2dc44c6 Compare October 31, 2025 02:00
@vepadulano vepadulano requested a review from pcanal October 31, 2025 02:00
@vepadulano vepadulano changed the title [tree] Remove dead code in TChain::Loop Remove unused method TChain::Loop Oct 31, 2025
The equivalent TTree method was removed 22 years ago root-project@5621165 .
@vepadulano vepadulano force-pushed the tree-remove-dead-code branch from 2dc44c6 to ea9bfd0 Compare October 31, 2025 20:30
@dpiparo dpiparo merged commit 0eccb29 into root-project:master Nov 2, 2025
48 of 50 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants