diff mbox series

[FFmpeg-devel] lavc/libx265: unbreak build for X265_BUILD >= 210

Message ID 20240811103300.11627-1-ffmpeg@gyani.pro
State New
Headers show
Series [FFmpeg-devel] lavc/libx265: unbreak build for X265_BUILD >= 210 | expand

Checks

Context Check Description
yinshiyou/make_loongarch64 success Make finished
yinshiyou/make_fate_loongarch64 success Make fate finished

Commit Message

Gyan Doshi Aug. 11, 2024, 10:33 a.m. UTC
x265 added support for alpha starting with build 210.
While doing so, x265_encoder_encode() changed its fifth arg to
an array of pointers to x265_picture. This broke building lavc/libx265.c

This patch simply unbreaks the build and maintains existing single-layer
non-alpha encoding support.

Fixes #11130
---
 libavcodec/libx265.c | 40 ++++++++++++++++++++++++++++++----------
 1 file changed, 30 insertions(+), 10 deletions(-)

Comments

Gyan Doshi Aug. 14, 2024, 4:31 a.m. UTC | #1
On 2024-08-11 04:03 pm, Gyan Doshi wrote:
> x265 added support for alpha starting with build 210.
> While doing so, x265_encoder_encode() changed its fifth arg to
> an array of pointers to x265_picture. This broke building lavc/libx265.c
>
> This patch simply unbreaks the build and maintains existing single-layer
> non-alpha encoding support.
>
> Fixes #11130

Plan to push tomorrow. Building with x265 HEAD has been broken for over 
a week now.

Regards,
Gyan

> ---
>   libavcodec/libx265.c | 40 ++++++++++++++++++++++++++++++----------
>   1 file changed, 30 insertions(+), 10 deletions(-)
>
> diff --git a/libavcodec/libx265.c b/libavcodec/libx265.c
> index 0dc7ab6eeb..3bc3b5a03e 100644
> --- a/libavcodec/libx265.c
> +++ b/libavcodec/libx265.c
> @@ -661,7 +661,13 @@ static int libx265_encode_frame(AVCodecContext *avctx, AVPacket *pkt,
>   {
>       libx265Context *ctx = avctx->priv_data;
>       x265_picture x265pic;
> -    x265_picture x265pic_out = { 0 };
> +#if X265_BUILD >= 210
> +    x265_picture x265pic_layers_out[MAX_SCALABLE_LAYERS];
> +    x265_picture* x265pic_lyrptr_out[MAX_SCALABLE_LAYERS];
> +#else
> +    x265_picture x265pic_solo_out = { 0 };
> +#endif
> +    x265_picture* x265pic_out;
>       x265_nal *nal;
>       x265_sei *sei;
>       uint8_t *dst;
> @@ -798,8 +804,16 @@ static int libx265_encode_frame(AVCodecContext *avctx, AVPacket *pkt,
>   #endif
>       }
>   
> +#if X265_BUILD >= 210
> +    for (i = 0; i < MAX_SCALABLE_LAYERS; i++)
> +        x265pic_lyrptr_out[i] = &x265pic_layers_out[i];
> +
> +    ret = ctx->api->encoder_encode(ctx->encoder, &nal, &nnal,
> +                                   pic ? &x265pic : NULL, x265pic_lyrptr_out);
> +#else
>       ret = ctx->api->encoder_encode(ctx->encoder, &nal, &nnal,
> -                                   pic ? &x265pic : NULL, &x265pic_out);
> +                                   pic ? &x265pic : NULL, &x265pic_solo_out);
> +#endif
>   
>       for (i = 0; i < sei->numPayloads; i++)
>           av_free(sei->payloads[i].payload);
> @@ -829,10 +843,16 @@ static int libx265_encode_frame(AVCodecContext *avctx, AVPacket *pkt,
>               pkt->flags |= AV_PKT_FLAG_KEY;
>       }
>   
> -    pkt->pts = x265pic_out.pts;
> -    pkt->dts = x265pic_out.dts;
> +#if X265_BUILD >= 210
> +    x265pic_out = x265pic_lyrptr_out[0];
> +#else
> +    x265pic_out = &x265pic_solo_out;
> +#endif
> +
> +    pkt->pts = x265pic_out->pts;
> +    pkt->dts = x265pic_out->dts;
>   
> -    switch (x265pic_out.sliceType) {
> +    switch (x265pic_out->sliceType) {
>       case X265_TYPE_IDR:
>       case X265_TYPE_I:
>           pict_type = AV_PICTURE_TYPE_I;
> @@ -850,16 +870,16 @@ static int libx265_encode_frame(AVCodecContext *avctx, AVPacket *pkt,
>       }
>   
>   #if X265_BUILD >= 130
> -    if (x265pic_out.sliceType == X265_TYPE_B)
> +    if (x265pic_out->sliceType == X265_TYPE_B)
>   #else
> -    if (x265pic_out.frameData.sliceType == 'b')
> +    if (x265pic_out->frameData.sliceType == 'b')
>   #endif
>           pkt->flags |= AV_PKT_FLAG_DISPOSABLE;
>   
> -    ff_side_data_set_encoder_stats(pkt, x265pic_out.frameData.qp * FF_QP2LAMBDA, NULL, 0, pict_type);
> +    ff_side_data_set_encoder_stats(pkt, x265pic_out->frameData.qp * FF_QP2LAMBDA, NULL, 0, pict_type);
>   
> -    if (x265pic_out.userData) {
> -        int idx = (int)(intptr_t)x265pic_out.userData - 1;
> +    if (x265pic_out->userData) {
> +        int idx = (int)(intptr_t)x265pic_out->userData - 1;
>           ReorderedData *rd = &ctx->rd[idx];
>   
>           pkt->duration           = rd->duration;
Gyan Doshi Aug. 15, 2024, 9:49 a.m. UTC | #2
On 2024-08-14 10:01 am, Gyan Doshi wrote:
>
>
> On 2024-08-11 04:03 pm, Gyan Doshi wrote:
>> x265 added support for alpha starting with build 210.
>> While doing so, x265_encoder_encode() changed its fifth arg to
>> an array of pointers to x265_picture. This broke building lavc/libx265.c
>>
>> This patch simply unbreaks the build and maintains existing single-layer
>> non-alpha encoding support.
>>
>> Fixes #11130
>
> Plan to push tomorrow. Building with x265 HEAD has been broken for 
> over a week now.

Pushed as 1f801dfdb5066aadf0ade9cb5e94d620f33eacdc

Regards,
Gyan
diff mbox series

Patch

diff --git a/libavcodec/libx265.c b/libavcodec/libx265.c
index 0dc7ab6eeb..3bc3b5a03e 100644
--- a/libavcodec/libx265.c
+++ b/libavcodec/libx265.c
@@ -661,7 +661,13 @@  static int libx265_encode_frame(AVCodecContext *avctx, AVPacket *pkt,
 {
     libx265Context *ctx = avctx->priv_data;
     x265_picture x265pic;
-    x265_picture x265pic_out = { 0 };
+#if X265_BUILD >= 210
+    x265_picture x265pic_layers_out[MAX_SCALABLE_LAYERS];
+    x265_picture* x265pic_lyrptr_out[MAX_SCALABLE_LAYERS];
+#else
+    x265_picture x265pic_solo_out = { 0 };
+#endif
+    x265_picture* x265pic_out;
     x265_nal *nal;
     x265_sei *sei;
     uint8_t *dst;
@@ -798,8 +804,16 @@  static int libx265_encode_frame(AVCodecContext *avctx, AVPacket *pkt,
 #endif
     }
 
+#if X265_BUILD >= 210
+    for (i = 0; i < MAX_SCALABLE_LAYERS; i++)
+        x265pic_lyrptr_out[i] = &x265pic_layers_out[i];
+
+    ret = ctx->api->encoder_encode(ctx->encoder, &nal, &nnal,
+                                   pic ? &x265pic : NULL, x265pic_lyrptr_out);
+#else
     ret = ctx->api->encoder_encode(ctx->encoder, &nal, &nnal,
-                                   pic ? &x265pic : NULL, &x265pic_out);
+                                   pic ? &x265pic : NULL, &x265pic_solo_out);
+#endif
 
     for (i = 0; i < sei->numPayloads; i++)
         av_free(sei->payloads[i].payload);
@@ -829,10 +843,16 @@  static int libx265_encode_frame(AVCodecContext *avctx, AVPacket *pkt,
             pkt->flags |= AV_PKT_FLAG_KEY;
     }
 
-    pkt->pts = x265pic_out.pts;
-    pkt->dts = x265pic_out.dts;
+#if X265_BUILD >= 210
+    x265pic_out = x265pic_lyrptr_out[0];
+#else
+    x265pic_out = &x265pic_solo_out;
+#endif
+
+    pkt->pts = x265pic_out->pts;
+    pkt->dts = x265pic_out->dts;
 
-    switch (x265pic_out.sliceType) {
+    switch (x265pic_out->sliceType) {
     case X265_TYPE_IDR:
     case X265_TYPE_I:
         pict_type = AV_PICTURE_TYPE_I;
@@ -850,16 +870,16 @@  static int libx265_encode_frame(AVCodecContext *avctx, AVPacket *pkt,
     }
 
 #if X265_BUILD >= 130
-    if (x265pic_out.sliceType == X265_TYPE_B)
+    if (x265pic_out->sliceType == X265_TYPE_B)
 #else
-    if (x265pic_out.frameData.sliceType == 'b')
+    if (x265pic_out->frameData.sliceType == 'b')
 #endif
         pkt->flags |= AV_PKT_FLAG_DISPOSABLE;
 
-    ff_side_data_set_encoder_stats(pkt, x265pic_out.frameData.qp * FF_QP2LAMBDA, NULL, 0, pict_type);
+    ff_side_data_set_encoder_stats(pkt, x265pic_out->frameData.qp * FF_QP2LAMBDA, NULL, 0, pict_type);
 
-    if (x265pic_out.userData) {
-        int idx = (int)(intptr_t)x265pic_out.userData - 1;
+    if (x265pic_out->userData) {
+        int idx = (int)(intptr_t)x265pic_out->userData - 1;
         ReorderedData *rd = &ctx->rd[idx];
 
         pkt->duration           = rd->duration;