[FFmpeg-devel] [PATCH] lavf/gifdec: add loop support.

[Don Moir]

----- Original Message ----- 
From: "Paul B Mahol" <onemda at gmail.com>
To: "FFmpeg development discussions and patches" <ffmpeg-devel at ffmpeg.org>
Sent: Thursday, April 18, 2013 7:50 PM
Subject: Re: [FFmpeg-devel] [PATCH] lavf/gifdec: add loop support.


> On 4/18/13, Clement Boesch <ubitux at gmail.com> wrote:
>> ---
>> Still missing FATE update (need to add -ignore_loop to them), but I'm
>> waiting for review on the patch to rework the tests.
>
> I would prefer that default is to not loop at all.

Not sure about this ignore loop value. If it is set to false by default then it could loop forever without an EOF. Ok for ffplay 
type app but not good when end of play means goto next media file or timeline type app.

In my own app I leave it up to user if he wants to loop or not as it applies to videos, audio, images etc.

I think it would be better for you to do nothing with ignore loop (no automatic looping) but make the iteration count available to 
an app and allow it to be controlled from the app rather than automatic.

Yes I see there is an option to turn ignore loop on.