Skip to content

fix: Clear journal #1927

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
rakita merged 12 commits into from
Dec 18, 2024
Merged

fix: Clear journal #1927

rakita merged 12 commits into from
Dec 18, 2024

Conversation

@rakita
Copy link
Member

@rakita rakita commented Dec 18, 2024

Fix journal cleanup and rm depth from Gas Inspector

@github-actions
Copy link
Contributor

github-actions bot commented Dec 18, 2024

Valgrind Results:

==4673== Cachegrind, a high-precision tracing profiler
==4673== Copyright (C) 2002-2017, and GNU GPL'd, by Nicholas Nethercote et al.
==4673== Using Valgrind-3.22.0 and LibVEX; rerun with -h for copyright info
==4673== Command: target/release/revme bench snailtracer
==4673== 
Running snailtracer example!
elapsed: 1.132633171s
==4673== 
==4673== I refs:        462,199,146

@rakita rakita merged commit 1f6d438 into main Dec 18, 2024
27 checks passed
@github-actions github-actions bot mentioned this pull request Dec 18, 2024
Closed
@rakita rakita deleted the rakita/fix_journal_clear branch December 18, 2024 15:38
royvardhan pushed a commit to royvardhan/revm that referenced this pull request Dec 20, 2024
@rakita @royvardhan
fix: Clear journal (bluealloy#1927)
cace866 
* chore: Simplify GasInspector

* fmt

* chore: add depth to GasInspector

* rm doc

* chore: tie journal database with database getter

* use exec

* fmt

* include box

* fix: Clear Journal
This was referenced Dec 20, 2024
Open
Open
Open
Open
Closed
Open
Closed
This was referenced Dec 31, 2024
Open
Open
Open
Open
This was referenced Jan 7, 2025
Open
Closed
This was referenced Feb 7, 2025
Closed
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rakita