VerifyPeak: use floating point for error abs to keep static analyzer happy even though we don't care about the fractional part of the value

azonenberg · azonenberg · commit 491245d05f0a · 2026-05-17T17:07:08.000-07:00
diff --git a/tests/Primitives/DualFFT.cpp b/tests/Primitives/DualFFT.cpp
@@ -164,7 +164,7 @@ void VerifyPeak(
 		uhz.PrettyPrint(peak_uhz, 6).c_str(),
 		uhz.PrettyPrint(expectedPeakUhz, 6).c_str(),
 		uhz.PrettyPrint(error_uhz, 6).c_str());
-	REQUIRE(labs(error_uhz) < toleranceUhz);
+	REQUIRE(fabs(error_uhz) < toleranceUhz);
 }
 
 size_t FindPeak(AcceleratorBuffer<float>& buf, size_t start, size_t end)

Original file line number	Diff line number	Diff line change
`@@ -164,7 +164,7 @@ void VerifyPeak(`
`164`	`164`	`uhz.PrettyPrint(peak_uhz, 6).c_str(),`
`165`	`165`	`uhz.PrettyPrint(expectedPeakUhz, 6).c_str(),`
`166`	`166`	`uhz.PrettyPrint(error_uhz, 6).c_str());`
`167`		`- REQUIRE(labs(error_uhz) < toleranceUhz);`
	`167`	`+ REQUIRE(fabs(error_uhz) < toleranceUhz);`
`168`	`168`	`}`
`169`	`169`
`170`	`170`	`size_t FindPeak(AcceleratorBuffer<float>& buf, size_t start, size_t end)`