scripted-diff: fix leftover references to `policy/fees.h` #33864

pull ismaelsadeeq wants to merge 1 commits into bitcoin:master from ismaelsadeeq:11-2025-fix-stale-reference-fees.h changing 6 files +6 −6
  1. ismaelsadeeq commented at 5:43 PM on November 12, 2025: member

    Fixes #33863

    ryanofsky wrote

    I still see some references to the src/policy/fees.h file removed by this PR:

    $ git grep -n policy/fees.h
    src/wallet/rpc/spend.cpp:206: * [@param](/bitcoin-bitcoin/contributor/param/)[in]     conf_target       UniValue integer; confirmation target in blocks, values between 1 and 1008 are valid per policy/fees.h;
    test/functional/rpc_estimatefee.py:39:        # max value of 1008 per src/policy/fees.h
    test/functional/rpc_psbt.py:604:                assert_raises_rpc_error(-8, "Invalid conf_target, must be between 1 and 1008",  # max value of 1008 per src/policy/fees.h
    test/functional/wallet_basic.py:337:            assert_raises_rpc_error(-8, "Invalid conf_target, must be between 1 and 1008",  # max value of 1008 per src/policy/fees.h
    test/functional/wallet_fundrawtransaction.py:851:                assert_raises_rpc_error(-8, "Invalid conf_target, must be between 1 and 1008",  # max value of 1008 per src/policy/fees.h
    test/functional/wallet_send.py:315:                expect_error=(-8, "Invalid conf_target, must be between 1 and 1008"))  # max value of 1008 per src/policy/fees.h
    

    This is fixed in this PR by running a script that searches for what he greps and replaces it with the right reference.

    git grep -l "policy\/fees\.h" | xargs sed -i "s/policy\/fees.h/policy\/fees\/block_policy_estimator.h/g"
    
  2. scripted-diff: fix leftover references to `policy/fees.h`
    -BEGIN VERIFY SCRIPT-
    git grep -l "policy\/fees\.h" | xargs sed -i "s/policy\/fees.h/policy\/fees\/block_policy_estimator.h/g"
    -END VERIFY SCRIPT-
    b0a3887154
  3. DrahtBot added the label Refactoring on Nov 12, 2025
  4. ismaelsadeeq marked this as ready for review on Nov 12, 2025
  5. DrahtBot commented at 5:44 PM on November 12, 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/33864.

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK kevkevinpal, janb84, rkrux
    Concept ACK vicjuma

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

    <!--5faf32d7da4f0f540f40219e4f7537a3-->

  6. kevkevinpal commented at 6:21 PM on November 12, 2025: contributor

    ACK b0a3887

    A doc change and scripted-diff are pretty straightforward

  7. janb84 commented at 7:44 PM on November 12, 2025: contributor

    ACK b0a38871546dfcdd3a578c1ae4c28a88b6ee32d5

    Trivial cleanup PR. This time I did some extra code searches to find any possible leftovers, could not find one.

  8. vicjuma commented at 8:21 PM on November 12, 2025: contributor

    ACK the OP’s latest commit.

    Confirmed that git grep "policy/fees.h" does not return any lines, meaning that the solution works.

    git grep -l "policy/fees.h" | xargs sed -i "s/policy/fees.h/policy/fees/block_policy_estimator.h/g"

  9. rkrux approved
  10. rkrux commented at 7:24 AM on November 13, 2025: contributor

    lgtm ACK b0a38871546dfcdd3a578c1ae4c28a88b6ee32d5

  11. fanquake merged this on Nov 13, 2025
  12. fanquake closed this on Nov 13, 2025

  13. ryanofsky commented at 1:03 PM on November 13, 2025: contributor

    Thanks for the cleanup!

  14. ismaelsadeeq deleted the branch on Nov 14, 2025
  15. yuvicc referenced this in commit f620a2206b on Nov 16, 2025
  16. stringintech referenced this in commit 52fffa1e5d on Nov 24, 2025
  17. alexanderwiederin referenced this in commit 7334556a3f on Dec 1, 2025
  18. stickies-v referenced this in commit 9fa736f8e4 on Jan 22, 2026
  19. Kino1994 referenced this in commit 3e61d10220 on Jun 28, 2026
  20. BigcoinBGC referenced this in commit f8d4e9e277 on Jun 30, 2026
  21. Kino1994 referenced this in commit b3d41c0942 on Aug 19, 2026
  22. bitcoin locked this on Nov 14, 2026

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: 2026-11-15 02:51 UTC

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