Skip to content

[pull] master from bitcoinj:master#97

Merged
pull[bot] merged 17 commits intoAll-Blockchains:masterfrom
bitcoinj:master
Mar 17, 2026
Merged

[pull] master from bitcoinj:master#97
pull[bot] merged 17 commits intoAll-Blockchains:masterfrom
bitcoinj:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Mar 17, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

…lockStore, SPVBlockStore: annotate nullable fields and methods
It is more idiomatic to use `this.` in an overridden method and
it avoids a reference to a nullable field.
Using `setChainHead()` breaks a non-null guarantee because it can throw,
so we set it directly.
By handling the `FileNotFoundException` from creating the `RandomAccessFile`
separately we are able to guarantee that `SPVBlockStore` will not be
successfully constructed without `randomAccessFile` being initialized.

This also guarantees in the second catch block that `randomAccessFile`
is not null.
This will clear the block cache even if an IO error occurs.
…d` directly in constructor

The `setChainHead()` and `setVerifiedChainHead()` methods are not necessary
at object initialization time and declare exceptions that break compiler
guarantees of initialization and non-nullability.
…st: make implicit nullability assumptions explicit
@pull pull bot locked and limited conversation to collaborators Mar 17, 2026
@pull pull bot added the ⤵️ pull label Mar 17, 2026
@pull pull bot merged commit 3272e5e into All-Blockchains:master Mar 17, 2026
16 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant