We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

[patch] A patch that adds loops count limit handling for looping OGGs

ogg-loops-count-limit-handling.patch (text/plain), 1.22 KB, created by Vitaly Novichkov on 2019-11-12 09:36:07 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Vitaly Novichkov

Created: 2019-11-12 09:36:07 UTC

Size: 1.22 KB

patch

obsolete

># HG changeset patch
># User v.novichkov
># Date 1573551059 -10800
>#      Tue Nov 12 12:30:59 2019 +0300
># Node ID 3613e937199bb5174beb9fd6354c2c7b1d7165b8
># Parent  f57650d505a35fe8169e716305db92b3edf3dc90
>OGG Vorbis: Handle loops count limit in ogg files with loops
>
>diff -r f57650d505a3 -r 3613e937199b music_ogg.c
>--- a/music_ogg.c	Mon Nov 11 11:41:02 2019 +0300
>+++ b/music_ogg.c	Tue Nov 12 12:30:59 2019 +0300
>@@ -419,12 +419,18 @@
>     }
> 
>     pcmPos = vorbis.ov_pcm_tell(&music->vf);
>-    if ((music->loop == 1) && (pcmPos >= music->loop_end)) {
>-        amount -= (int)((pcmPos - music->loop_end) * music->channels) * sizeof(Sint16);
>+    if ((music->loop == 1) && (music->play_count != 0) && (pcmPos >= music->loop_end)) {
>+        amount -= (int)((pcmPos - music->loop_end) * music->channels) * (int)sizeof(Sint16);
>         result = vorbis.ov_pcm_seek(&music->vf, music->loop_start);
>         if (result < 0) {
>             set_ov_error("ov_pcm_seek", result);
>             return -1;
>+        } else {
>+            int play_count = -1;
>+            if (music->play_count > 0) {
>+                play_count = (music->play_count - 1);
>+            }
>+            music->play_count = play_count;
>         }
>         looped = SDL_TRUE;
>     }

Actions: View | Diff

Attachments on bug 4857: 4042

We are currently migrating Bugzilla to GitHub issues.Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.