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 matching SFX echo channels to music echo channels by 1DFA command #181

Open
wants to merge 20 commits into
base: master
Choose a base branch
from

Conversation

KungFuFurby
Copy link
Owner

A new $1DFA command ID, $0A, has been added on for the purpose. Forcing all SFX
channels to have their echo off or on causes this to be disabled.

This commit mentions #137.

A new $1DFA command ID, $0A, has been added on for the purpose. Forcing all SFX
channels to have their echo off or on causes this to be disabled.

This commit mentions #137.
@KungFuFurby KungFuFurby added the spc-side Involves the SPC700 code. label Jun 20, 2021
Comes complete with a !noSFX section added on because of the merger of !noSFX
into master combined with being removed from an enclosing !noSFX section.
Accidentally forgot to do so during merge conflict resolution
This section need not apply when SFX is not present.
@KungFuFurby KungFuFurby mentioned this pull request Jun 25, 2021
9 tasks
Avoid accidental slowdown on jump and girder SFX caused by not doing this
…outine

First of all, we don't want to accidentally execute ProcessAPU1SFX twice.
Secondly, it's less memory efficient.
Shouldn't have gotten rid of that second endif...
This is being done because the branch goes directly to a RET in this case.
Accidentally removed the case to turn the carry off during a merge.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spc-side Involves the SPC700 code.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant