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

bug(DevTools): break on startup has no effect #2007

Open
axel7083 opened this issue Apr 1, 2024 · 4 comments
Open

bug(DevTools): break on startup has no effect #2007

axel7083 opened this issue Apr 1, 2024 · 4 comments

Comments

@axel7083
Copy link

axel7083 commented Apr 1, 2024

Bug

When selecting break on startup the game does not break at startup

no-break-startup.mp4

Reproduce

Tested with master 31a7f53

  • (1) git checkout
  • (2) follow build instruction standalone windows
  • (3) start emulator
  • (4) change settings for AMR9
  • (5) restart the emulator
  • (6) boot a game
  • (7) no break
@axel7083
Copy link
Author

axel7083 commented May 6, 2024

@PoroCYon do you have some ideas ?

@PoroCYon
Copy link
Contributor

PoroCYon commented May 6, 2024

sorry, don't really have time to look into it

could you try bisecting between 3ab752b and 31a7f53 to see which commit introduced the bug? there's been some refactoring going on so it could be something like that

@RSDuck
Copy link
Member

RSDuck commented May 7, 2024

from a glance it looks like GDB args are only passed in when the NDS object is created (

Config::GdbEnabled ? std::make_optional(gdbargs) : std::nullopt,
). They should be updated here as well
NDS->SPU.SetDegrade10Bit(static_cast<AudioBitDepth>(Config::AudioBitDepth));

Though then the setting should work under the condition that the setting was enabled during the construction of the NDS object (i.e. program startup or when switching between DS or DSi mode).

@PoroCYon
Copy link
Contributor

PoroCYon commented May 7, 2024

Though then the setting should work under the condition that the setting was enabled during the construction of the NDS object (i.e. program startup or when switching between DS or DSi mode).

Yeah, this was also a limitation from before the refactoring.

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

3 participants