[FFmpeg-devel] [PATCH 1/4] avcodec/hevcdec: add timecode metadata

lance.lmwang at gmail.com lance.lmwang at gmail.com
Wed Jul 8 04:16:08 EEST 2020


On Thu, Jul 02, 2020 at 10:33:24PM +0800, lance.lmwang at gmail.com wrote:
> From: Limin Wang <lance.lmwang at gmail.com>
> 
> Signed-off-by: Limin Wang <lance.lmwang at gmail.com>
> ---
>  libavcodec/hevcdec.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/libavcodec/hevcdec.c b/libavcodec/hevcdec.c
> index e363e68..3691814 100644
> --- a/libavcodec/hevcdec.c
> +++ b/libavcodec/hevcdec.c
> @@ -2820,6 +2820,7 @@ static int set_side_data(HEVCContext *s)
>  
>      if (s->sei.timecode.present) {
>          uint32_t *tc_sd;
> +        char tcbuf[AV_TIMECODE_STR_SIZE];
>          AVFrameSideData *tcside = av_frame_new_side_data(out, AV_FRAME_DATA_S12M_TIMECODE,
>                                                           sizeof(uint32_t) * 4);
>          if (!tcside)
> @@ -2836,6 +2837,8 @@ static int set_side_data(HEVCContext *s)
>              int   ff = s->sei.timecode.n_frames[i];
>  
>              tc_sd[i + 1] = av_timecode_get_smpte(s->avctx->framerate, drop, hh, mm, ss, ff);
> +            av_timecode_make_smpte_tc_string(tcbuf, tc_sd[i + 1], 0);
> +            av_dict_set(&out->metadata, "timecode", tcbuf, 0);
>          }
>  
>          s->sei.timecode.num_clock_ts = 0;
> -- 
> 1.8.3.1
> 

ping, please comments whether it's proper to add timecode metadata in this way.

-- 
Thanks,
Limin Wang


More information about the ffmpeg-devel mailing list