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

[Ronald S. Bultje]

Hi,

On Sun, Feb 6, 2011 at 9:24 AM, Stefano Sabatini
<stefano.sabatini-lala at poste.it> wrote:
> 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.

They're at fosdem, I'll leave docs for Diego to review. Give 'em a few
days to come back. :-).

Ronald