highperfocused/bitcoin
1
0
Fork 0
mirror of https://github.com/bitcoin/bitcoin.git synced 2026-03-24 22:45:41 +01:00
Code Issues Packages Projects Releases Wiki Activity

Merge bitcoin/bitcoin#33781: clang-tidy: Remove no longer needed NOLINT

Browse Source 
038849e2e0 clang-tidy: Remove no longer needed NOLINT (Hennadii Stepanov)

Pull request description:

  From https://github.com/bitcoin/bitcoin/pull/33714/files#r2491476516:
  > Actually, the `NOLINT` was fixed and can be removed? You've confirmed that it is undeclared on the listed platforms, so it can't be hit by `readability-redundant-declaration`

ACKs for top commit:
  maflcko:
    lgtm ACK 038849e2e0
  l0rinc:
    I wanted to ask the same on the original PR but forgot - ACK 038849e2e0

Tree-SHA512: c0b24235425e80baeac3158c7169122364f31140367bc289430d34f01cd38f9f6a3931319f6fe4e1dc86bc4d87e21a5b4b8a2263c199e8083593f89ce592a177
This commit is contained in:
merge-script
2025-11-05 10:06:36 +00:00
parent ffd7ca3c46 038849e2e0
commit 50d106a4d6
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/randomenv.cpp
View File

@@ -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 {