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

Tracing implemented. #772

Merged
merged 15 commits into from Mar 19, 2016
Merged

Tracing implemented. #772

merged 15 commits into from Mar 19, 2016

Conversation

gavofyork
Copy link
Contributor

TODO:

  • track output;
  • track depth;
  • usher through to level higher than OpenBlock;
  • tests.

TODO:
- make it optional;
- track output;
- usher through to level higher than ExecutionResult.
@gavofyork gavofyork added the A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. label Mar 18, 2016
@gavofyork gavofyork mentioned this pull request Mar 18, 2016
3 tasks
@gavofyork gavofyork added A0-pleasereview 🤓 Pull request needs code review. and removed A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Mar 19, 2016
let mut output = vec![];

// execute
let (res, callcreates) = {
let mut ex = TestExt::new(&mut state,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed whitespace

}

impl<'a> Externalities<'a> {
/// Basic `Externalities` constructor.
pub fn new(state: &'a mut State,
env_info: &'a EnvInfo,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

whitespace mainly.

@arkpar arkpar added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Mar 19, 2016
gavofyork added a commit that referenced this pull request Mar 19, 2016
@gavofyork gavofyork merged commit c729f9d into master Mar 19, 2016
@gavofyork gavofyork deleted the tracing branch March 19, 2016 22:52
@arkpar arkpar added this to the 1.1 Civility milestone Mar 20, 2016
@gavofyork gavofyork removed this from the 1.1 Civility milestone Mar 21, 2016
@gavofyork gavofyork removed this from the 1.1 Civility milestone Mar 21, 2016
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants