fceux
fceux copied to clipboard
TASEmulators
De[bug]ger: Don't show address alongside symbol names
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.
Default register names were a relatively recent addition, I think by Feos?
Not really recent. https://github.com/TASVideos/fceux/commit/20d0e3d6c21717990558cfd665d149cc02ca4ef6
Seems like we really need an option to hide/show addresses for symbol debugging.
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.
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.
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.