Add missing const to CAddrMan::Check_() #22601
pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2108-addrmanConstCheck changing 2 files +14 −11-
MarcoFalke commented at 12:28 pm on August 2, 2021: member
-
Add missing const to CAddrMan::Check_()
Also: Always compile the function signature to avoid similar issues in the future.
-
MarcoFalke closed this on Aug 2, 2021
-
MarcoFalke deleted the branch on Aug 2, 2021
-
in src/addrman.h:747 in fa8629b98e
742 } 743 744-#ifdef DEBUG_ADDRMAN 745 //! Perform consistency check. Returns an error code or zero. 746 int Check_() const EXCLUSIVE_LOCKS_REQUIRED(cs); 747-#endif
jonatack commented at 1:19 pm on August 2, 2021:This patch builds and runs for me. Note, it completely removes the#ifdef DEBUG_ADDRMAN
checks fromsrc/addrman.h
.DrahtBot locked this on Aug 18, 2022
github-metadata-mirror
This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-17 12:12 UTC
This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-17 12:12 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me
More mirrored repositories can be found on mirror.b10c.me