refactor: use NodeClock for m_headers_sync_timeout #35329

pull kevkevinpal wants to merge 1 commits into bitcoin:master from kevkevinpal:refactor-m_headers_sync_timeout changing 1 files +7 −9

kevkevinpal commented at 12:18 PM on May 20, 2026: contributor

Summary

This is a follow-up motivated by #34882#pullrequestreview-4113223083 and #34882 (comment)

Please let me know if there is another PR already covering this and I can close this but I checked and didn't see anything.

Currently, m_headers_sync_timeout does not use NodeClock, so this refactors it to instead use NodeClock
refactor: use NodeClock for m_headers_sync_timeout 00ce7fd7a1
DrahtBot added the label Refactoring on May 20, 2026
DrahtBot commented at 12:18 PM on May 20, 2026: 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/35329.



Reviews

See the guideline for information on the review process. A summary of reviews will appear here.
fanquake commented at 12:19 PM on May 20, 2026: member

Duplicate of #35315?
fanquake closed this on May 20, 2026
kevkevinpal commented at 12:23 PM on May 20, 2026: contributor

Duplicate of #35315?

ah yes it is thanks!

Contributors

Labels

Summary