fuzz: Call const member functions in addrman fuzz test only once #21941

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2105-fuzzAddrConst changing 1 files +9 −14
  1. MarcoFalke commented at 10:17 am on May 13, 2021: member

    Logically based on #21940

    Currently the fuzz test may spend a long time generating random numbers:

    Screenshot from 2021-05-13 12-14-09

    Fix that by calling const member functions only once.

    Hopefully fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=34224

  2. DrahtBot added the label Tests on May 13, 2021
  3. DrahtBot commented at 3:22 pm on May 13, 2021: member

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Conflicts

    No conflicts as of last run.

  4. practicalswift commented at 11:46 am on May 15, 2021: contributor
    Concept ACK
  5. MarcoFalke marked this as ready for review on Jun 13, 2021
  6. DrahtBot added the label Needs rebase on Jun 13, 2021
  7. fuzz: Call const member functions in addrman fuzz test only once faf7623106
  8. MarcoFalke force-pushed on Jun 13, 2021
  9. MarcoFalke commented at 11:54 am on June 13, 2021: member
    Rebased to remove the logical dependency. This is still logically based on #21940, but not strictly required for merging.
  10. DrahtBot removed the label Needs rebase on Jun 13, 2021
  11. practicalswift commented at 4:55 pm on June 13, 2021: contributor
    cr ACK faf7623106af2e106a3d9777d714d6afeb2bbd83: touches only src/test/fuzz/addrman.cpp
  12. MarcoFalke merged this on Jun 13, 2021
  13. MarcoFalke closed this on Jun 13, 2021

  14. MarcoFalke deleted the branch on Jun 13, 2021
  15. gwillen referenced this in commit 8923cd37e5 on Jun 1, 2022
  16. DrahtBot locked this on Aug 16, 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-07-01 10:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me