Oscar Guindzberg

Always use ConfidenceChange listener

9

If connected to 1 peer, it will be disconnected and reconnected after the broadcast, so we in fact will hear the inv from that peer. Otherwise, when connected to just...

StoredBlock.serializeCompact() trims AuxPoW

4

See https://github.com/bitcoinj/bitcoinj/blob/v0.14.2/core/src/main/java/org/bitcoinj/core/StoredBlock.java#L131 This method is invoked by bitcoinj's blockstores (eg LevelDBBlockStore) before storing a block, so AuxPow is missing from blocks stored in blockstores.

Unlock tx are forced to have a change output

1

DogeTx.parseTransaction() fails for an unlock tx if the tx does not contains a change output sending doges to an operator. DogeToken.processTransaction() needs to know the value sent to the operator,...

DogeClaimManager requires 6.5 million gas to deploy

1

Split it into several contracts

Rename claimId to [superblockId or superblockHash]

1

claimId and superblockId are now the same id, but it is confusing to have both.

Timestamp uses uint256 but a smaller type should be enough Other fields have the same problem

Do this by comparing block gasUsed and blockGasLimit values

Operators nor superblock submitters have a fixed cost for letting a user cross the bridge. So, there is no need to pass on a minimum cost to users. Just a...