[FFmpeg-devel] [PATCH] Document null muxer.

[Stefano Sabatini]

On date Tuesday 2011-02-01 11:39:38 +0100, Stefano Sabatini encoded:
> On date Tuesday 2011-02-01 08:13:43 +0100, Diego Biurrun encoded:
> > On Tue, Feb 01, 2011 at 01:09:43AM +0100, Stefano Sabatini wrote:
> > > ---
> > >  doc/muxers.texi |   13 +++++++++++++
> > >  1 files changed, 13 insertions(+), 0 deletions(-)
> > > 
> > > diff --git a/doc/muxers.texi b/doc/muxers.texi
> > > index d98abef..7076959 100644
> > > --- a/doc/muxers.texi
> > > +++ b/doc/muxers.texi
> > > @@ -109,4 +109,17 @@ ffmpeg -i file.mpg -acodec copy -vcodec copy \
> > >       -y out.ts
> > >  @end example
> > >  
> > > + at section null
> > > +
> > > +Rawvideo null video format.
> > 
> > raw video null video?
> > 
> > What does this mean exactly?  I don't think it's clear to a non-expert
> > reader.
> 
> Indeed it was misleading, changed to "Null muxer".
> 
> > > +Rawvideo null video format.
> > > +
> > > +This muxer does not generate any output file, it is mainly useful for
> > > +testing or benchmarking purposes.
> > 
> > You always start your text with a non-sentence giving a catchy description
> > of the muxer and then continue with "this muxer...".  To my ear it would
> > sound better if the two were merged into one, i.e. "The rawvideo null
> > video format muxer does not...".
> 
> Done for consistency with the other files (in/outdevs,demuxers).
> Feel free to change it globally.
> 
> Added more explanation and another example.

Ping.
-- 
FFmpeg = Fierce & Fast Minimal Plastic Extended Glue