Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Baseline API #314

Merged
merged 2 commits into from
May 4, 2021
Merged

Baseline API #314

merged 2 commits into from
May 4, 2021

Conversation

chfast
Copy link
Member

@chfast chfast commented May 4, 2021

No description provided.

@codecov
Copy link

codecov bot commented May 4, 2021

Codecov Report

Merging #314 (77ec2ea) into master (1afff51) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #314   +/-   ##
=======================================
  Coverage   99.76%   99.76%           
=======================================
  Files          24       24           
  Lines        3829     3829           
=======================================
  Hits         3820     3820           
  Misses          9        9           
Flag Coverage Δ
consensus 99.17% <100.00%> (ø)
unittests 99.76% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
lib/evmone/baseline.cpp 99.82% <100.00%> (ø)
lib/evmone/evmone.cpp 100.00% <100.00%> (ø)

@chfast chfast force-pushed the execute_api branch 2 times, most recently from ce3a5ab to 1306ff1 Compare May 4, 2021 14:05
Base automatically changed from execute_api to master May 4, 2021 15:12
@chfast chfast merged commit 1fa2922 into master May 4, 2021
@chfast chfast deleted the baseline_api branch May 4, 2021 15:33
@@ -31,7 +31,7 @@ evmc_set_option_result set_option(evmc_vm* vm, char const* name, char const* val
{
if (value[0] == '0' && value[1] == '\0') // O=0
{
vm->execute = evmone::baseline_execute;
vm->execute = evmone::baseline::execute;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Will you also move the other one to the evmone::advanced namesapce?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was easy esthetic change, but I don't plan to do it more right now.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants