Jan Ekström: > The newer of these two are the separate integers for content light > level, introduced in 3952bf3e98c76c31594529a3fe34e056d3e3e2ea , > with X265_BUILD 75. As we already require X265_BUILD of at least > 89, no further conditions are required. > --- > libavcodec/libx265.c | 82 ++++++++++++++++++++++++++++++++++++ > tests/fate/enc_external.mak | 5 +++ > tests/ref/fate/libx265-hdr10 | 16 +++++++ > 3 files changed, 103 insertions(+) > create mode 100644 tests/ref/fate/libx265-hdr10 > > diff --git a/libavcodec/libx265.c b/libavcodec/libx265.c > index 873b3021ee..2204b5146a 100644 > --- a/libavcodec/libx265.c > +++ b/libavcodec/libx265.c > @@ -28,9 +28,11 @@ > #include <float.h> > > #include "libavutil/avassert.h" > +#include "libavutil/bprint.h" > #include "libavutil/buffer.h" > #include "libavutil/internal.h" > #include "libavutil/common.h" > +#include "libavutil/mastering_display_metadata.h" > #include "libavutil/opt.h" > #include "libavutil/pixdesc.h" > #include "avcodec.h" > @@ -179,6 +181,79 @@ static av_cold int > libx265_param_parse_int(AVCodecContext *avctx, > return 0; > } > > +static int handle_mdcv(AVCodecContext *avctx, const x265_api *api,
You are not using an AVCodecContext* at all in this function; you only need a logcontext (i.e. void*). > + x265_param *params, > + const AVMasteringDisplayMetadata *mdcv) > +{ > + int ret = AVERROR_BUG; > + static const char *option = "master-display"; This pointer is completely unnecessary. > + AVBPrint buf; > + av_bprint_init(&buf, 0, AV_BPRINT_SIZE_AUTOMATIC); > + > + // G(%hu,%hu)B(%hu,%hu)R(%hu,%hu)WP(%hu,%hu)L(%u,%u) > + av_bprintf( > + &buf, > + "G(%"PRId64",%"PRId64")B(%"PRId64",%"PRId64")R(%"PRId64",%"PRId64")" > + "WP(%"PRId64",%"PRId64")L(%"PRId64",%"PRId64")", > + av_rescale_q(1, mdcv->display_primaries[1][0], (AVRational){ 1, > 50000 }), > + av_rescale_q(1, mdcv->display_primaries[1][1], (AVRational){ 1, > 50000 }), > + av_rescale_q(1, mdcv->display_primaries[2][0], (AVRational){ 1, > 50000 }), > + av_rescale_q(1, mdcv->display_primaries[2][1], (AVRational){ 1, > 50000 }), > + av_rescale_q(1, mdcv->display_primaries[0][0], (AVRational){ 1, > 50000 }), > + av_rescale_q(1, mdcv->display_primaries[0][1], (AVRational){ 1, > 50000 }), > + av_rescale_q(1, mdcv->white_point[0], (AVRational){ 1, 50000 }), > + av_rescale_q(1, mdcv->white_point[1], (AVRational){ 1, 50000 }), > + av_rescale_q(1, mdcv->max_luminance, (AVRational){ 1, 10000 }), > + av_rescale_q(1, mdcv->min_luminance, (AVRational){ 1, 10000 })); > + > + if (!av_bprint_is_complete(&buf)) { > + ret = AVERROR(ENOMEM); When using AV_BPRINT_SIZE_AUTOMATIC the buffer being incomplete does not mean that an allocation failure occurred, as no allocations are attempted at all. Furthermore, I don't see the point of using the AVBPrint API here at all. All you need is a big enough buffer and snprintf. > + goto end; > + } > + > + if (api->param_parse(params, option, buf.str) == > + X265_PARAM_BAD_VALUE) { > + av_log(avctx, AV_LOG_ERROR, > + "Invalid value \"%s\" for param \"%s\".\n", > + buf.str, option); > + ret = AVERROR(EINVAL); > + goto end; > + } > + > + ret = 0; > + > +end: > + av_bprint_finalize(&buf, NULL); This is unnecessary: AV_BPRINT_SIZE_AUTOMATIC uses only the buffer available in the AVBPrint itself; there are no dynamic allocations. > + > + return ret; > +} > + > +static int handle_side_data(AVCodecContext *avctx, const x265_api *api, > + x265_param *params) > +{ > + const AVFrameSideDataSet set = avctx->side_data_set; > + const AVFrameSideData *cll_sd = > + av_side_data_set_get_item(set, AV_FRAME_DATA_CONTENT_LIGHT_LEVEL); > + const AVFrameSideData *mdcv_sd = > + av_side_data_set_get_item(set, > AV_FRAME_DATA_MASTERING_DISPLAY_METADATA); > + > + if (cll_sd) { > + const AVContentLightMetadata *cll = > + (AVContentLightMetadata *)cll_sd->data; > + > + params->maxCLL = cll->MaxCLL; > + params->maxFALL = cll->MaxFALL; > + } > + > + if (mdcv_sd) { > + int ret = handle_mdcv(avctx, api, params, > (AVMasteringDisplayMetadata *)mdcv_sd->data); > + if (ret < 0) > + return ret; > + } > + > + return 0; > +} > + > static av_cold int libx265_encode_init(AVCodecContext *avctx) > { > libx265Context *ctx = avctx->priv_data; > @@ -339,6 +414,13 @@ FF_ENABLE_DEPRECATION_WARNINGS > return AVERROR_BUG; > } > > + ret = handle_side_data(avctx, ctx->api, ctx->params); > + if (ret < 0) { > + av_log(avctx, AV_LOG_ERROR, "Failed handling side data! (%s)\n", > + av_err2str(ret)); > + return ret; > + } > + > if (ctx->crf >= 0) { > char crf[6]; > > diff --git a/tests/fate/enc_external.mak b/tests/fate/enc_external.mak > index 90d8894d04..8d6df2febd 100644 > --- a/tests/fate/enc_external.mak > +++ b/tests/fate/enc_external.mak > @@ -12,5 +12,10 @@ FATE_ENC_EXTERNAL-$(call ENCDEC, LIBX264 HEVC, MOV, > HEVC_DEMUXER H264_DECODER) + > fate-libx264-hdr10: CMD = enc_external > $(TARGET_SAMPLES)/hevc/hdr10_plus_h265_sample.hevc \ > mp4 "-c:v libx264" "-show_frames -show_entries frame=side_data_list -of > flat" > > +# test for x265 MDCV and CLL passthrough during encoding > +FATE_ENC_EXTERNAL-$(call ENCDEC, LIBX265 HEVC, MOV, HEVC_DEMUXER) += > fate-libx265-hdr10 > +fate-libx265-hdr10: CMD = enc_external > $(TARGET_SAMPLES)/hevc/hdr10_plus_h265_sample.hevc \ > + mp4 "-c:v libx265" "-show_frames -show_entries frame=side_data_list -of > flat" > + > FATE_SAMPLES_FFMPEG_FFPROBE += $(FATE_ENC_EXTERNAL-yes) > fate-enc-external: $(FATE_ENC_EXTERNAL-yes) > diff --git a/tests/ref/fate/libx265-hdr10 b/tests/ref/fate/libx265-hdr10 > new file mode 100644 > index 0000000000..571c837cac > --- /dev/null > +++ b/tests/ref/fate/libx265-hdr10 > @@ -0,0 +1,16 @@ > +frames.frame.0.side_data_list.side_data.0.side_data_type="H.26[45] User Data > Unregistered SEI message" > +frames.frame.0.side_data_list.side_data.1.side_data_type="H.26[45] User Data > Unregistered SEI message" > +frames.frame.0.side_data_list.side_data.2.side_data_type="Mastering display > metadata" > +frames.frame.0.side_data_list.side_data.2.red_x="13250/50000" > +frames.frame.0.side_data_list.side_data.2.red_y="34500/50000" > +frames.frame.0.side_data_list.side_data.2.green_x="7500/50000" > +frames.frame.0.side_data_list.side_data.2.green_y="3000/50000" > +frames.frame.0.side_data_list.side_data.2.blue_x="34000/50000" > +frames.frame.0.side_data_list.side_data.2.blue_y="16000/50000" > +frames.frame.0.side_data_list.side_data.2.white_point_x="15635/50000" > +frames.frame.0.side_data_list.side_data.2.white_point_y="16450/50000" > +frames.frame.0.side_data_list.side_data.2.min_luminance="50/10000" > +frames.frame.0.side_data_list.side_data.2.max_luminance="10000000/10000" > +frames.frame.0.side_data_list.side_data.3.side_data_type="Content light > level metadata" > +frames.frame.0.side_data_list.side_data.3.max_content=1000 > +frames.frame.0.side_data_list.side_data.3.max_average=200 _______________________________________________ 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".