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

[Florent Tribouilloy]

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 :

 > +      <xsd:attribute name="start_time"           type="xsd:float"/>
 > +      <xsd:attribute name="start_pts"            type="xsd:int"/>
 > +      <xsd:attribute name="end_time"             type="xsd:float"/>
 > +      <xsd:attribute name="end_pts"              type="xsd:int"/>

-- 
Florent Tribouilloy
Intern at Smartjog