IntersectMBO / ouroboros-consensus

Implementation of a Consensus Layer for the Ouroboros family of protocols
https://ouroboros-consensus.cardano.intersectmbo.org
Apache License 2.0
36 stars 23 forks source link

ChainSelection could pass around a `ExtLedgerState` instead of an STM action for it #1327

Open jasagredo opened 4 days ago

jasagredo commented 4 days ago

As noted by Alex here, it seems reasonable that we can pass around an ExtLedgerState because the ChainSelection is the only component that can alter that one, so the STM action will always return the same value.

However in order not to risk the implementation, we followed what was already there. Lets try it after the merge.