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

Take latest changes from go-ethereum #4

Merged
merged 26 commits into from
Jan 4, 2018

Conversation

AlexeyAkhunov
Copy link
Owner

No description provided.

sorin and others added 26 commits December 28, 2017 14:15
Initial implementation of ENR according to ethereum/EIPs#778
* accounts/abi: handle named ouputs prefixed with underscores

* accounts/abi: handle collinding outputs for struct unpacks

* accounts: handle purely underscore output names
This avoids setting CC for the go run invocation, which fails on go1.10.
All inputs are saved into history, including 'invalid' inputs.
* eth: uncaptialize tracer preimage error message

* eth: improve very important error message
This reverts commits 85a1eda (#15792) and c495bca (#15787)
because they introduce database writes during tracing.
all: switch gas limits from big.Int to uint64
…15790)

* cmd/utils/flags.go: Applying a String len guard for the gitCommit param of the NewApp()

* cmd/utils: remove redundant clause in if condition
Also raise traceLimit once again and print the VM
error and output on failure.
@AlexeyAkhunov AlexeyAkhunov merged commit 50ad80e into AlexeyAkhunov:master Jan 4, 2018
AlexeyAkhunov pushed a commit that referenced this pull request Jan 22, 2019
* added tests for new abi encoding features (#4)

* added tests from bytes32[][] and string[]

* added offset to other types

* formatting

* Abi/dynamic types (#5)

* Revert "Abi/dynamic types (#5)" (#6)

This reverts commit dabca31.

* Abi/dynamic types (#7)

* some cleanup

* Apply suggestions from code review

apply suggestions

Co-Authored-By: vedhavyas <vedhavyas.singareddi@gmail.com>

* added better formatting (#8)

* review chnages

* better comments
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.