IntersectMBO / IntersectMBO/cardano-ledger

Fix `CompactForm DeltaCoin`

Open
#3,317 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Haskell
Stars
295
Forks
179
Avg merge
4d 7h
Merged PRs (30d)
29

Description

Compact form of the `DeltaCoin` isn't really used anywhere, but it is required by the `Val` type class.
The fact that it is not used should not be an excuse for it to be wrong.

`DeltaCoin` is allowed to be negative, but the compact form is backed by `Word64`, which will make negative values overflow upon compaction. This needs to be fixed.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.