Skip to content

Commit

Permalink
Correcting coordination flag names
Browse files Browse the repository at this point in the history
cfSetVolume, if used on a PSG track, will affect FM channels.
cfChangeVolume, if used on an FM track, will not have an effect until
voice change. Also, they both alter the volume; they do not set it.
  • Loading branch information
Clownacy committed Apr 19, 2015
1 parent f398e20 commit 4d56bb8
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions s2.sounddriver.asm
Expand Up @@ -2364,7 +2364,7 @@ coordflagLookup:
jp cfSetTempoDivider ; E5
nop
; ---------------------------------------------------------------------------
jp cfSetVolume ; E6
jp cfChangeFMVolume ; E6
nop
; ---------------------------------------------------------------------------
jp cfPreventAttack ; E7
Expand All @@ -2382,7 +2382,7 @@ coordflagLookup:
jp cfSetTempoMod ; EB
nop
; ---------------------------------------------------------------------------
jp cfChangeVolume ; EC
jp cfChangePSGVolume ; EC
nop
; ---------------------------------------------------------------------------
jp cfUnused1 ; ED
Expand Down Expand Up @@ -2569,8 +2569,8 @@ cfSetTempoDivider:
; ---------------------------------------------------------------------------

; (via Saxman's doc): Change channel volume BY xx; xx is signed
;zloc_DBB
cfSetVolume:
;zloc_DBB cfSetVolume
cfChangeFMVolume:
add a,(ix+zTrack.Volume) ; Add to current volume
ld (ix+zTrack.Volume),a ; Update volume
jp zSetChanVol ; Immediately set this new volume
Expand Down Expand Up @@ -2645,8 +2645,8 @@ zVolTLMaskTbl:
; volume change at voice may be necessary... or my guess anyway.

; Alternatively, just think of it as a volume setting optimized for PSG :P
;zloc_DF9
cfChangeVolume:
;zloc_DF9 cfChangeVolume
cfChangePSGVolume:
add a,(ix+zTrack.Volume) ; Add to channel volume
ld (ix+zTrack.Volume),a ; Store updated volume
ret
Expand Down

0 comments on commit 4d56bb8

Please sign in to comment.