[FFmpeg-devel] [FFMPEG DEVEL] [PATCH v6] fftools/ffprobe: Add S12M Timecode output as side data (such as SEI TC)

[Marton Balint]

On Fri, 24 May 2019, Antonin Gouzer wrote:

> ---
> Add S12M Timecode output with the show_frame option
> Multiple timecodes (3) for one frame support
> Control side date Size to 16
> Correct ffrpobe.xsd to allow multiple timecodes in side_data element
> ---
> doc/ffprobe.xsd   |  8 ++++++++
> fftools/ffprobe.c | 14 +++++++++++++-
> 2 files changed, 21 insertions(+), 1 deletion(-)

I almost committed this, but found another issue: the JSON output 
contained the "timecode" key multiple times. Apparently in order for the 
JSON output to work, we need a separate section for arrays.

I managed to make those changes (see the attached patch), it affected the 
XML output slightly as well. Please let me know if this is satisfactory to 
you.

Thanks,
Marton
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-fftools-ffprobe-Add-S12M-Timecode-output-as-side-dat.patch
Type: text/x-patch
Size: 4991 bytes
Desc: 
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20190525/6ca10faf/attachment.bin>