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

EIP161 - Working Branch #144

Closed
wants to merge 29 commits into from
Closed

EIP161 - Working Branch #144

wants to merge 29 commits into from

Conversation

sdtsui
Copy link
Contributor

@sdtsui sdtsui commented Jul 12, 2017

  • This PR enables multiple ethereumjs contributors.
  • If you'd like to help, check out ethereumjs/ethereumjs-lib on Gitter.
  • Please be careful when pushing changes to this branch, especially if it reverts the work of others. If you aren't sure what to do next, ask in Gitter!
  • If a commit you're making causes a change in the tests, include a comment here with the number of tests remaining, and the name of the next failing test. This will help the next contributor pick up where you left off.

Thanks!

axic and others added 29 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
@jwasinger
Implement 161b for CALL
2fc7a00
@jwasinger
add check for empty account to CALL
0078066
@jwasinger
add EIP161b logic for SUICIDE
2ecb52a
@jwasinger
EIP161a courtesy of holgerd77
4805a1a
@jwasinger
add trie state pruning
8de884f
@jwasinger
fix lint errors
cc97f2d
@cdetrio
Copy link
Contributor

cdetrio commented Jul 27, 2017

replaced by #147

@cdetrio cdetrio closed this Jul 27, 2017
holgerd77 added a commit that referenced this pull request Dec 1, 2020
@holgerd77
Merge pull request #144 from ethereumjs/typescript-transition
53a13db 
TypeScript Transition Start + Base Functionality
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.
7 participants
@sdtsui @cdetrio @evertonfraga @axic @hugo-dc @inclu-media @jwasinger