[FFmpeg-devel] [PATCH 18/24] sws: move initializing dither_error higher up

Anton Khirnov anton at khirnov.net
Mon May 31 10:55:09 EEST 2021


It does not interact in any way with the code setting up the image
pointers/strides, so it should not be intermixed with it.
---
 libswscale/swscale.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/libswscale/swscale.c b/libswscale/swscale.c
index 2d6ed95cfb..b9b2826911 100644
--- a/libswscale/swscale.c
+++ b/libswscale/swscale.c
@@ -907,6 +907,10 @@ int attribute_align_arg sws_scale(struct SwsContext *c,
     if (c->cascaded_context[0] && srcSliceY == 0 && srcSliceH == c->cascaded_context[0]->srcH)
         return scale_cascaded(c, srcSlice, srcStride, srcSliceY, srcSliceH, dst, dstStride);
 
+    if (!srcSliceY && (c->flags & SWS_BITEXACT) && c->dither == SWS_DITHER_ED && c->dither_error[0])
+        for (i = 0; i < 4; i++)
+            memset(c->dither_error[i], 0, sizeof(c->dither_error[0][0]) * (c->dstW+2));
+
     memcpy(src2,       srcSlice,  sizeof(src2));
     memcpy(dst2,       dst,       sizeof(dst2));
     memcpy(srcStride2, srcStride, sizeof(srcStride2));
@@ -959,10 +963,6 @@ int attribute_align_arg sws_scale(struct SwsContext *c,
         src2[0] = base;
     }
 
-    if (!srcSliceY && (c->flags & SWS_BITEXACT) && c->dither == SWS_DITHER_ED && c->dither_error[0])
-        for (i = 0; i < 4; i++)
-            memset(c->dither_error[i], 0, sizeof(c->dither_error[0][0]) * (c->dstW+2));
-
     if (c->sliceDir != 1) {
         // slices go from bottom to top => we flip the image internally
         for (i=0; i<4; i++) {
-- 
2.30.2



More information about the ffmpeg-devel mailing list