clang-tidy: Remove no longer needed NOLINT
What the developer wrote
Authored by Hennadii Stepanov
clang-tidy: Remove no longer needed NOLINT
What changed, and why it matters
This is a minor code cleanup change. It removes a comment marker used by an automated linting tool (clang-tidy) because the tool no longer needs it. The actual code behavior is unchanged.
No security action needed. This is a non-functional cleanup commit.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit removes a NOLINT(readability-redundant-declaration) suppression comment from the declaration extern char** environ; in src/randomenv.cpp. The declaration itself remains identical and functional. This is purely a linting/comment cleanup with no executable code change.
Changed components
src/randomenv.cppInspect captured patch +1 / −1
diff --git a/src/randomenv.cpp b/src/randomenv.cpp
index 3aff81d1..829a610f 100644
--- a/src/randomenv.cpp
+++ b/src/randomenv.cpp
@@ -62,7 +62,7 @@
defined(__NetBSD__) || \
defined(__OpenBSD__) || \
defined(__illumos__)
-extern char** environ; // NOLINT(readability-redundant-declaration): Necessary on the above platforms
+extern char** environ; // Necessary on the above platforms
#endif
namespace {
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.