Message ID | 20210531075515.19544-1-anton@khirnov.net |
---|---|
State | Accepted |
Commit | f8c21ccbfcb7f736b4f673523de18f0611271967 |
Headers | show |
Series | [FFmpeg-devel,01/24] sws: remove unnecessary braces | expand |
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 |
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 [...]
Pushed the non-controversial parts of the thread. Will rework and resend the rest later.
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;