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.

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

EIP 150 finished #143

Merged
merged 24 commits into from
Jul 16, 2017
Merged

EIP 150 finished #143

merged 24 commits into from
Jul 16, 2017

Conversation

cdetrio
Copy link
Contributor

@cdetrio cdetrio commented Jul 10, 2017

rebased #129 (see also #95)

axic and others added 23 commits July 10, 2017 19:05
@axic @cdetrio
Remove unused _homestead flag
4970be5
@axic @cdetrio
Add homesteadReprice option
2f589c9
@axic @cdetrio
Implement EIP150.1c (suicide new account cost)
254522e
@axic @cdetrio
Implement EIP150.1b (all but 64th rule)
3eff9d3
@sdtsui @cdetrio
Increase gas baseCosts for EIP150
5eaab3d
@sdtsui @cdetrio
Improve comments in runTx
06356a9
@sdtsui @cdetrio
Change gas baseCost to finalized EIP150 spec, 700 instead of 4000
2f74596
@sdtsui @cdetrio
Add gasLimit to CREATE and CALL opFns
cf667f4
@sdtsui @cdetrio
Resolve conflicts, bump FORK_CONFIG, bump ethereumjs-testing version
1b9980c
@sdtsui @cdetrio
Separate OOG from checkCallMemCost, so EIP150 limit can be applied
0245c77
@sdtsui @cdetrio
Cleanup repo, remove support for flags in calcCallLimit
44d10c8
@sdtsui @cdetrio
Fix application of stipend in CALL
1d27a58
@sdtsui @cdetrio
Fix from @inclu-media for checkOutOfGas - 335 failing tests
8d56f86
@sdtsui @cdetrio
Fix callcall_00_OOGE test with @cdetrio
5432e8e
@sdtsui @cdetrio
Cleanup, remove commented code
78357eb
@sdtsui @cdetrio
Add @inclu-media\'s use of BN in checkOutofGas
001033c
@cdetrio
Fixed last commit (code parts left)
7d841dc
@hugo-dc @cdetrio
fix some initializations, validate results return length
b5da2a1
@cdetrio
Removed double-run of checkCallMemCost/checkOutOfGas in CREATE opcode…
86e8895 
…, moved stipend addition after check calls
@inclu-media @cdetrio
fixed refund handling in SUICIDE
67c9749
@inclu-media @cdetrio
subGas in CALLCODE, unnecessary stateManager call removed
095b8ac
@sdtsui @cdetrio
Fix lint errors in opFns, runCode, and test hooks
fb2172e
@cdetrio
Removed unintentionally added package-lock.json file, added package-l…
b0b1b07 
…ock.json to .gitignore
@sdtsui sdtsui mentioned this pull request Jul 11, 2017
Closed
@holgerd77
Copy link
Member

One thing: shouldn't this enableHomestead option be removed now? Otherwise things get inconsistent if (not) used!

@cdetrio
Copy link
Contributor Author

cdetrio commented Jul 11, 2017

@holgerd77 yes good catch, enableHomestead should be removed everywhere.

@wanderer wanderer merged commit 1971754 into ethereumjs:master Jul 16, 2017
@cdetrio cdetrio mentioned this pull request Jul 25, 2017
Closed
holgerd77 added a commit that referenced this pull request Mar 11, 2021
@holgerd77
Merge pull request #143 from ihordiachenko/eip155_support
5c3b651 
EIP-155 support
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
package: vm
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@cdetrio @holgerd77 @evertonfraga @wanderer @axic @sdtsui @hugo-dc @inclu-media