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

Debug assert results in a panic when no gas #115

Open
davidiw opened this issue Mar 10, 2022 · 0 comments
Open

Debug assert results in a panic when no gas #115

davidiw opened this issue Mar 10, 2022 · 0 comments
Assignees

Comments

@davidiw
Copy link
Contributor

davidiw commented Mar 10, 2022

We're exploring gas, and I just submitted a transaction with no gas associated. It hits this debug_assert. Any objections to removing it? And/or putting in some logging line if it is valuable still?

sahithiacn pushed a commit to sahithiacn/move that referenced this issue Jan 12, 2023
Copying over the existing VM spec from the old repo. This makes no attempt to update it with changes since it was originally written (e.g. new bytecode instructions), but it is useful to have here nonetheless.

Signed-off-by: sahithiacn <[email protected]>
bors-diem pushed a commit that referenced this issue Jan 12, 2023
Copying over the existing VM spec from the old repo. This makes no attempt to update it with changes since it was originally written (e.g. new bytecode instructions), but it is useful to have here nonetheless.

Signed-off-by: sahithiacn <[email protected]>
Closes: #335
bors-diem pushed a commit that referenced this issue Jan 13, 2023
Copying over the existing VM spec from the old repo. This makes no attempt to update it with changes since it was originally written (e.g. new bytecode instructions), but it is useful to have here nonetheless.

Signed-off-by: sahithiacn <[email protected]>
Closes: #335
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

No branches or pull requests

4 participants