Skip to content

Mod dis #512

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

Merged
merged 2 commits into from
Feb 21, 2019
Merged

Mod dis #512

merged 2 commits into from
Feb 21, 2019

Conversation

cthulahoops
Copy link
Collaborator

Initial implementation of dis module with just disassemble implemented. It's mostly the rust code for pretty printing CodeObject.

@codecov-io
Copy link

Codecov Report

Merging #512 into master will decrease coverage by 0.07%.
The diff coverage is 44.89%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #512      +/-   ##
==========================================
- Coverage   54.07%   53.99%   -0.08%     
==========================================
  Files          67       68       +1     
  Lines       12948    13042      +94     
  Branches     3146     3158      +12     
==========================================
+ Hits         7001     7042      +41     
- Misses       4138     4183      +45     
- Partials     1809     1817       +8
Impacted Files Coverage Δ
vm/src/stdlib/mod.rs 86.95% <100%> (-13.05%) ⬇️
vm/src/bytecode.rs 41.07% <38.37%> (-3.07%) ⬇️
vm/src/stdlib/dis.rs 90.9% <90.9%> (ø)
vm/src/builtins.rs 55.43% <0%> (+0.35%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dc3a44f...5ebfd55. Read the comment docs.

@windelbouwman windelbouwman merged commit 1d3db31 into master Feb 21, 2019
@windelbouwman windelbouwman deleted the mod_dis branch February 27, 2019 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants