diem / diem/move

Debug assert results in a panic when no gas

Open
#115 0 comments 0 reactions 3 assignees Claimed by @wrwg View on GitHub
Dominant language
Rust
Stars
378
Forks
137
PR merge metrics
No merged PRs in 30d

Description

https://github.com/diem/move/blob/3e7db9589b1d584db35d5f49a4970984403f38d6/language/move-binary-format/src/errors.rs#L70

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?

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.