Skip to content

eth/tracers: clean-up tracer collection#24320

Merged
holiman merged 2 commits intoethereum:masterfrom
s1na:tracer/cleanup
Feb 1, 2022
Merged

eth/tracers: clean-up tracer collection#24320
holiman merged 2 commits intoethereum:masterfrom
s1na:tracer/cleanup

Conversation

@s1na
Copy link
Copy Markdown
Contributor

@s1na s1na commented Jan 31, 2022

Includes a few changes:

  • Make native prestate tracer the default and rename the JS one to prestateTracerLegacy
  • Make native noop tracer the default and rename the JS one to noopTracerLegacy
  • Drop the scope-based JS call tracer. Now we only have the native scope-based call tracer and the step-based JS legacy call tracer

Copy link
Copy Markdown
Contributor

@holiman holiman left a comment

Choose a reason for hiding this comment

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

LGTM

@holiman holiman added this to the 1.10.16 milestone Feb 1, 2022
@holiman holiman merged commit c5436c8 into ethereum:master Feb 1, 2022
sidhujag pushed a commit to syscoin/go-ethereum that referenced this pull request Feb 1, 2022
* eth/tracers: clean-up tracer collection

* Rm test for dropped tracer
JacekGlen pushed a commit to JacekGlen/go-ethereum that referenced this pull request May 26, 2022
* eth/tracers: clean-up tracer collection

* Rm test for dropped tracer
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Jul 28, 2025
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Jul 30, 2025
@FATBARB84
Copy link
Copy Markdown

Includes a few changes:

  • Make native prestate tracer the default and rename the JS one to prestateTracerLegacy
  • Make native noop tracer the default and rename the JS one to noopTracerLegacy
  • Drop the scope-based JS call tracer. Now we only have the native scope-based call tracer and the step-based JS legacy call tracer

@FATBARB84
Copy link
Copy Markdown

Uploading 1753470800838.jpg…

gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Aug 13, 2025
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Sep 8, 2025
gzliudan added a commit to XinFinOrg/XDPoSChain that referenced this pull request Sep 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants