-
Notifications
You must be signed in to change notification settings - Fork 874
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
add more info for invalid opcode #4183
Conversation
✅ Deploy Preview for remixproject ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@@ -83,7 +83,7 @@ export function checkError (execResult, compiledContracts) { | |||
const error = `Error occured: ${execResult.errorMessage}.\n` | |||
let msg = '' | |||
if (exceptionError === errorCode.INVALID_OPCODE) { |
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.
in my case, exceptionError
was invalid opcode: PUSH0
so it didn't get in that "if". Is this possible to handle that case? just using includes
might just do the trick.
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.
Done
fixes #4079
and fixes #4071