This bug report was migrated from our old Bugzilla tracker.

These attachments are available in the static archive:

• Patch - music_flac.c - Fix Initial Stutter (music_flac_fix_initial_stutter.patch, text/plain, 2019-11-18 03:28:23 +0000, 575 bytes)

Reported in version: 2.0.4
Reported for operating system, platform: Linux, x86_64

Comments on the original bug report:

On 2019-06-21 10:06:47 +0000, Braden Obrzut wrote:

When playing a FLAC file the first frame will be repeated. This is caused by FLAC__stream_decoder_seek_absolute automatically processing a frame, which in turn causes the callback to be run which puts data on the audio stream. Since FLAC_Play calls FLAC_Seek the frame is decoded there, and then music_internal_play decides to also seek to 0 with music_internal_position causing the frame to be decoded a second time.

Given that all of the Play functions call Seek I'm not sure why music_internal_play needs to do a seek to 0. But removing that won't be perfect since if a position besides 0 is given the FLAC player will still decode the first frame before starting at the seeked position. Probably the correct solution is to clear the audio stream in FLAC_Seek.

On 2019-11-18 03:28:23 +0000, Michael Day wrote:

Created attachment 4059
Patch - music_flac.c - Fix Initial Stutter

I think the fix is as simple as clearing the audio stream before seeking. See the attached patch.

On 2019-11-18 04:51:10 +0000, Ozkan Sezer wrote:

Applied this patch: https://hg.libsdl.org/SDL_mixer/rev/7b663db4eef1

Braden Obrzut: Please make sure that this fixes your issue.

On 2019-12-03 11:59:37 +0000, Ozkan Sezer wrote:

No response from original bug reporter since two weeks. Closing
as fixed. Drop a note here if problem persists.