Skip to content

Commit

Permalink
[ntuple] Remove redundant LoadEntry call
Browse files Browse the repository at this point in the history
The entry is currently loaded twice: once on `RNTupleProcessor::Advance`,
which is called when the iterator is incremented, and once when the
iterator is dereferenced. This second call is unnecessary and only
causes the processor to take more time, so this commit removes it.
  • Loading branch information
enirolf committed Jan 8, 2025
1 parent 097ec02 commit e573016
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions tree/ntuple/v7/inc/ROOT/RNTupleProcessor.hxx
Original file line number Diff line number Diff line change
Expand Up @@ -220,11 +220,7 @@ public:
return obj;
}

reference operator*()
{
fProcessor.LoadEntry();
return *fProcessor.fEntry;
}
reference operator*() { return *fProcessor.fEntry; }

friend bool operator!=(const iterator &lh, const iterator &rh)
{
Expand Down

0 comments on commit e573016

Please sign in to comment.