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

[Marton Balint]

On Sat, 25 May 2019, Antonin Gouzer wrote:

> Hello,
> Shame on me, I didn't test the patch with the json format, only xml and flat.
> It's ok now.
> ok for XML too.
>
> Thanks!

Thanks, applied.

Regards,
Marton

>
>
> Le sam. 25 mai 2019 à 00:27, Marton Balint <cus at passwd.hu> a écrit :
>>
>>
>>
>> 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_______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>> To unsubscribe, visit link above, or email
>> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".