Сryptocurrency development analysis

Description

Somebody decided to read in the yaml configuration at compile time, and then use it to give reflection configurations `HasConfigurations`. That’s ridiculous, so it’s been removed. Formerly broken benchmarks now use the `benchProtocolConstants` and `benchProtocolMagic`, values sourced from what they were when read in from `lib/configuration.yaml` `test` section.

## Linked issue

## Type of change

– [ ] 🐞 Bug fix (non-breaking change which fixes an issue)
– [ ] 🛠 New feature (non-breaking change which adds functionality)
– [ ] ⚠️ Breaking change (fix or feature that would cause existing functionality to change)
– [ ] 🏭 Refactoring that does not change existing functionality but does improve things like code readability, structure etc
– [ ] 🔨 New or improved tests for existing code
– [ ] ⛑ git-flow chore (backport, hotfix, etc)

## Developer checklist

– [ ] I have read the [style guide](https://github.com/input-output-hk/cardano-sl/blob/develop/docs/style-guide.md) document, and my code follows the code style of this project.
– [ ] If my code deals with exceptions, it follows the [guidelines](https://github.com/input-output-hk/cardano-sl/blob/develop/docs/exceptions.md).
– [ ] I have updated any documentation accordingly, if needed. Documentation changes can be reflected in opening a PR on [cardanodocs.com](https://github.com/input-output-hk/cardanodocs.com), amending the inline [Haddock](https://www.haskell.org/haddock/) comments, any relevant README file or one of the document listed in the [docs](https://github.com/input-output-hk/cardano-sl/tree/develop/docs) directory.

## Testing checklist

– [ ] I have added tests to cover my changes.
– [ ] All new and existing tests passed.

## QA Steps

## Screenshots (if available)

You can view, comment on, or merge this pull request online at:

https://github.com/input-output-hk/cardano-sl/pull/3092

— Commit Summary —

* make cardano-sl benchmarks compile

— File Changes —

M block/src/Pos/Block/Lrc.hs (4)
M core/src/Pos/Core/Slotting/LocalSlotIndex.hs (8)
M lib/bench/Bench/Configuration.hs (31)
M lib/bench/Bench/Pos/Criterion/FollowTheSatoshiBench.hs (11)
M lib/bench/Bench/Pos/Criterion/TxSigningBench.hs (13)
M lib/src/Pos/DB/DB.hs (11)
M lib/src/Pos/Launcher/Launcher.hs (3)
M lrc/src/Pos/Lrc/DB/Leaders.hs (8)
M lrc/src/Pos/Lrc/DB/Lrc.hs (7)
M lrc/src/Pos/Lrc/Fts.hs (24)
M lrc/src/Pos/Lrc/Genesis.hs (16)

— Patch Links —

https://github.com/input-output-hk/cardano-sl/pull/3092.patch
https://github.com/input-output-hk/cardano-sl/pull/3092.diff

This post was last modified on June 14, 2018, 5:11 pm

Categories: cryptocurrency