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

Stefano Sabatini stefasab at gmail.com
Tue Jul 23 00:17:16 CEST 2013


On date Monday 2013-07-22 11:48:14 +0200, Florent Tribouilloy encoded:
> Option -show_programs for ffprobe will sort the output
> 
> Signed-off-by: Florent Tribouilloy <florent.tribouilloy at smartjog.com>
> ---
>  doc/ffprobe.texi |   14 ++++++++---
>  doc/ffprobe.xsd  |   26 +++++++++++++++++++++
>  ffprobe.c        |   68 +++++++++++++++++++++++++++++++++++++++++++++++-------
>  3 files changed, 97 insertions(+), 11 deletions(-)
[...]
> +static void show_program(WriterContext *w, AVFormatContext *fmt_ctx, AVProgram *program)
> +{
> +    int i;
> +
> +    writer_print_section_header(w, SECTION_ID_PROGRAM);
> +    print_int("program_id", program->id);
> +    print_int("program_num", program->program_num);
> +    print_int("nb_streams", program->nb_stream_indexes);
> +    print_int("pmt_pid", program->pmt_pid);
> +    print_int("pcr_pid", program->pcr_pid);
> +    show_tags(w, program->metadata, SECTION_ID_PROGRAM_TAGS);
> +    for (i = 0; i < program->nb_stream_indexes; i++) {
> +        if (selected_streams[program->stream_index[i]])
> +            show_stream(w, fmt_ctx, program->stream_index[i], 1);
> +    }
> +    writer_print_section_footer(w);

No start_time/end_time? Those could be potentially useful.
(Note: we could have start/end_time and start/end_pts). Yes this can
be added in a further patch.

Should be good, when merged with the fate reference changes, so that
we don't introduce regressions (I'll do it myself and push if you
don't reply in a few days).
-- 
FFmpeg = Foolish and Free Mastering Practical Enlightened Glue


More information about the ffmpeg-devel mailing list