Skip to content
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.

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

Jsonlogs #412

Closed
wants to merge 16 commits into from
Closed

Jsonlogs #412

wants to merge 16 commits into from

Conversation

zelig
Copy link
Contributor

@zelig zelig commented Mar 3, 2015

directly based on #409
ethereum/system-testing#54

@sveneh
Copy link

sveneh commented Mar 5, 2015

Any ETA of this merge?

@obscuren
Copy link
Contributor

obscuren commented Mar 5, 2015

This needs proper rebasing

@tgerring
Copy link
Contributor

tgerring commented Mar 5, 2015

obsoleted by #431

@tgerring tgerring closed this Mar 5, 2015
@zelig zelig deleted the jsonlogs branch March 8, 2015 10:06
@zelig zelig restored the jsonlogs branch March 8, 2015 10:06
@zelig zelig deleted the jsonlogs branch March 8, 2015 10:08
nolash pushed a commit to nolash/go-ethereum that referenced this pull request Apr 27, 2018
swarm: creates a 200 server response when localhost is pinged by JSON
ngtuna added a commit to ngtuna/tomochain that referenced this pull request Jan 29, 2019
Add rollback flag command for rollback chain at block hash.
maoueh pushed a commit to streamingfast/go-ethereum that referenced this pull request Nov 12, 2021
…ethereum#412)

* add extension for eth protocol handshake

* fix comments
tony-ricciardi pushed a commit to tony-ricciardi/go-ethereum that referenced this pull request Jan 20, 2022
maoueh pushed a commit to streamingfast/go-ethereum that referenced this pull request Dec 9, 2022
tanishqjasoria pushed a commit to tanishqjasoria/go-ethereum that referenced this pull request Apr 11, 2024
… value-bearing rule (ethereum#412)

Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
garyschulte pushed a commit to garyschulte/go-ethereum that referenced this pull request Apr 17, 2024
* simplified gas accounting layer

* integrate some review feedback

* Apply suggestions from code review

Co-authored-by: Ignacio Hagopian <jsign.uy@gmail.com>

* more suggestions from code review

* don't charge creation gas + charge code chunks in create

* A couple more fixes

* make linter happy

* fix create init gas consumption issue

* fix: in gas funcs, use tx witness instead of global witness

* fix linter issue

* Apply suggestions from code review

Co-authored-by: Ignacio Hagopian <jsign.uy@gmail.com>

* fix: EXTCODECOPY gas consumption

* fix warm gas costs

* fix the order gas is charged in during contract creation epilogue

* fix selfdestruct

* fix ethereum#365 in eip rewrite (ethereum#407)

* fix: OOG type in code creation OOG (ethereum#408)

* core/vm: charge BLOCKHASH witness cost (ethereum#409)

* core/vm: charge BLOCKHASH witness cost

Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>

* remove gas optimization for now

Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>

---------

Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>

* remove redundant logic for contract creation (ethereum#413)

Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>

* fix precompile address check for charging witness costs & fix missing value-bearing rule (ethereum#412)

Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>

* core/vm: fix wrong check (ethereum#416)

Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>

* charge for account creation if selfdestruct creates a new account (ethereum#417)

* add key comparison test (ethereum#418)

* core/vm: charge contract init before execution logic (ethereum#419)

* core/vm: charge contract init before execution logic

Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>

* fix CREATE2 as well

---------

Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
Co-authored-by: Guillaume Ballet <3272758+gballet@users.noreply.github.com>

* quell linter

---------

Signed-off-by: Ignacio Hagopian <jsign.uy@gmail.com>
Co-authored-by: Ignacio Hagopian <jsign.uy@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants