[FFmpeg-devel] [PATCH 1/2] Add programs to ffprobe

[Stefano Sabatini]

On date Friday 2013-07-26 18:29:04 +0200, Florent Tribouilloy encoded:
> On 07/26/2013 04:59 PM, Stefano Sabatini wrote:
> >On date Thursday 2013-07-25 12:55:40 +0200, Florent Tribouilloy encoded:
> >>Option -show_programs for ffprobe will sort the output by programs.
> >>
> >>Signed-off-by: Florent Tribouilloy<florent.tribouilloy at smartjog.com>
> >>---
> >>  doc/ffprobe.texi               |   14 ++++++--
> >>  doc/ffprobe.xsd                |   24 +++++++++++++
> >>  ffprobe.c                      |   76 +++++++++++++++++++++++++++++++++++-----
> >>  tests/ref/fate/ffprobe_compact |    2 +-
> >>  tests/ref/fate/ffprobe_csv     |    2 +-
> >>  tests/ref/fate/ffprobe_default |    1 +
> >>  tests/ref/fate/ffprobe_flat    |    1 +
> >>  tests/ref/fate/ffprobe_ini     |    1 +
> >>  tests/ref/fate/ffprobe_json    |    1 +
> >>  tests/ref/fate/ffprobe_xml     |    2 +-
> >>  10 files changed, 110 insertions(+), 14 deletions(-)
> >
> >I tested the patch and was not very satisifed with the output, since I
> >liked to have an array with all the nested program streams, check
> >attachment (consider the JSON output in particular).
> 
> I did not print the section 'program_streams' because it was a
> little bit redundant (but, at that time, I did not print any other
> information about the program). Now, it makes sense :)
> 

> In the documentation, ffprobe.xsd, you should add "use="required"" to :

No, that is already handled by the print_ts/time functions.

Note: ffprobe.xsd is not only documentation but must be validated
through xmllint or similar tool.

I applied some more fixes, and pushed, thanks.
-- 
FFmpeg = Fantastic Fast Moronic Political Earthshaking Gangster