Opened 22 years ago

Closed 22 years ago

Last modified 6 years ago

#8098 closed patch

Looping CD/MP3 audio

Reported by: eriktorbjorn Owned by: SF/ender
Priority: normal Component: Audio
Version: Keywords:
Cc: Game:

Description

This should fix bug #549544 ("CD audio/mp3 music does not loop").

For the CD case the code to handle looping was already there - at least in the SDL backend - but slightly broken. I've changed it so that when told to play one whole CD track, it plays only that track, not every track starting from that one.

I haven't checked the other backends.

The MP3 version didn't even try to loop. I've added some code to fix that.

There are still some glitches that I know of that are *not* fixed by this patch:

* When playing from CD, the old track doesn't stop when you load a savegame.

* When playing from CD, pausing the game does not pause the music.

* When playing from MP3, there's a brief noise at the end of the track. It's as if the mixer is trying to loop the track on its own, but only for a fraction of a second. I'm not sure why. I guess the easiest workaround would be to lie and tell ScummVM that the tracks are just a little bit shorter than they really are.

Ticket imported from: #625904. Ticket imported from: patches/203.

Attachments (1)

cd-loop.diff (2.9 KB ) - added by eriktorbjorn 22 years ago.
Patch against an October 19 CVS snapshot

Download all attachments as: .zip

Change History (5)

by eriktorbjorn, 22 years ago

Attachment: cd-loop.diff added

Patch against an October 19 CVS snapshot

comment:1 by SF/ender, 22 years ago

Appli-or-ed!

comment:2 by SF/ender, 22 years ago

Owner: set to SF/ender
Status: newclosed

comment:3 by fingolfin, 22 years ago

Can somebody please update the bug (https://sourceforge.net/tracker/ index.php?func=detail&aid=549544&group_id=37116&atid=418820) with the current status, or even close it (depending on what is more approriate).

comment:4 by digitall, 6 years ago

Component: Audio
Note: See TracTickets for help on using tickets.