-
Notifications
You must be signed in to change notification settings - Fork 306
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
Go bindings: add missing error codes #72
Conversation
bindings/go/evmc/evmc.go
Outdated
@@ -120,7 +126,7 @@ func (err Error) Error() string { | |||
return fmt.Sprintf("evmc: unknown internal error (%d)", int32(code)) |
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.
Maybe remove "unknown" from here as we do not predefine any internal errors at the moment.
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.
Perhaps "specific internal error" ? "user-defined internal error" ?
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.
I'd go with "internal error" only.
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.
Then the other internal error
should also have a value IIRC.
bindings/go/evmc/evmc.go
Outdated
@@ -110,6 +110,12 @@ func (err Error) Error() string { | |||
return "evmc: precompile failure" | |||
case C.EVMC_CONTRACT_VALIDATION_FAILURE: | |||
return "evmc: contract validation failure" | |||
case C.EVMC_ARGUMENT_OUT_RANGE: |
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.
Should be EVMC_ARGUMENT_OUT_OF_RANGE
. See Go CI builds.
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.
Fixed already, just didn't push before the other one is decided.
No description provided.