the function appears to be unused throughout the codebase
./src/script/signingprovider.cpp:248:std::set<CScriptID> FillableSigningProvider::GetCScripts() const
./src/script/signingprovider.h:330: virtual std::set<CScriptID> GetCScripts() const;
script: remove unused GetCScripts #35921
pull vicjuma wants to merge 1 commits into bitcoin:master from vicjuma:remove-unused-getcscripts changing 2 files +0 −11-
vicjuma commented at 9:54 PM on August 6, 2026: none
-
77f976cbc4
script: remove unused GetCScripts
the function is not used in the codebase
- DrahtBot added the label Consensus on Aug 6, 2026
-
DrahtBot commented at 9:54 PM on August 6, 2026: 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/35921.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline and AI policy for information on the review process. A summary of reviews will appear here.
<!--5faf32d7da4f0f540f40219e4f7537a3-->
- vicjuma closed this on Aug 6, 2026
