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

Add some error handling in sceSas, delay sceSasCore() #2186

Merged
merged 4 commits into from
Jun 10, 2013

Conversation

unknownbrackets
Copy link
Collaborator

Per tests, this is what the functions seem to do. Fixes the huge delay in Kingdom Hearts.

Final Fantasy Tactics is still playing weird screwy sound effects, though...

-[Unknown]

Not sure if this fixes anything, but scheduling issues can be tricky.
This is no longer needed for games I have, and it breaks Final Fantasy
Tactics' sound effects.

Looking at JPCSP for insight, I saw that it re-enables loops if you set a
voice even with a different address (I think?) but that seems strange.
hrydgard added a commit that referenced this pull request Jun 10, 2013
Add some error handling in sceSas, delay sceSasCore()
@hrydgard hrydgard merged commit 3185594 into hrydgard:master Jun 10, 2013
@unknownbrackets unknownbrackets deleted the sas-minor2 branch June 10, 2013 06:57
@unknownbrackets
Copy link
Collaborator Author

Oh, I added a commit that reverted the previous (I think wrong) attempt to fix undead VAG loops, which broke nothing I had and fixed Final Fantasy Tactics.

-[Unknown]

@hrydgard
Copy link
Owner

Great!

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.

2 participants