Skip to content
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

Support null PersistingBlock in NeoToken.DistributeGas #1880

Merged
merged 1 commit into from
Aug 28, 2020
Merged

Support null PersistingBlock in NeoToken.DistributeGas #1880

merged 1 commit into from
Aug 28, 2020

Conversation

devhawk
Copy link
Contributor

@devhawk devhawk commented Aug 27, 2020

When running under the debugger, engine.Snapshot.PersistingBlock will be null. This leads to a null ref exception in NeoToken.DistributeGas. This change updates NeoToken.DistributeGas to be a no-op instead of throwing an exception when engine.Snapshot.PersistingBlock is null.

When running under the debugger, engine.Snapshot.PersistingBlock will be null. This leads to a null ref exception in NeoToken.DistributeGas. This change updates NeoToken.DistributeGas to be a no-op instead of throwing an exception when engine.Snapshot.PersistingBlock is null.
@devhawk devhawk changed the title NeoToken.DistributeGas supports null PersistingBlock supports null PersistingBlock in NeoToken.DistributeGas Aug 27, 2020
@devhawk devhawk changed the title supports null PersistingBlock in NeoToken.DistributeGas Support null PersistingBlock in NeoToken.DistributeGas Aug 27, 2020
@erikzhang erikzhang merged commit 0f4244d into neo-project:master Aug 28, 2020
shargon added a commit that referenced this pull request Aug 28, 2020
ShawnYun pushed a commit to ShawnYun/neo that referenced this pull request Jan 8, 2021
ixje added a commit to CityOfZion/neo-mamba that referenced this pull request Jan 15, 2021
cloud8little pushed a commit to cloud8little/neo that referenced this pull request Jan 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants