[FFmpeg-devel] [PATCH] Add documentation for the image2 muxer.

[Stefano Sabatini]

On date Wednesday 2011-02-02 23:07:46 +0100, Diego Biurrun encoded:
> On Fri, Jan 28, 2011 at 10:48:23PM +0100, Stefano Sabatini wrote:
> > 
> > All points addressed.
> > 
> > --- a/doc/muxers.texi
> > +++ b/doc/muxers.texi
> > @@ -18,4 +18,55 @@ enabled muxers.
> >  
> > +Image file muxer.
> > +
> > +This muxer writes video frames to multiple image files specified by a
> > +pattern.
> > +
> > +The pattern may contain the string "%d" or "%0 at var{N}d", which
> > +specifies the position of the characters representing a numbering in
> > +the filenames.
> 
> It's not clear whether "which" refers to the first or second string.
> 
> I'd start with a simpler and more general description, i.e.
> 
>   The image file muxer writes video frames to image files.
> 
> and then go on to describe what can be done with patterns in the
> output filename.
> 
>   Patterns in the output file name can be used to produce sequentially
>   numbered series of files.
> 
> or
> 
>   To produce sequentially numbered series of files a printf-style
>   pattern can be embedded in the output filename.
> 
> > If the form "%d0 at var{N}d" is used, the string
> 
> stray 'd' before '0'?

The documentation was already applied, patch fixes it.
-- 
FFmpeg = Funny and Freak Minimal Philosofic Evangelical Guru