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

De[bug]ger: Don't show address alongside symbol names #358

Open
warmCabin opened this issue Jun 3, 2021 · 9 comments
Open

De[bug]ger: Don't show address alongside symbol names #358

warmCabin opened this issue Jun 3, 2021 · 9 comments

Comments

@warmCabin
Copy link

warmCabin commented Jun 3, 2021

Looking at this snippet from Rockman 2's NMI routine, it seems a little odd that the PPU register names get substituted in but my symbol names sit next to their addresses. That's why I think this might a bug.

Shoutouts to that lazy guy.

@ClusterM
Copy link
Collaborator

Yes, seems like a bug.

@bbbradsmith
Copy link
Contributor

Default register names were a relatively recent addition, I think by Feos?

@vadosnaprimer
Copy link

Not really recent. 20d0e3d

@ClusterM
Copy link
Collaborator

Seems like we really need an option to hide/show addresses for symbol debugging.

@bbbradsmith
Copy link
Contributor

Heh, wow that was a lot further back than I realized.

Yeah I'd agree it'd be a useful option. I'd keep it on, myself, because I like the addresses, but it's definitely come up now and again. (As an aside: maybe the debugger could use a menu bar or a settings window, instead of trying to fit all the options into the panel?)

It occurs to me that the reason I keep the default register names off is entirely that it hides the numerical address, which I hadn't realized was also an inconsistency with other debugger labels.

@vadosnaprimer
Copy link

Menu sounds like a great idea.

@warmCabin
Copy link
Author

I have a bit of a PR open where I'm addressing this and some other things I wanted to add. I removed the functionality completely because I am lazy and I really really hate it :/ But if other people like it, let's add a menu option!

Maybe we could make the address show up as a tooltip when you mouse over a label.

This interim build already has a menu bar along the top.

@bbbradsmith
Copy link
Contributor

For a menu bar, I'd suggest grouping it into things that can have one-word headings. I think there's 8 controls that are crowded in the bottom right that could really stand to be out of the way instead of taking up permanent space there. Something like:

Options

  • Auto open on ROM load
  • IDA Font
  • Reset Window Size (new)

Symbols

  • Reload
  • Load .DEB file

  • Symbolic Debug
  • Show Symbol Addresses (new)
  • Default Register Names

Colors

  • (as new Syntax highlight menu)

Tools

  • ROM Patcher

As for tooltips when mousing over a label... I think that's fine as long as it's not a substitute for the option to actually display the address in the debugging panel.

@warmCabin
Copy link
Author

@bbbradsmith Your menu ideas are great! I'm slowly making them come to life over in my branch.

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

No branches or pull requests

4 participants