Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix balanceOfStart usage. #327

Merged
merged 2 commits into from
Jun 6, 2022
Merged

Fix balanceOfStart usage. #327

merged 2 commits into from
Jun 6, 2022

Conversation

adlerjohn
Copy link
Contributor

@adlerjohn adlerjohn commented Jun 6, 2022

Fixes #228.

balanceOf wasn't actually a thing, so changed to balanceOfStart. Should be no functional change (panics on the helper functions are fully captured by other panic conditions).

@adlerjohn adlerjohn added bug Something isn't working documentation Improvements or additions to documentation comp:FVM Component: FuelVM labels Jun 6, 2022
@adlerjohn adlerjohn self-assigned this Jun 6, 2022
@adlerjohn adlerjohn changed the title Fix balanceOfStart usage. Fix balanceOfStart usage. Jun 6, 2022
@adlerjohn adlerjohn merged commit f32f5dd into master Jun 6, 2022
@adlerjohn adlerjohn deleted the adlerjohn/fix-balance-of branch June 6, 2022 19:31
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working comp:FVM Component: FuelVM documentation Improvements or additions to documentation
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Clarify use of balanceOf, balanceOfStart, and balance helpers
2 participants