Thank you for the feedback Anton.

I only care about this value being displayed when calling ffprobe, so does it 
make sense to do this in ffprobe.c or could this be added to the existing mov.c 
demuxer?

“demuxer-private AV_OPT_FLAG_EXPORT” is new territory for me, so was wondering 
if there some example of how the similar values could be added to allow ffprobe 
to stdout the value – perhaps some existing commit request that shows how to 
make use of exposing values using AV_OPT_FLAG_EXPORT?

Thank you for your time.


From: ffmpeg-devel <ffmpeg-devel-boun...@ffmpeg.org> on behalf of Anton Khirnov 
<an...@khirnov.net>
Date: Wednesday, July 13, 2022 at 5:19 AM
To: Bryce Chester Newman <ffmpegag...@gmail.com>, ffmpeg-devel@ffmpeg.org 
<ffmpeg-devel@ffmpeg.org>
Subject: Re: [FFmpeg-devel] [EXTERNAL] [PATCH] [PATCH] libavformat/mov: Expose 
Quicktime poster_time value as metadata TAG.
Quoting Bryce Newman (2022-07-12 19:20:17)
> Hello,
> Can someone please have a look at this patch request?
> Thank you in advance.

Metadata is intended for user-presentable strings, not structured
values.

The least bad way of exporting this I can think of is adding a
demuxer-private AV_OPT_FLAG_EXPORT option.

Also, avoid floating point - it is inexact.

--
Anton Khirnov
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
    
    

Bryce Chester Newman | Principal Developer
 
p: +12069255045 | 



_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to