What the developer wrote
Authored by MarcoFalke
test: Remove outdated comment
The serialize related methods were removed in commit
30007fda76aa7ba4e4090f7a16298874a7722926.
If someone wants to see the tested methods, they can just read the test
itself, instead of relying on the wrong comment.
What changed, and why it matters
This commit only removes an outdated comment in a test file. It changes no actual code behavior and has no security relevance.
No action needed. This is a non-functional documentation cleanup in test code.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit updates a single line in src/test/arith_uint256_tests.cpp, removing a stale comment listing methods (GetHex, operator=, size(), GetLow64, GetSerializeSize, Serialize, Unserialize) that were previously tested in this BOOST_AUTO_TEST_CASE. The referenced prior commit removed the serialize-related methods, making the comment inaccurate. No executable code was modified.
Changed components
src/test/arith_uint256_tests.cppInspect captured patch +1 / −1
diff --git a/src/test/arith_uint256_tests.cpp b/src/test/arith_uint256_tests.cpp
index 37a39adb..1dca87cf 100644
--- a/src/test/arith_uint256_tests.cpp
+++ b/src/test/arith_uint256_tests.cpp
@@ -368,7 +368,7 @@ static bool almostEqual(double d1, double d2)
return fabs(d1-d2) <= 4*fabs(d1)*std::numeric_limits<double>::epsilon();
}
-BOOST_AUTO_TEST_CASE(methods) // GetHex operator= size() GetLow64 GetSerializeSize, Serialize, Unserialize
+BOOST_AUTO_TEST_CASE(methods)
{
BOOST_CHECK(R1L.GetHex() == R1L.ToString());
BOOST_CHECK(R2L.GetHex() == R2L.ToString());
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.