[FFmpeg-devel] [PATCH] ffprobe integration

[Ramiro Polla]

[snip replies, just a quick review usage- and documentation-wise of
the latest patch]

It took me a while just to understand how this thing worked. I read
the manpage, but had to resort to the source code to really understand
it. An example command line would help a lot too.

[...]
> +static const AVOption ffprobe_options[] = {
> +    {"format", "set the flags which control how to display values", OFFSET(format_flags), FF_OPT_TYPE_FLAGS, DEFAULT, 0, UINT_MAX, D, "format_flags"},
> +    {"unit",   "show unit", 0, FF_OPT_TYPE_CONST, FORMAT_USE_UNIT, INT_MIN, INT_MAX, D, "format_flags"},
> +    {"prefix", "show binary or decimal SI prefixes", 0, FF_OPT_TYPE_CONST, FORMAT_USE_PREFIX,  INT_MIN, INT_MAX, D, "format_flags"},
> +    {"binary_prefix", "force binary SI prefixes", 0, FF_OPT_TYPE_CONST, FORMAT_USE_BINARY_PREFIX,  INT_MIN, INT_MAX, D, "format_flags"},
> +    {"byte_binary_prefix", "force binary SI prefixes with byte measure units", 0, FF_OPT_TYPE_CONST, FORMAT_USE_BYTE_BINARY_PREFIX,  INT_MIN, INT_MAX, D, "format_flags"},
> +    {"sexagesimal", "use sexagesimal time format", 0, FF_OPT_TYPE_CONST, FORMAT_USE_SEXAGESIMAL_TIME_FORMAT,  INT_MIN, INT_MAX, D, "format_flags"},
> +    {"pretty", "prettify format, the equivalent of unit+prefix+byte_binary_prefix+sexagesimal", 0, FF_OPT_TYPE_CONST,
> +     FORMAT_USE_UNIT|FORMAT_USE_PREFIX|FORMAT_USE_BYTE_BINARY_PREFIX|FORMAT_USE_SEXAGESIMAL_TIME_FORMAT,  INT_MIN, INT_MAX, D, "format_flags"},
> +
> +    {"show", "set the flags which control what to display", OFFSET(show_flags), FF_OPT_TYPE_FLAGS, DEFAULT, 0, UINT_MAX, D, "show_flags"},
> +    {"streams", "show streams", 0, FF_OPT_TYPE_CONST, SHOW_STREAMS, INT_MIN, INT_MAX, D, "show_flags"},

> +    {"format", "show streams", 0, FF_OPT_TYPE_CONST,  SHOW_FORMAT,  INT_MIN, INT_MAX, D, "show_flags"},

"show streams" twice?

[...]
> Index: ffmpeg/doc/ffprobe-doc.texi
> ===================================================================
> --- /dev/null	1970-01-01 00:00:00.000000000 +0000
> +++ ffmpeg/doc/ffprobe-doc.texi	2010-02-07 16:48:53.000000000 +0100
> @@ -0,0 +1,87 @@
[...]
> + at item -show_format
> +Show information about the container format of the input multimedia
> +stream.
> + at item -show_streams
> +Show information about each media stream contained in the input
> +multimedia stream.

This doesn't seem to work.

[...]