Modify computePhi to constexpr and clean includes

f3sch · web-flow · commit de80d431af8f · 2026-06-01T09:48:14.000+02:00
Updated computePhi function to be constexpr and removed unnecessary include.
diff --git a/Detectors/ITSMFT/ITS/tracking/include/ITStracking/MathUtils.h b/Detectors/ITSMFT/ITS/tracking/include/ITStracking/MathUtils.h
@@ -16,8 +16,6 @@
 #ifndef O2_ITS_TRACKING_MATHUTILS_H_
 #define O2_ITS_TRACKING_MATHUTILS_H_
 
-#include <cstdint>
-
 #include "CommonConstants/MathConstants.h"
 #include "ITStracking/Constants.h"
 #include "MathUtils/Utils.h"
@@ -27,7 +25,7 @@
 namespace o2::its::math_utils
 {
 
-GPUhdi() float computePhi(float x, float y)
+GPUhdi() constexpr float computePhi(float x, float y)
 {
   return o2::math_utils::fastATan2(-y, -x) + o2::constants::math::PI;
 }

Original file line number	Diff line number	Diff line change
`@@ -16,8 +16,6 @@`
`16`	`16`	`#ifndef O2_ITS_TRACKING_MATHUTILS_H_`
`17`	`17`	`#define O2_ITS_TRACKING_MATHUTILS_H_`
`18`	`18`
`19`		`-#include <cstdint>`
`20`		`-`
`21`	`19`	`#include "CommonConstants/MathConstants.h"`
`22`	`20`	`#include "ITStracking/Constants.h"`
`23`	`21`	`#include "MathUtils/Utils.h"`
`@@ -27,7 +25,7 @@`
`27`	`25`	`namespace o2::its::math_utils`
`28`	`26`	`{`
`29`	`27`
`30`		`-GPUhdi() float computePhi(float x, float y)`
	`28`	`+GPUhdi() constexpr float computePhi(float x, float y)`
`31`	`29`	`{`
`32`	`30`	`return o2::math_utils::fastATan2(-y, -x) + o2::constants::math::PI;`
`33`	`31`	`}`