diff mbox series

[FFmpeg-devel,01/24] sws: remove unnecessary braces

Message ID 20210531075515.19544-1-anton@khirnov.net
State Accepted
Commit f8c21ccbfcb7f736b4f673523de18f0611271967
Headers show
Series [FFmpeg-devel,01/24] sws: remove unnecessary braces | expand

Checks

Context Check Description
andriy/x86_make success Make finished
andriy/x86_make_fate success Make fate finished
andriy/PPC64_make success Make finished
andriy/PPC64_make_fate success Make fate finished

Commit Message

Anton Khirnov May 31, 2021, 7:54 a.m. UTC 
There used to be more code inside them, but it was removed in
6de58b49032a206985602effec91e5e46c886ea2.
---
 libswscale/swscale.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

Comments

Michael Niedermayer June 1, 2021, 10:28 a.m. UTC | #1 
On Mon, May 31, 2021 at 09:54:52AM +0200, Anton Khirnov wrote:
> There used to be more code inside them, but it was removed in
> 6de58b49032a206985602effec91e5e46c886ea2.
> ---
>  libswscale/swscale.c | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)

LGTM

thx

[...]
Anton Khirnov July 3, 2021, 2:27 p.m. UTC | #2 
Pushed the non-controversial parts of the thread.

Will rework and resend the rest later.
diff mbox series

Patch

diff --git a/libswscale/swscale.c b/libswscale/swscale.c
index 12160a169a..6b73c71e41 100644
--- a/libswscale/swscale.c
+++ b/libswscale/swscale.c
@@ -487,10 +487,8 @@  static int swscale(SwsContext *c, const uint8_t *src[],
                            yuv2packed1, yuv2packed2, yuv2packedX, yuv2anyX, use_mmx_vfilter);
         }
 
-        {
-            for (i = vStart; i < vEnd; ++i)
-                desc[i].process(c, &desc[i], dstY, 1);
-        }
+        for (i = vStart; i < vEnd; ++i)
+            desc[i].process(c, &desc[i], dstY, 1);
     }
     if (isPlanar(dstFormat) && isALPHA(dstFormat) && !needAlpha) {
         int length = dstW;