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
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-
Chand-ra commented at 2:57 PM on March 10, 2025: none
-
1030949440
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
-
DrahtBot commented at 2:57 PM on March 10, 2025: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--006a51241073e994b41acfe9ec718e94-->
Code Coverage & Benchmarks
For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32024.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process. A summary of reviews will appear here.
- DrahtBot added the label Tests on Mar 10, 2025
-
dergoegge commented at 3:56 PM on March 10, 2025: member
Thanks for your contribution but
FuzzedDataProvider.his not maintained by us. This should be opened against llvm: https://github.com/llvm/llvm-project/blob/main/compiler-rt/include/fuzzer/FuzzedDataProvider.h. - hebasto closed this on Mar 10, 2025
- bitcoin locked this on Mar 10, 2025
