-
Notifications
You must be signed in to change notification settings - Fork 143
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move block validation code to BlockChain<T> #1010
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left comments, that are all trivial suggestions; feel free to ignore them.
<TargetFramework>netcoreapp3.0</TargetFramework> | ||
</PropertyGroup> | ||
|
||
<PropertyGroup Condition=" '$(MSBuildRuntimeType)'=='Mono' And | ||
'$(BuildingByReSharper)'!='true'"> | ||
<PropertyGroup Condition=" '$(MSBuildRuntimeType)'=='Mono' And '$(BuildingByReSharper)'!='true'"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes in this file are apparently unnecessary.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right, but I changed it because Rider modifies this file whenever I add or remove files.
Co-authored-by: Hong Minhee (洪 民憙) <hong.minhee@gmail.com>
ab7fa85
to
11d8bfe
Compare
Codecov Report
@@ Coverage Diff @@
## main #1010 +/- ##
==========================================
- Coverage 88.94% 88.89% -0.05%
==========================================
Files 329 330 +1
Lines 29892 29883 -9
==========================================
- Hits 26586 26565 -21
- Misses 1741 1748 +7
- Partials 1565 1570 +5
|
`BlockPolicy.ValidateNextBlock` has been become not to throw `InvalidBlockStateRootHashException` since planetarium#1010
`BlockPolicy.ValidateNextBlock` has been become not to throw `InvalidBlockStateRootHashException` since planetarium#1010 [skip changelog]
`BlockPolicy.ValidateNextBlock` has been become not to throw `InvalidBlockStateRootHashException` since planetarium#1010 [skip changelog]
This moves block validation code to
BlockChain<T>
fromBlockPolicy<T>
and also the methods ofBlockPolicy<T>
became to virtual so that it can be overridden.