Skip to content

Commit be175ce

Browse files
committed
Merge bitcoin/bitcoin#22557: fuzz: silence a compiler warning about unused CBanEntry comparator
787296e fuzz: silence a compiler warning about unused CBanEntry comparator (Vasil Dimov) Pull request description: ``` test/fuzz/banman.cpp:35:13: warning: unused function 'operator==' [-Wunused-function] static bool operator==(const CBanEntry& lhs, const CBanEntry& rhs) ^ 1 warning generated. ``` See bitcoin/bitcoin#22517 (comment) ACKs for top commit: MarcoFalke: cr ACK 787296e practicalswift: cr ACK 787296e hebasto: ACK 787296e Tree-SHA512: 72e483cef249170160879cf4b69b787fb6c539d61dda423f618e2c5f130bee8c42897487751e5b58e7679cdb0153eb80efcb104e8a85095daa60d47e39ce78b8
2 parents 61cefde + 787296e commit be175ce

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

src/test/fuzz/banman.cpp

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -109,8 +109,9 @@ FUZZ_TARGET_INIT(banman, initialize_banman)
109109
BanMan ban_man_read{banlist_file, /* client_interface */ nullptr, /* default_ban_time */ 0};
110110
banmap_t banmap_read;
111111
ban_man_read.GetBanned(banmap_read);
112-
// Temporarily disabled to allow the remainder of the fuzz test to run while a fix is being worked on:
113-
// assert(banmap == banmap_read);
112+
// Assert temporarily disabled to allow the remainder of the fuzz test to run while a
113+
// fix is being worked on. See https://github.com/bitcoin/bitcoin/pull/22517
114+
(void)(banmap == banmap_read);
114115
}
115116
}
116117
fs::remove(banlist_file.string() + ".dat");

0 commit comments

Comments
 (0)