diff mbox

[FFmpeg-devel,2/2] libavcodec/ivi_dsp.c: fix warning due to misleading indentation

Message ID CAHucTJuN7-RLWuiSH+udaNWwM5_jZ3u9QZhWNcTeaO+bqsZwzg@mail.gmail.com
State Accepted
Headers show

Commit Message

Adriano Pallavicino Oct. 2, 2016, 6:46 p.m. UTC

Comments

Josh Dekker Oct. 3, 2016, 2:40 p.m. UTC | #1 
On 02/10/2016 19:46, Adriano Pallavicino wrote:

LGTM. Will apply both patches squashed in a day if there are no further 
comments.
Josh Dekker Oct. 4, 2016, 5:44 p.m. UTC | #2 
On 03/10/2016 15:40, Josh de Kock wrote:
> On 02/10/2016 19:46, Adriano Pallavicino wrote:
>
> LGTM. Will apply both patches squashed in a day if there are no further
> comments.
>

Applied.
diff mbox

Patch

From ca69f377f019dc72e7f34020634b57e1e9c6a8d2 Mon Sep 17 00:00:00 2001
From: Adriano Pallavicino <adriano.pallavicino@gmail.com>
Date: Sun, 2 Oct 2016 20:42:44 +0200
Subject: [PATCH] Fix warning due to misleading indendation in ivi_dsp.c line
 592
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

libavcodec/ivi_dsp.c: In function ‘ff_ivi_inverse_slant_4x4’:
libavcodec/ivi_dsp.c:592:11: warning: this ‘else’ clause does not guard... [-Wmisleading-indentation]
         } else
           ^~~~
libavcodec/ivi_dsp.c:595:13: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘else’
             src++;

Changes to be committed:
	modified:   libavcodec/ivi_dsp.c

Signed-off-by: Adriano Pallavicino <adriano.pallavicino@gmail.com>
---
 libavcodec/ivi_dsp.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/libavcodec/ivi_dsp.c b/libavcodec/ivi_dsp.c
index 99a7ad5..5211269 100644
--- a/libavcodec/ivi_dsp.c
+++ b/libavcodec/ivi_dsp.c
@@ -592,8 +592,8 @@  void ff_ivi_inverse_slant_4x4(const int32_t *in, int16_t *out, uint32_t pitch, c
         } else
             dst[0] = dst[4] = dst[8] = dst[12] = 0;
 
-            src++;
-            dst++;
+        src++;
+        dst++;
     }
 #undef COMPENSATE
 
-- 
2.7.4