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

evm: expose getActiveOpcodes #2365

Closed
wants to merge 1 commit into from
Closed

Conversation

jochem-brouwer
Copy link
Member

This now allows to access getActiveOpcodes if EVMInterface is of type EVM. For instance, in VM:

const opcodes = vm.evm.getActiveOpcodes!()

Closes #2363

@codecov
Copy link

codecov bot commented Oct 18, 2022

Codecov Report

Merging #2365 (6f536ad) into master (f8c731c) will increase coverage by 3.81%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

Flag Coverage Δ
block ?
blockchain 90.21% <ø> (ø)
client ?
common 98.13% <ø> (ø)
devp2p 91.63% <ø> (ø)
ethash ?
evm ?
rlp ∅ <ø> (∅)
statemanager ?
trie 90.36% <ø> (-0.30%) ⬇️
tx ?
util 88.99% <ø> (ø)
vm ?

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

@holgerd77
Copy link
Member

Oh, sorry, have done exactly the same here ac16d71 2 minutes ago. 😆 Is it ok if we close here (otherway around is also fine to me)?

@jochem-brouwer
Copy link
Member Author

Yeah I just noticed haha. I will close :)

@holgerd77 holgerd77 deleted the update-evm-interface branch October 18, 2022 07:53
@holgerd77
Copy link
Member

Lol, yeah, funny sometimes! 🙂 Thanks for the flexibility!

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

Successfully merging this pull request may close these issues.

Expose getActiveOpcodes to the VM object
2 participants