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

mGBA Audio Keeps Playing Notes after Rewind #1195

Closed
NarryG opened this issue May 13, 2018 · 9 comments
Closed

mGBA Audio Keeps Playing Notes after Rewind #1195

NarryG opened this issue May 13, 2018 · 9 comments
Labels
Core: mGBA Game Boy Advance (GBA) core

Comments

@NarryG
Copy link
Contributor

NarryG commented May 13, 2018

With the latest developer build, some GBA audio channels appear to not be properly cutting off when using rewind.

Example with Mario & Luigi Superstar Saga
https://streamable.com/lv1f1

This issue does not exist in the latest 0.6.X build of mGBA standalone.

@NarryG
Copy link
Contributor Author

NarryG commented May 13, 2018

Tagging @nattthebear as I know he deals with the mGBA core.

@vadosnaprimer
Copy link
Contributor

Updated mgba. Still bugged?

@NarryG
Copy link
Contributor Author

NarryG commented Jun 22, 2018

This commit appears to have actually broken the core completely
image

@NarryG
Copy link
Contributor Author

NarryG commented Jun 23, 2018

The issue still persists @vadosnaprimer
I've compared against mGBA 0.6.3 stable and the issue appears to be upstream. I'll report it there. I must have made a mistake when I originally said it wasn't happening in standalone

@NarryG NarryG closed this as completed Jun 23, 2018
@nattthebear
Copy link
Contributor

Why did we revert the palette changes? What's going on here?

@vadosnaprimer
Copy link
Contributor

vadosnaprimer commented Jun 23, 2018

It was only reverted in my 1.x-only fork. tasvideos/mgba is fine, and it's what I used to build our current mgba.dll. I just accidentally used my 1.x build here initially.

@NarryG
Copy link
Contributor Author

NarryG commented Jun 24, 2018

The issue has been fixed upstream in the 0.7 branch. I requested endrift backport to 0.6.X
mgba-emu/mgba#1107

@NarryG NarryG reopened this Jun 24, 2018
@NarryG
Copy link
Contributor Author

NarryG commented Jun 24, 2018

It has been backported so it's now just a matter of updating the fork again.

@vadosnaprimer
Copy link
Contributor

@YoshiRulz YoshiRulz added the Core: mGBA Game Boy Advance (GBA) core label Feb 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Core: mGBA Game Boy Advance (GBA) core
Projects
None yet
Development

No branches or pull requests

4 participants