https://gitlab.synchro.net/main/sbbs/-/commit/312794de0530321312bcf568
Modified Files:
src/doors/syncconquer/door/soundio_termgfx.cpp
Log Message:
syncalert: fix the build -- pass the music `loop` argument termgfx requires
commit b042fa1da1 (ripe-12-dies) gave termgfx_audio_music_play() and termgfx_audio_music() a trailing `loop` argument, and updated the two
call sites it knew about -- src/doors/syncdoom/i_termmusic.c and src/doors/syncduke/syncduke_stubs.c. It missed syncconquer, whose door
had landed the day before in f5578ea57f (foam-43-clip), so this door's
three call sites in door/soundio_termgfx.cpp have been one argument
short ever since:
error C2660: 'termgfx_audio_music_play': function does not take 3 arguments
error C2660: 'termgfx_audio_music': function does not take 8 arguments
That is a hard compile error on every platform, not just Windows -- the syncalert door has not built anywhere since b042fa1da1.
Pass loop=1, as both siblings do. It is also what this door already
assumes: SoundImp_Sample_Status() runs its own wall-clock duration model precisely so the terminal can loop the score forever while Red Alert's ThemeClass::Still_Playing() is told the song "ended" and advances to the
next track (see the "(looping) path" comments on as_music).
Co-Authored-By: Claude Opus 4.8 (1M context) <
noreply@anthropic.com>
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net