-
Notifications
You must be signed in to change notification settings - Fork 285
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
Refactor system_call()
#976
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #976 +/- ##
=======================================
Coverage 94.29% 94.29%
=======================================
Files 143 144 +1
Lines 16149 16155 +6
=======================================
+ Hits 15228 15234 +6
Misses 921 921
Flags with carried forward coverage won't be shown. Click here to find out more.
|
25d220b
to
a94a3ec
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thx for the comment, I think it's useful, I only scanned 4788 and missed the empty account part
@@ -0,0 +1,74 @@ | |||
// evmone: Fast Ethereum Virtual Machine implementation | |||
// Copyright 2023 The evmone Authors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// Copyright 2023 The evmone Authors. | |
// Copyright 2024 The evmone Authors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2023 is the year then this code was introduced.
Refactor and abstract the `system_call()` to make adding new system contracts easier. Make address of system contracts public.
5fc2762
to
87a8918
Compare
Refactor and abstract the
system_call()
to make adding new system contracts easier.