Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

evm #52

Merged
merged 59 commits into from
Jan 11, 2016
Merged

evm #52

merged 59 commits into from
Jan 11, 2016

Conversation

debris
Copy link
Collaborator

@debris debris commented Dec 23, 2015

No description provided.

@debris debris changed the title evm adapter [in progress] evm [please review] Dec 28, 2015
use util::uint::*;
use state::*;

pub struct Env {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing docs.

@debris debris changed the title evm [please review] evm Jan 7, 2016
@@ -293,10 +386,10 @@ pub mod ffi {
}

#[link(name="evmjit")]
// EnvHandle does not have to by a C type
// ExtHandle does not have to by a C type
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

have to what?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be ExtHandle is not a C type, so we need to allow "improper_ctypes". Will fix it

arkpar added a commit that referenced this pull request Jan 11, 2016
@arkpar arkpar merged commit 81c2fa0 into master Jan 11, 2016
gavofyork pushed a commit that referenced this pull request Jan 17, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants