ViewHelper code tidy
Fixing whitespace and formatting, since this is supposed to be a "helper" class and therefore it's important to keep it readable.
Also removing a bunch of unused code and methods, on the assumption that by now we've stabilised on correct usage patterns and should guide people towards them rather than providing alternatives.
Merge request reports
Activity
Tagging @tbold
- Resolved by Benjamin Michael Wynne
- Resolved by Benjamin Michael Wynne
Looks ok to me. Since this is mostly about a propper coding style, maybe could fit better to atlas coding convention. I.e. method/varaible names start from the small letter?
added master review-pending-level-1 labels
CI Result SUCCESS (hash 34a76146)Athena AthSimulation 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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 10796] CI Result SUCCESS (hash dcf7be32)Athena AthSimulation 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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 10807]added review-approved label and removed review-pending-level-1 label
added 101 commits
-
dcf7be32...1c8f021c - 99 commits from branch
atlas:master
- f808439e - Merge branch 'master' of https://gitlab.cern.ch/atlas/athena into ViewHelperTidy
- c3703d30 - Renaming for the new alg
-
dcf7be32...1c8f021c - 99 commits from branch
added review-pending-level-1 label and removed review-approved label
added review-approved label and removed review-pending-level-1 label