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
  1. MarcoFalke commented at 12:28 pm on August 2, 2021: member
  2. Add missing const to CAddrMan::Check_()
    Also: Always compile the function signature to avoid similar issues in
    the future.
    fa8629b98e
  3. MarcoFalke closed this on Aug 2, 2021

  4. MarcoFalke deleted the branch on Aug 2, 2021
  5. jnewbery commented at 1:05 pm on August 2, 2021: member
    I’ve cherry picked this commit into #20233
  6. 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 from src/addrman.h.
  7. 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 site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me