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

Don Moir donmoir at comcast.net
Sat Apr 20 03:25:28 CEST 2013

>On Thu, Apr 18, 2013 at 11:50:01PM +0000, Paul B Mahol wrote:
>> 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.

>As you wish, disabled by default. Can be changed later easily if

disabled by default is good since app has a convienent way of breaking or continuing but if total_iter is not available to app then 
it does not know what the intended loop value is.

Also you might want to check that there are at least 2 frames if ignore_loop is false. While total_iter should not be set if there 
is only one frame it could be.

Again, you don't need ignore loop. app just needs total_iter so it has a way of knowing and can do whatever. 

More information about the ffmpeg-devel mailing list