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.

Sign up for GitHub

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

Jump to bottom

Add API to read & write registers for Linux #98

Merged
merged 4 commits into from
Nov 26, 2020
Merged

Add API to read & write registers for Linux #98

merged 4 commits into from
Nov 26, 2020

Conversation

nbaksalyar
Copy link
Member

@nbaksalyar nbaksalyar commented Aug 29, 2020 •
edited
Loading

Implements #87.

  • Add trait definition.
  • Implement register functions for the Linux target.
  • Refactor the repl example to use the new API.
  • Refactor the gui example to use the new API.
  • Clean up the code and fix CI builds.

@nbaksalyar
Add target::registers::Registers trait
e8f87ae 
This trait can be used to implement register reading & writing functions
for target architectures.
@nbaksalyar nbaksalyar changed the title [WIP] Add API to read & write registers Add API to read & write registers for Linux Nov 25, 2020
@nbaksalyar nbaksalyar marked this pull request as ready for review November 25, 2020 04:14
bjorn3
bjorn3 requested changes Nov 25, 2020
View reviewed changes
examples/repl.rs Outdated Show resolved Hide resolved
src/target/linux.rs Show resolved Hide resolved
@nbaksalyar
Copy link
Member Author

@bjorn3 should be fixed now, thanks!

@bjorn3 bjorn3 merged commit 8e9424a into headcrab-rs:master Nov 26, 2020
@nbaksalyar nbaksalyar deleted the registers branch November 26, 2020 06:38
@nbaksalyar nbaksalyar mentioned this pull request Nov 26, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bjorn3 bjorn3 bjorn3 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nbaksalyar @bjorn3