Optimization idea from #171 (review):
Maintain a HashMap<txid, confirmations>. This would replace both the unmined_txids and mined_txids HashSet with a single map, while avoiding repeated get_transaction_confirmations RPC calls during sequences like MineBlock -> RecvChannelReady until state.config.minimum_depth is reached
Optimization idea from #171 (review):