Minor TEvent fixes
Fix TEvent
issues discovered in !39406 (comment 4068296)
Also change ut_xaodrootaccess_proxydict_test
variable to something more common in R22.
/cc @akraszna
Merge request reports
Activity
added bugfix label
added Core master review-pending-level-1 labels
The reason that I didn't open a merge request myself with this was that I thought it would be faster if the small patches toTEvent.cxx
would be done as part of !39406 (merged)...But other than that, and the fact that the copyright dates were not updated
, all these updates are good of course.It probably won't be faster in !39406 (merged) as I have yet to run a lot of tests
And we did decide a while ago that copyright bumps are not needed for trivial changes
CI Result SUCCESS (hash 0db10861)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 3
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 25799]Compilation warnings are unrelated and have already been fixed in !39404 (merged).
added review-approved label and removed review-pending-level-1 label
mentioned in commit 6c64d260
added sweep:ignore label