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

Fix tracing failed calls. #7412

Merged
merged 1 commit into from
Jan 2, 2018
Merged

Fix tracing failed calls. #7412

merged 1 commit into from
Jan 2, 2018

Conversation

tomusdrw
Copy link
Collaborator

Closes #7187

@tomusdrw tomusdrw added A0-pleasereview 🤓 Pull request needs code review. M4-core ⛓ Core client code / Rust. B0-patch labels Dec 29, 2017
@5chdn 5chdn added this to the Patch milestone Dec 30, 2017
@5chdn 5chdn added the P5-sometimesoon 🌲 Issue is worth doing soon. label Dec 30, 2017
@@ -21,3 +21,5 @@ rustc-hex = "1.0"

[features]
jit = ["evmjit"]
evm-debug = []
evm-debug-tests = ["evm-debug"]
Copy link
Collaborator

Choose a reason for hiding this comment

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

why introducing a new feature? is it used anywhere else? I can't see

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It is part of evm crate now and used to be part of ethcore - the feature was not correctly migrated when those two were split. It prints out VMtracing when running interpreter tests and is quite helpful.

@debris debris added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Jan 2, 2018
@debris debris merged commit 6a01113 into master Jan 2, 2018
@debris debris deleted the td-trace-failed-calls branch January 2, 2018 14:43
@fanatid fanatid mentioned this pull request Jan 2, 2018
@debris debris mentioned this pull request Jan 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well. M4-core ⛓ Core client code / Rust. P5-sometimesoon 🌲 Issue is worth doing soon.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants