search

capstone-rust / capstone-rs

high-level Capstone system bindings for Rust
217 stars 75 forks source link

capstone: add cs_regs_access #77

Open SWW13 opened 4 years ago

SWW13 commented 4 years ago

Adds support for cs_regs_access, fixes #63.

I'm not sure how to implement the tests yet, the instructions_* tests are a bit complicated on first sight.

codecov[bot] commented 4 years ago

Codecov Report

Merging #77 into master will increase coverage by 0.35%. The diff coverage is 71.75%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #77      +/-   ##
==========================================
+ Coverage   93.81%   94.17%   +0.35%     
==========================================
  Files          20       20              
  Lines        2701     3983    +1282     
==========================================
+ Hits         2534     3751    +1217     
- Misses        167      232      +65
Impacted Files Coverage Δ
capstone-rs/src/test.rs 97.27% <100%> (+1.26%) :arrow_up:
capstone-rs/src/capstone.rs 96.74% <100%> (+2.78%) :arrow_up:
capstone-rs/src/instruction.rs 76.22% <48.61%> (-11.85%) :arrow_down:
capstone-rs/src/error.rs 63.41% <0%> (-1.59%) :arrow_down:
capstone-rs/src/constants.rs 79.06% <0%> (-0.45%) :arrow_down:
capstone-rs/src/arch/tms320c64x.rs 97.67% <0%> (+0.07%) :arrow_up:
capstone-rs/src/arch/mod.rs 90.9% <0%> (+0.08%) :arrow_up:
capstone-rs/src/arch/m68k.rs 91.94% <0%> (+0.15%) :arrow_up:
... and 1 more

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 efdddcb...0475878. Read the comment docs.

tmfink commented 4 years ago

@SWW13 Thanks for the pull-request! This is great work.

If you have any questions related to review comments or capstone, feel free to reach out.