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

Add debugger GUI (dev branch) #1110

Draft
wants to merge 52 commits into
base: dev
Choose a base branch
from

Conversation

lhsazevedo
Copy link
Contributor

This PR builds upon #919 from @vkedwardli (quoted below) with the following changes requested by @flyinghead:

  • Target dev branch
  • Add option to enable the debugger in "Debug" tab of the settings screen:
    image

In the future, we could move some debug options from "Advanced" tab as well.


Original PR message:

Original work by @lhsazevedo in https://github.com/lhsazevedo/flycast/tree/dbgnet, PR with his consent

I have simplified some stuff, fixed some bugs, and remove Tokyo Bus related code to come up with this PR.

New debugger button (didn't mean to use full width button, but it's odd number...)
image

Default view:
image

Breakpoint can be added by double clicking the empty space also image

Co-authored-by: Lucas Azevedo <lhs_azevedo@hotmail.com>
@lhsazevedo lhsazevedo changed the base branch from master to dev June 13, 2023 00:23
@lhsazevedo
Copy link
Contributor Author

There is a lot of room for improvement in the UI, which I plan to do next as soon as I have some free time. But some people are wiling to use it as is because there aren't good alternatives available.

@lhsazevedo lhsazevedo marked this pull request as draft June 13, 2023 23:08
@bbayles
Copy link

bbayles commented Jun 1, 2024

This is really impressive work - just fantastic. Can't wait to start diving in with these tools.

If you're taking feature requests: the ability to save the current memory dump to a file with a click would be a huge benefit for hacking. Mednafen has this and it enables really rapid testing.

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.

5 participants