Bitcoin & Me

bitcoin.core.md

I spend some of my free time on Bitcoin development to learn new things and contribute in the process.

If you like my work, you can donate: bc1q4k2umwjcnkv8h5ldq02apmwn8n64s52m73y8g0 (please drop me an email, if you contribute to avoid tax issues).

bitcoin/bitcoin

Replace boost::filesystem with std::filesystem

Why? Relying on less external dependencies is a good thing.

• [Idea]
• [Prototype PR]: 11 Jun 2020 - 22 Dec 2020
• [Adopted by fanquake] as bitcoin/bitcoin#20744
  • PRs that should make it easier to merge the fanquake's PR:
    • [PR] - refactor: "Replace fs::unique_path with GetUniquePath(path) calls" Merged ✓
    • [PR] - test: Change BOOST_CHECK to BOOST_CHECK_EQUAL for paths ✓
  • Help with review of #22937
    • bitcoin/bitcoin#22937 (comment)
    • bitcoin/bitcoin#22937 (comment)
  • Additional attempts to help with 20744:
    • bitcoin/bitcoin#20744 (comment)
    • bitcoin/bitcoin#20744 (comment)
    • bitcoin/bitcoin#20744 (comment)
    • bitcoin/bitcoin#20744 (comment)
    • bitcoin/bitcoin#20744 (comment)

GetDataDir & ArgsManager

Why? GetDataDir() function relies on the global state. This is hard to think about in general (for me). Moving this function to ArgsManager and passing ArgsManager to classes can have a similar (yet much smaller) benefit as tree-wide: De-globalize ChainstateManager.

• [issue: Remove gArgs]
• [PR] - Move GetDataDir to ArgsManager ✓
• [PR] - Remove GetDataDir(net_specific) function ✓
• [PR] - Remove gArgs from wallet.h and wallet.cpp ✓
• [PR] - refactor: Remove gArgs from wallet.h and wallet.cpp (2) ✓
• [PR] - refactor: Remove gArgs from bdb.h and sqlite.h ✓

Add EditorConfig file

Why? .editorconfig can help avoid some code formatting issues and thus less Cirrus CI runs may be needed to merge a Bitcoin Core PR.

• [PR]: Merged ✓

Type hints & Python scripts

Why? Type hints are helpful to understand Python code better (subjectively). This may prove useful for new Bitcoin contributors.

• [PR]: Merged ✓

Feerate histogram

Why? It seems like a great feature to have and the work is sponsored by zkSNACKs 🙏

• PR: Rejected

rpc: Add level 3 verbosity to getblock RPC call.

Why? This work is sponsored by zkSNACKs 🙏 and I enjoy learning more about this stuff.

• I attempted to document the PR a bit: bitcoin/bitcoin#21245 (comment)
• Proposed minor improvements: bitcoin/bitcoin#21245 (comment)
• ... and attempted to polish the PR so that there's bigger chance the PR gets merged.
• Adopted #21245 by creating a new PR #22918 which is merged now.

PRs:

• [#22918]: Merged ✓
• [#23320] (rpc: Add RPC help for verbosity level 3): Merged ✓

Reviews

Github has a list of PRs where I did some review work.

rust-bitcoin/rust-miniscript

https://github.com/rust-bitcoin/rust-miniscript/pulls?q=author%3Akiminuo+is%3Amerged some minor stuff :)

zkSnacks/WasabiWallet

Hand-picked contributions to the privacy oriented Bitcoin wallet are here.