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

EIP150.1c #2591

Merged
merged 34 commits into from
Oct 15, 2016
Merged

EIP150.1c #2591

merged 34 commits into from
Oct 15, 2016

Conversation

gavofyork
Copy link
Contributor

@gavofyork gavofyork added the A0-pleasereview 🤓 Pull request needs code review. label Oct 12, 2016
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.02%) to 86.419% when pulling 7569654 on eip150.1c into eb40750 on master.

@tomusdrw tomusdrw added the M4-core ⛓ Core client code / Rust. label Oct 12, 2016
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.09%) to 86.352% when pulling 2d9484f on eip150.1c into eb40750 on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.06%) to 86.383% when pulling 2cdaa31 on eip150.1c into eb40750 on master.

@pdaian pdaian mentioned this pull request Oct 13, 2016
Closed
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling b59a3cb on eip150.1c into * on master*.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling b59a3cb on eip150.1c into * on master*.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 7d2dc37 on eip150.1c into * on master*.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 7d2dc37 on eip150.1c into * on master*.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 28c6011 on eip150.1c into * on master*.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 0c6a7f6 on eip150.1c into * on master*.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 18a9d7a on eip150.1c into * on master*.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 18a9d7a on eip150.1c into * on master*.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 18a9d7a on eip150.1c into * on master*.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 3c86b91 on eip150.1c into * on master*.

@gavofyork gavofyork added A0-pleasereview 🤓 Pull request needs code review. and removed A0-pleasereview 🤓 Pull request needs code review. labels Oct 15, 2016
@arkpar arkpar added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Oct 15, 2016
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 79be0ad on eip150.1c into * on master*.

@gavofyork gavofyork merged commit 85eeb3e into master Oct 15, 2016
@gavofyork gavofyork deleted the eip150.1c branch October 15, 2016 12:39
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 79be0ad on eip150.1c into * on master*.

tomusdrw pushed a commit that referenced this pull request Oct 18, 2016
@gavofyork
EIP150.1c (#2591)
735c7c2 
* EIP150.1b first part.

* Add documentation.

[ci:skip]

* Tweak comment

* Fix some tests.

* Fix create

* Add the new EIP150 test. (#2563)

* Fix create some more.

* Another fix for create.

* Final fixes.

* Enable EIP150 test.

* Revert the costly suicides to respect EIP150.1b.

* Fixups for EIP150.1c

* Update tests.

* Updating tests

* Adding state tests

* Updating tests

* Updating HF block

* Removing flushln [ci:skip]

* Commenting out suicide test

* New tests.

* Switch to EIP tests.

* Rename some stuff, combine transition tests.

* Fix U256 overflows.

* Name cleanups and an effort to fix tests.

* Rename back for clarity.

* Fix non-EIP-150 OOG CALLs

* New transition block number & additional EIP150 tests

* Morden transition block set; Tests updated

* Added missing test

Conflicts:
	ethcore/res/ethereum/tests
	ethcore/src/json_tests/homestead_chain.rs
	ethcore/src/json_tests/state.rs
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@tomusdrw tomusdrw tomusdrw approved these changes

Assignees
No one assigned
Labels
A8-looksgood 🦄 Pull request is reviewed well. M4-core ⛓ Core client code / Rust.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@gavofyork @coveralls @tomusdrw @arkpar