refactor: Remove unused #include in common/system
What the developer wrote
Authored by MarcoFalke
refactor: Remove unused #include in common/system
Now that clamping to std::numeric_limits<size_t>::max() is removed, the limits include is no longer needed.
Similarly after changing size_t -> u64, the cstddef include needs to be replaced by cstdint.
What changed, and why it matters
This is a minor code cleanup that removes one unused header file include and swaps another for a more specific one. It does not change any program behavior or fix any security issue.
No security action needed. This is a routine refactoring commit.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit removes #include <limits> from src/common/system.cpp and replaces #include <cstddef> with #include <cstdint> in src/common/system.h. The commit message explains these includes are no longer needed after prior changes (removal of clamping to numeric_limits
Changed components
src/common/system.cppsrc/common/system.hInspect captured patch +1 / −2
diff --git a/src/common/system.cpp b/src/common/system.cpp
index dc55617d..72bf9da9 100644
--- a/src/common/system.cpp
+++ b/src/common/system.cpp
@@ -28,7 +28,6 @@
#include <cstdint>
#include <cstdlib>
-#include <limits>
#include <locale>
#include <optional>
#include <stdexcept>
diff --git a/src/common/system.h b/src/common/system.h
index c1f5f0ab..ad217e0d 100644
--- a/src/common/system.h
+++ b/src/common/system.h
@@ -10,7 +10,7 @@
#include <util/time.h>
-#include <cstddef>
+#include <cstdint>
#include <optional>
#include <string>
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.