Necessary Maintenance #17926
Implement a better name for the track proxy accessor to the unfitted trajectory.
Experiment:
-
Description
It so happens now that a track proxy object can access the fitted trajectory of the track via track->Trajectory()
and the associated, unfitted trajectory as track.trajectory()
(and its pointer with trajectoryPtr()
).
This accidental ambiguity needs to be removed.
Associated revisions
History
#1 Updated by Lynn Garren about 3 years ago
- Status changed from New to Assigned
- Assignee set to Gianluca Petrillo
We need the fix for this weeks larsoft release.
#2 Updated by Gianluca Petrillo about 3 years ago
- Status changed from Assigned to Resolved
- % Done changed from 0 to 100
Names have been changed:
Old name | New name |
hasTrajectory() |
hasOriginalTrajectory() |
trajectory() |
originalTrajectory() |
trajectoryPtr() |
originalTrajectoryPtr() |
The change has been pushed into lardata
develop
branch directly, despite it being a breaking change.
No committed code has shown any dependency on these methods (tried the whole LArSoft suite plus six experiment repositories).
#3 Updated by Gianluca Petrillo about 3 years ago
- Status changed from Resolved to Closed
Track proxy metod name trajectory() (and related) renamed.
The functions hasTrajectory(), trajectory() and trajectoryPtr() have
been renamed.
This solves issue #17926.