diff mbox

[FFmpeg-devel,2/2] reindent

Message ID 20170529145133.24967-2-jdarnley@obe.tv
State Accepted
Commit 0dea0114fb29a8c533d7c36f20df7a7440605758
Headers show

Commit Message

James Darnley May 29, 2017, 2:51 p.m. UTC 
---
 libavcodec/x86/idctdsp_init.c | 38 +++++++++++++++++++-------------------
 1 file changed, 19 insertions(+), 19 deletions(-)

Comments

James Darnley May 30, 2017, 10:44 a.m. UTC | #1 
On 2017-05-29 16:51, James Darnley wrote:
> Commit message: reindent

Is this acceptable?  Should I be more verbose?
Ronald S. Bultje May 30, 2017, 10:48 a.m. UTC | #2 
Hi,

On May 30, 2017 6:44 AM, "James Darnley" <jdarnley@obe.tv> wrote:

On 2017-05-29 16:51, James Darnley wrote:
> Commit message: reindent

Is this acceptable?  Should I be more verbose?


Make it x86/idctdsp_init: reindent. Otherwise ok.

Thanks,
Ronald
diff mbox

Patch

diff --git a/libavcodec/x86/idctdsp_init.c b/libavcodec/x86/idctdsp_init.c
index 1f308cc079..f1c915aa00 100644
--- a/libavcodec/x86/idctdsp_init.c
+++ b/libavcodec/x86/idctdsp_init.c
@@ -68,16 +68,16 @@  av_cold void ff_idctdsp_init_x86(IDCTDSPContext *c, AVCodecContext *avctx,
         c->put_pixels_clamped        = ff_put_pixels_clamped_mmx;
         c->add_pixels_clamped        = ff_add_pixels_clamped_mmx;
 
-            if (!high_bit_depth &&
-                avctx->lowres == 0 &&
-                (avctx->idct_algo == FF_IDCT_AUTO ||
-                 avctx->idct_algo == FF_IDCT_SIMPLEAUTO ||
-                 avctx->idct_algo == FF_IDCT_SIMPLEMMX)) {
-                    c->idct_put  = ff_simple_idct_put_mmx;
-                    c->idct_add  = ff_simple_idct_add_mmx;
-                    c->idct      = ff_simple_idct_mmx;
-                    c->perm_type = FF_IDCT_PERM_SIMPLE;
-            }
+        if (!high_bit_depth &&
+            avctx->lowres == 0 &&
+            (avctx->idct_algo == FF_IDCT_AUTO ||
+                avctx->idct_algo == FF_IDCT_SIMPLEAUTO ||
+                avctx->idct_algo == FF_IDCT_SIMPLEMMX)) {
+                c->idct_put  = ff_simple_idct_put_mmx;
+                c->idct_add  = ff_simple_idct_add_mmx;
+                c->idct      = ff_simple_idct_mmx;
+                c->perm_type = FF_IDCT_PERM_SIMPLE;
+        }
     }
 
     if (EXTERNAL_SSE2(cpu_flags)) {
@@ -85,15 +85,15 @@  av_cold void ff_idctdsp_init_x86(IDCTDSPContext *c, AVCodecContext *avctx,
         c->put_pixels_clamped        = ff_put_pixels_clamped_sse2;
         c->add_pixels_clamped        = ff_add_pixels_clamped_sse2;
 
-            if (!high_bit_depth &&
-                avctx->lowres == 0 &&
-                (avctx->idct_algo == FF_IDCT_AUTO ||
-                 avctx->idct_algo == FF_IDCT_SIMPLEAUTO ||
-                 avctx->idct_algo == FF_IDCT_SIMPLEMMX)) {
-                    c->idct_put  = ff_simple_idct_put_sse2;
-                    c->idct_add  = ff_simple_idct_add_sse2;
-                    c->perm_type = FF_IDCT_PERM_SIMPLE;
-            }
+        if (!high_bit_depth &&
+            avctx->lowres == 0 &&
+            (avctx->idct_algo == FF_IDCT_AUTO ||
+                avctx->idct_algo == FF_IDCT_SIMPLEAUTO ||
+                avctx->idct_algo == FF_IDCT_SIMPLEMMX)) {
+                c->idct_put  = ff_simple_idct_put_sse2;
+                c->idct_add  = ff_simple_idct_add_sse2;
+                c->perm_type = FF_IDCT_PERM_SIMPLE;
+        }
     }
 
     if (ARCH_X86_64 && avctx->lowres == 0) {