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

tracing backport #1770

Merged
merged 6 commits into from
Jul 30, 2016
Merged

tracing backport #1770

merged 6 commits into from
Jul 30, 2016

Conversation

debris
Copy link
Collaborator

@debris debris commented Jul 29, 2016

debris added 2 commits July 29, 2016 13:57
* tracing suicide

* fixed #1635

* fixed typo
* flat trace serialization

* tracing finds transaction which creates contract

* flatten traces before inserting them to the db
@debris debris added the A8-backport 🕸 Pull request is already reviewed well in another branch. label Jul 29, 2016
gavofyork and others added 4 commits July 29, 2016 14:35
* Trace through DELEGATECALL and CALLCODE

Add them to the JSON output and RLP database store.

* Fix tests.

* Fix all tests.

* Fix one more test.
* tracing finds transaction which creates contract

* comma cleanup

Remove when following `}`s, add to final entries.
* Various improvements to tracing & diagnostics.

- Manage possibility of `Account` not having code for `PodAccount`
- New RPC: `trace_sendRawTransaction`
- See raw transaction dump when inspecting over RPC

* Fix test

* Remove one of the dupe error messages

* Remove unneeded `&`s

* Reformat and extremely minor optimisation

* Minor optimisation

* Remove unneeded let

* Fix tests.

* Additional fix.

* Minor rename.

[ci:skip]

* Bowing to the pressure.
* executive tracer builds flat traces without intermediate struct

* temporarilt commented out tests for traces

* fixed new way of building trace address

* fixed new way of building trace address

* updating state tests with flat tracing in progress

* fixed flat tracing tests

* fixed compiling ethcore-rpc with new flat traces

* removed warnings from ethcore module

* remove unused data structures
@gavofyork gavofyork merged commit 03fad08 into beta Jul 30, 2016
@gavofyork gavofyork deleted the debris-beta branch July 30, 2016 13:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-backport 🕸 Pull request is already reviewed well in another branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants