From fdbc92f8cc59e5ae59949d68319c8a61e270b701 Mon Sep 17 00:00:00 2001
From: Andri Yngvason <andri@yngvason.is>
Date: Thu, 22 Aug 2024 21:26:59 +0000
Subject: H264Libav: Use nearest-neighbour sampling in scaler

sws_getCachedContext will set a default sampling method if 0 is passed
to the flags argument. This means that when it is called again, the
flags argument will not match the flags in the context, so a new one
will be allocated every time.

To get around this problem, we assign an explicit sampling method, one
that also happens to be more efficient and just as good for this
use-case as the default one, which is bicubic interpolation.

(cherry picked from commit c68714aee15f1e6c75d53c398ea9b9178f64fcfb)
---
 common/rfb/H264LibavDecoderContext.cxx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'common')

diff --git a/common/rfb/H264LibavDecoderContext.cxx b/common/rfb/H264LibavDecoderContext.cxx
index a7c817bf..789d0228 100644
--- a/common/rfb/H264LibavDecoderContext.cxx
+++ b/common/rfb/H264LibavDecoderContext.cxx
@@ -215,7 +215,7 @@ void H264LibavDecoderContext::decode(const uint8_t* h264_in_buffer,
 
   sws = sws_getCachedContext(sws, frame->width, frame->height, avctx->pix_fmt,
                              frame->width, frame->height, AV_PIX_FMT_RGB32,
-                             0, NULL, NULL, NULL);
+                             SWS_POINT, NULL, NULL, NULL);
 
   int inFull, outFull, brightness, contrast, saturation;
   const int* inTable;
-- 
cgit v1.2.3