fuzz: Increase branch coverage of the float fuzz target #22180

pull MarcoFalke wants to merge 2 commits into bitcoin:master from MarcoFalke:2106-fuzzFloat changing 2 files +29 −2
  1. MarcoFalke commented at 11:55 am on June 7, 2021: member

    Currently the branch coverage for the float fuzz target is only 50% : https://marcofalke.github.io/btc_cov/fuzz.coverage/src/test/fuzz/float.cpp.gcov.html

    This is caused by the Fuzzed Data Provider only picking “nice” floats.

  2. fuzz: Remove confusing return keyword from CallOneOf
    The return type is already enforced to be void by the
    ternary operator:
    
    ./test/fuzz/util.h:47:25: error: right operand to ? is void, but left operand is of type *OTHER_TYPE*
        ((i++ == call_index ? callables() : void()), ...);
                            ^ ~~~~~~~~~~~   ~~~~~~
    fad0c58c3e
  3. fuzz: Increase branch coverage of the float fuzz target fa13f34bf3
  4. fanquake added the label Tests on Jun 7, 2021
  5. practicalswift commented at 8:15 pm on June 7, 2021: contributor

    Good catch!

    cr ACK fa13f34bf35129b38af699a0faf32c39d2ba8576: patch looks correct

  6. MarcoFalke merged this on Jun 8, 2021
  7. MarcoFalke closed this on Jun 8, 2021

  8. sidhujag referenced this in commit f2f568a26e on Jun 9, 2021
  9. MarcoFalke deleted the branch on Jun 14, 2021
  10. gwillen referenced this in commit aad3d400b8 on Jun 1, 2022
  11. 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-11-23 15:12 UTC

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