Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

pow: re-add support for algorithms where only linear verification is possible#4843

Merged
sorpaas merged 3 commits intomasterfrom
sp-pow-fix
Feb 7, 2020
Merged

pow: re-add support for algorithms where only linear verification is possible#4843
sorpaas merged 3 commits intomasterfrom
sp-pow-fix

Conversation

@sorpaas
Copy link
Copy Markdown
Contributor

@sorpaas sorpaas commented Feb 6, 2020

This restores the API of PoW engine after #4652. No-parent verification now again becomes optional (called preliminary_verify). This is needed because for some algorithms only linear verification is possible (for PoW algorithms that just relies on parent block information). Also fixed some unused generic parameters.

@sorpaas sorpaas added A0-please_review Pull request needs code review. B2-breaksapi labels Feb 6, 2020
@sorpaas sorpaas requested review from bkchr and gnunicorn February 6, 2020 13:55
sp_consensus::Error
> where
B: BlockT,
C: ProvideRuntimeApi<B> + HeaderBackend<B> + BlockOf + ProvideCache<B> + AuxStore,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder how many of those generic bounds never get deleted elsewhere

@sorpaas sorpaas merged commit 4944bd1 into master Feb 7, 2020
@sorpaas sorpaas deleted the sp-pow-fix branch February 7, 2020 10:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A0-please_review Pull request needs code review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants