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.

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

Data opcodes zero padding #741

Merged
merged 1 commit into from
Dec 5, 2023
Merged

Data opcodes zero padding #741

merged 1 commit into from
Dec 5, 2023

Conversation

gumb0
Copy link
Member

@gumb0 gumb0 commented Nov 30, 2023

No description provided.

@gumb0 gumb0 added the EOF label Nov 30, 2023
Copy link

codecov bot commented Nov 30, 2023

Codecov Report

Merging #741 (7048874) into master (ecc83d9) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #741   +/-   ##
=======================================
  Coverage   97.84%   97.84%           
=======================================
  Files         109      109           
  Lines       10303    10318   +15     
=======================================
+ Hits        10081    10096   +15     
  Misses        222      222           
Flag Coverage Δ
blockchaintests 60.47% <0.00%> (-0.32%) ⬇️
statetests 61.79% <0.00%> (-0.14%) ⬇️
statetests-silkpre 26.23% <0.00%> (-0.04%) ⬇️
unittests 95.91% <100.00%> (+<0.01%) ⬆️

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

Files Coverage Δ
lib/evmone/instructions.hpp 100.00% <100.00%> (ø)
test/unittests/evm_eof_test.cpp 100.00% <100.00%> (ø)

@gumb0 gumb0 requested review from chfast, pdobacz and rodiazet November 30, 2023 14:35
@gumb0 gumb0 merged commit 754fc9e into master Dec 5, 2023
25 checks passed
@gumb0 gumb0 deleted the data-zero-padding branch December 5, 2023 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants