[FFmpeg-devel] [PATCH] ffprobe integration

[Stefano Sabatini]

On date Sunday 2010-01-03 01:59:14 +0100, Diego Biurrun encoded:
> On Sun, Jan 03, 2010 at 12:21:20AM +0100, Stefano Sabatini wrote:
> > On date Saturday 2010-01-02 22:17:01 +0100, Diego Biurrun encoded:
> > > On Sat, Jan 02, 2010 at 10:02:00PM +0100, Stefano Sabatini wrote:
> > > > 
> > > > --- /dev/null	1970-01-01 00:00:00.000000000 +0000
> > > > +++ ffmpeg/doc/ffprobe-doc.texi	2010-01-02 22:00:22.000000000 +0100
> > > > @@ -0,0 +1,73 @@
> > > > +
> > > > +It may be used to print information about the format of a multimedia
> > > > +file or stream, of each single stream and of each single packet or
> > > > +frame contained in it.
> > > 
> > > This is still confusing.
> > > 
> > > What else can it be used for?  Clean dishes?  Just say that it prints
> > > information, not that it can be used to print information, which
> > > implies it has other uses.
> > > 
> > > I also don't understand what the second half of the sentence after the
> > > comma is trying to say.  What do single stream/packet refer to?
> > 
> > Multimedia stream -> contains many media streams
> > 
> > I tried to reformulate the program description, please check again.
> >  
> > --- /dev/null	1970-01-01 00:00:00.000000000 +0000
> > +++ ffmpeg/doc/ffprobe-doc.texi	2010-01-03 00:19:41.000000000 +0100
> > @@ -0,0 +1,74 @@
> > +
> > +FFprobe is a simple and portable program using the FFmpeg libraries to
> > +probe the content of a multimedia stream, and print information
> > +relative to its format and content.
> 
> Leave out "simple and portable".  I know it is used in other places, but
> there is no need for advertising.
> 
> What about
> 
>   FFprobe gathers information from multimedia streams and prints it in
>   human- and machine-readable fashion.

Fine to me.
 
> > +FFprobe may be used both as a standalone application or in combination
> > +with a textual filter, which may perform more sophisticated
> > +processing, e.g. statistical processing or plotting.
> > + at c man end
> 
> You employ the word "use" all the time, check out
> 
> http://thesaurus.reference.com/browse/use

Mmh..

> > + at item -show_format
> > +Show information relative the container format of the input multimedia
> > +stream.
> > + at item -show_streams
> > +Show information relative to each media stream contained in the input
> > +multimedia stream.
> 
> Information is not relative to, but about something; same in other
> places.

Fixed.

Patch incoming as a reply to another message.
-- 
FFmpeg = Furious Fierce Mere Proud Ecletic Goblin