- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
All Stories
Jan 19 2021
Stick with original PR content.
API optimization can still happen later if needed.
Jan 18 2021
Fix typo in Diff description
The config is needed down the road is just saying nothing at all.
relayout comments
Fix tests, better variable names
@deadalnix I added a line with the rationale in the summary:
The config is needed down the road, and provides a facility to retrieve the chainparams, so storing it makes it possible to have a simpler API..
Let me know if you think it's not worth it.
You say that it differs from the original by passing a config, but provide no reason as to why. It doesn't looks like to me that this difference is justified.
@bot build-linux32 build-linux-arm build-linux-aarch64
removed commented leftover debugging line and adapted the test for Coin::DynamicMemoryUsage() on 32-bit