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

Generate save state IDs for homebrew #7997

Merged
merged 1 commit into from
Sep 25, 2015
Merged

Generate save state IDs for homebrew #7997

merged 1 commit into from
Sep 25, 2015

Conversation

hrydgard
Copy link
Owner

Now they won't all save to the same state.

Fixes #7795 .

If you load a loose elf file, the savegame will be the name of the elf. So cube.elf will have save ID "cube".

If you load a PBP from a PSP/GAME folder like cave story, it'll take the last part of the path. So PSP/GAME/cavestory will have save ID "cavestory".

I think this is good enough, although of course two users could name their game folders differently, but that's their problem then :)

@hrydgard hrydgard mentioned this pull request Sep 23, 2015
@vnctdj
Copy link
Contributor

vnctdj commented Sep 23, 2015

I don't understand...
It appears that pull request doesn't make any change ! Weird...
I have tested with 3 different homebrews (downloaded from our Homebrew Store) and all of their states were saved as "_#".
Here is a screenshot :

0

@hrydgard
Copy link
Owner Author

Really? Weird. Worked for me, let me test some more...

Yeah, there's definitely a problem. I'll fix it.

@hrydgard
Copy link
Owner Author

@vnctdj thanks for testing, fixed it now I think :)

@vnctdj
Copy link
Contributor

vnctdj commented Sep 23, 2015

I confirm the fix, it should be ok for merging now :)

@unknownbrackets
Copy link
Collaborator

Looks like the UI.vcxproj ended up in here.

-[Unknown]

@vnctdj
Copy link
Contributor

vnctdj commented Sep 24, 2015

@unknownbrackets
Oh, the _ARCH_* things ? Indeed ^^
Maybe @hrydgard was distracted :)

hrydgard added a commit that referenced this pull request Sep 25, 2015
Generate save state IDs for homebrew
@hrydgard hrydgard merged commit d2a2a11 into master Sep 25, 2015
@hrydgard hrydgard deleted the homebrew-savestate branch September 25, 2015 18:37
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.

3 participants