test: get rid of redundant TODO tag in fuzz tests #32024

pull Chand-ra wants to merge 1 commits into bitcoin:master from Chand-ra:todo-v1 changing 1 files +0 −1
  1. Chand-ra commented at 2:57 pm on March 10, 2025: none

    list.size() is determined at runtime, so using static_assert on it as suggested by the TODO comment is not feasible and produces the following error when done:

    error: static assertion expression is not an integral constant expression

  2. test: get rid of redundant TODO tag in fuzz tests
    'list.size()' is determined at runtime, so using static_assert
    on it as suggested by the TODO comment is not feasible and
    produces the following error when done:
    
    error: static assertion expression is not an integral constant expression
    1030949440
  3. DrahtBot commented at 2:57 pm on March 10, 2025: contributor

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

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32024.

    Reviews

    See the guideline for information on the review process. A summary of reviews will appear here.

  4. DrahtBot added the label Tests on Mar 10, 2025
  5. dergoegge commented at 3:56 pm on March 10, 2025: member
    Thanks for your contribution but FuzzedDataProvider.h is not maintained by us. This should be opened against llvm: https://github.com/llvm/llvm-project/blob/main/compiler-rt/include/fuzzer/FuzzedDataProvider.h.
  6. hebasto closed this on Mar 10, 2025

  7. bitcoin locked this on Mar 10, 2025

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: 2025-03-29 06:12 UTC

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