-
Notifications
You must be signed in to change notification settings - Fork 198
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
registerVM-gasSchedule-notifier #3491
Conversation
Codecov Report
@@ Coverage Diff @@
## development #3491 +/- ##
===============================================
- Coverage 73.86% 73.85% -0.01%
===============================================
Files 581 581
Lines 73843 73847 +4
===============================================
- Hits 54543 54541 -2
- Misses 14927 14930 +3
- Partials 4373 4376 +3
Continue to review full report at Codecov.
|
0c86eb4
@@ -131,7 +131,9 @@ func (g *gasScheduleNotifier) EpochConfirmed(epoch uint32, _ uint64) { | |||
|
|||
g.lastGasSchedule = newGasSchedule | |||
for _, handler := range g.handlers { | |||
handler.GasScheduleChange(g.lastGasSchedule) | |||
if !check.IfNil(handler) { |
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.
this is a safe check but a nil handler could never be added in the handlers as the RegisterNotifyHandler function has already a nil check for each handler it adds
7cfffce
Register VM to gasschedule notifier