Remove the function deltaR used only once in MCTruthClassifier

• Remove the function deltaR used only once
• Use Athena::Units::GeV and std::abs explicitly
• delete some not useful comments

This merge request affects 1 package:

• PhysicsAnalysis/MCTruthClassifier

This merge request affects 3 files:

• PhysicsAnalysis/MCTruthClassifier/MCTruthClassifier/MCTruthClassifier.h
• PhysicsAnalysis/MCTruthClassifier/Root/MCRecoToTruth.cxx
• PhysicsAnalysis/MCTruthClassifier/src/MCTruthClassifierAthena.cxx

Adding @zmarshal as watcher

added Analysis analysis-review-required main review-pending-level-1 labels

added 1 commit

• a67c354e - Use std

Compare with previous version

changed the description

This merge request affects 1 package:

• PhysicsAnalysis/MCTruthClassifier

This merge request affects 4 files:

• PhysicsAnalysis/MCTruthClassifier/MCTruthClassifier/MCTruthClassifier.h
• PhysicsAnalysis/MCTruthClassifier/Root/MCRecoToTruth.cxx
• PhysicsAnalysis/MCTruthClassifier/Root/MCTruthClassifierGen.cxx
• PhysicsAnalysis/MCTruthClassifier/src/MCTruthClassifierAthena.cxx

Adding @zmarshal as watcher

CI Result SUCCESS (hash 3d7d2692)

	Athena	AthGeneration	AnalysisBase	AthAnalysis
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: 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 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 6467] (remote access info)

CI Result SUCCESS (hash a67c354e)

	Athena	AthGeneration	AnalysisBase	AthAnalysis
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: 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 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 6468] (remote access info)

MR looks good to me, approving.

Cheers, L1

removed review-pending-level-1 label

added review-approved label

removed analysis-review-required label

added analysis-review-approved label

Looks ok to me. Approving from the analysis release side

merged

mentioned in commit a4c098f4