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

(iOS RetroArch) hard crash when exiting pictochat on DS #28

Closed
duckfromdiscord opened this issue Jul 11, 2023 · 7 comments
Closed

(iOS RetroArch) hard crash when exiting pictochat on DS #28

duckfromdiscord opened this issue Jul 11, 2023 · 7 comments
Labels
bug Something isn't working frontend This issue isn't directly solvable in melonDS DS, and is the responsibility of a libretro frontend. ios For issues related to iOS. (I don't own an iOS device, these may take longer to resolve.)

Comments

@duckfromdiscord
Copy link

the RetroArch app immediately crashes after following these steps:

  • open RetroArch
  • select core menu
  • pick melonDS DS
  • load core directly
  • open pictochat
  • pick a room
  • hit B to exit and A to confirm

retroarch2.log

@MrRedstone54
Copy link

Following up on this issue:
The same crash occurs when closing the core via the quickmenu option: Close Content
retroarch__2023_07_11__14_56_41.log

@JesseTG JesseTG added bug Something isn't working ios For issues related to iOS. (I don't own an iOS device, these may take longer to resolve.) labels Jul 11, 2023
@JesseTG JesseTG added the help wanted Extra attention is needed label Aug 8, 2023
@JesseTG
Copy link
Owner

JesseTG commented Aug 14, 2023

Pretty sure I fixed this, but I'd like to be certain. Could you try out the latest build and report back?

@duckfromdiscord
Copy link
Author

@JesseTG
Copy link
Owner

JesseTG commented Oct 12, 2023

This may actually be two separate bugs, neither of which is actually in melonDS DS.

The first was fixed by this PR yesterday.

The second relates to the "Load Dummy on Core Shutdown" setting, which by default is ON for most platforms but OFF for iOS and consoles. Calling RETRO_ENVIRONMENT_SHUTDOWN while this option is disabled will cause RetroArch itself to quit.

I can't detect this config setting at runtime, so this is what I'll do; I'll make the shutdown behavior configurable with one of two options:

  • Exit the core with RETRO_ENVIRONMENT_SHUTDOWN as happens now. Will be default on most platforms.
  • Show a shutdown screen with something like "it is now safe to exit the core", which will be the default on iOS and consoles.

@JesseTG JesseTG added the frontend This issue isn't directly solvable in melonDS DS, and is the responsibility of a libretro frontend. label Oct 12, 2023
@MrRedstone54
Copy link

To possibly follow up on you "second" bug:

It appears the same person who fixed the first bug also fixed the second one here

Something to look into?

@JesseTG
Copy link
Owner

JesseTG commented Oct 15, 2023

Yes, it turns out that no one there was sure of why iOS devices were treated specially with that config option anyway. So the default is being reset (but existing config values will stay in place).

@JesseTG JesseTG removed the help wanted Extra attention is needed label Oct 16, 2023
@JesseTG
Copy link
Owner

JesseTG commented Oct 16, 2023

This is now fixed!

@JesseTG JesseTG closed this as completed Oct 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working frontend This issue isn't directly solvable in melonDS DS, and is the responsibility of a libretro frontend. ios For issues related to iOS. (I don't own an iOS device, these may take longer to resolve.)
Projects
None yet
Development

No branches or pull requests

3 participants