[FFmpeg-devel] [PATCH 1/2] Add programs to ffprobe
Stefano Sabatini
stefasab at gmail.com
Sun Jul 28 16:43:02 CEST 2013
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
More information about the ffmpeg-devel
mailing list