[FFmpeg-devel] [PATCH] ffprobe integration

[Stefano Sabatini]

On date Saturday 2010-01-02 20:58:30 +0100, Diego Biurrun encoded:
> On Sat, Jan 02, 2010 at 06:46:11PM +0100, Stefano Sabatini wrote:
[...]
> Sometimes you use K&R function declarations, sometimes you don't...

K&R and other cosmetics issues fixed.

> > --- /dev/null	1970-01-01 00:00:00.000000000 +0000
> > +++ ffmpeg/doc/ffprobe-doc.texi	2010-01-02 18:44:56.000000000 +0100
> > @@ -0,0 +1,75 @@
> > +
> > +FFprobe is a very simple and portable prober using the FFmpeg
> > +libraries to probe the content of a file.
> 
> I would suggest s/prober/program/.
> 
> > +It may be used to print informations about the format and header of a
> 
> The word "information" has no plural.
> 
> > +multimedia file or stream, the multimedia streams and each single
> > +packet or frame.
> 
> a multimedia stream, the multimedia streams - you talk about these two
> things as if they were not the same..
> 
> > +FFprobe output consists in stanza of the form:
> 
> a stanza

Slightly changed, now is:
FFprobe output consists of stanzas of the form: ...

> > +and is designed in order to be easily parsable by a textual filter.
> 
> designed to be
> 
> > +FFprobe may be used both as a standalone application or in combination with
> > +a textual filter, which is supposed to perform more sophisticated
> > +elaboration (e.g. statistics elaboration or plotting).
> 
> elaboration?

All issues fixed.

I did other simplifications (the AVInputFile and AVInputStream structs
are currently not required, so removed), and renamed -show_file ->
-show_format which seems more meaningful.

Regards.
-- 
FFmpeg = Frenzy and Fierce Mystic Purposeless Enlightening Genius
-------------- next part --------------
A non-text attachment was scrubbed...
Name: add-ffprobe.patch
Type: text/x-diff
Size: 16693 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100102/338a02c9/attachment.patch>