-
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
Rename evmc_instance -> evmc_vm #430
Conversation
e941715
to
e90c13a
Compare
e90c13a
to
77a2a6e
Compare
{ | ||
ec = set_error(EVMC_LOADER_INSTANCE_CREATION_FAILURE, | ||
"creating EVMC instance of %s has failed", filename); | ||
ec = set_error(EVMC_LOADER_VM_CREATION_FAILURE, "creating EVMC VM of %s has failed", |
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'm not sure about this error message.
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.
would be great if it could mention the name of the function it called, but it doesn't look easy
@@ -408,12 +408,12 @@ class VM | |||
} | |||
|
|||
private: | |||
evmc_instance* m_instance = nullptr; | |||
evmc_vm* m_instance = nullptr; |
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 have left the name m_instance
because m_vm
would be worse. This can also be m_handle
(as in Go) or m_vm_ptr
.
b9f3582
to
1d4e084
Compare
1d4e084
to
28dfad3
Compare
What was the reasoning behind this? |
Looks better. Instance is like |
No description provided.