Re: [FFmpeg-devel] [PATCHv2] doc/demuxers: document gif demuxer

2014-09-02 Thread Timothy Gu
On Aug 31, 2014 3:16 PM, "Lou Logan"  wrote:

> +@item ignore_loop

> +GIF files can be set to loop infinitely or up to a certain number of
> +times.

can contain information to loop...

"Can be set" sounds like settable from a ffmpeg command.

> +
> +If @option{ignore_loop} is set to 1, then the loop setting from the
> +input will be ignored and looping will not occur.
> +
> +If set to 0, then looping will occur and will cycle the number of times
> +the GIF dictates.

Merge the above 3 paragraphs if you think it is appropriate.

> +
> +This option prevents a GIF stream from looping forever.

Not necessary. Covered in the second paragraph above.

> +
> +Default value is 1.

5 paragraphs for this trivial option? Ouch.

> +@end table
> +
> +For example, with the overlay filter, place an infinitely looping GIF
> +over another video:
> +@example

> +ffmpeg -i input.mp4 -ignore_loop 0 -i input.gif
>  -filter_complex overlay=shortest=1 out.mkv

What if input.gif contains information saying that it should only be looped
once? Doesn't this command break then?

Rest LGTM.

Timothy
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] [PATCHv2] doc/demuxers: document gif demuxer

2014-08-31 Thread Lou Logan
From ecad89e3edb64da662ae06a9d37c1fa84b02fadb Mon Sep 17 00:00:00 2001
From: Lou Logan 
Date: Thu, 28 Aug 2014 16:26:11 -0800
Subject: [PATCH] doc/demuxers: document gif demuxer

Signed-off-by: Lou Logan 
---
New patch addressing comments from Timothy and Clément.

 doc/demuxers.texi | 40 
 1 file changed, 40 insertions(+)

diff --git a/doc/demuxers.texi b/doc/demuxers.texi
index d51b9d0..2c9d142 100644
--- a/doc/demuxers.texi
+++ b/doc/demuxers.texi
@@ -174,6 +174,46 @@ See @url{http://quvi.sourceforge.net/} for more 
information.
 FFmpeg needs to be built with @code{--enable-libquvi} for this demuxer to be
 enabled.
 
+@section gif
+
+Animated GIF demuxer.
+
+It accepts the following options:
+
+@table @option
+@item min_delay
+Set the minimum valid delay between frames in hundredths of seconds.
+Range is 0 to 6000. Default value is 2.
+
+@item default_delay
+Set the default delay between frames in hundredths of seconds.
+Range is 0 to 6000. Default value is 10.
+
+@item ignore_loop
+GIF files can be set to loop infinitely or up to a certain number of
+times.
+
+If @option{ignore_loop} is set to 1, then the loop setting from the
+input will be ignored and looping will not occur.
+
+If set to 0, then looping will occur and will cycle the number of times
+the GIF dictates.
+
+This option prevents a GIF stream from looping forever.
+
+Default value is 1.
+@end table
+
+For example, with the overlay filter, place an infinitely looping GIF
+over another video:
+@example
+ffmpeg -i input.mp4 -ignore_loop 0 -i input.gif -filter_complex 
overlay=shortest=1 out.mkv
+@end example
+
+Note that in the above example the shortest option for overlay filter is
+used to end the output video at the length of the shortest input file,
+which in this case is @file{input.mp4} as the GIF loops infinitely.
+
 @section image2
 
 Image file demuxer.
-- 
2.0.4
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel