diff mbox

[FFmpeg-devel,3/3] lavc/mediacodecdec: fix size variable shadowing in ff_mediacodec_dec_decode

Message ID 20161006123157.8434-3-matthieu.bouron@gmail.com
State Accepted
Commit 09191516511840edb0356b0f13fc149f78f8a09e
Headers show

Commit Message

Matthieu Bouron Oct. 6, 2016, 12:31 p.m. UTC 
From: Matthieu Bouron <matthieu.bouron@stupeflix.com>

Fixes incompatible pointer type warning on 64-bit systems.
---
 libavcodec/mediacodecdec.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Michael Niedermayer Oct. 6, 2016, 1:59 p.m. UTC | #1 
On Thu, Oct 06, 2016 at 02:31:57PM +0200, Matthieu Bouron wrote:
> From: Matthieu Bouron <matthieu.bouron@stupeflix.com>
> 
> Fixes incompatible pointer type warning on 64-bit systems.
> ---
>  libavcodec/mediacodecdec.c | 1 -
>  1 file changed, 1 deletion(-)

LGTM

thx

[...]
Matthieu Bouron Oct. 6, 2016, 2:51 p.m. UTC | #2 
On Thu, Oct 06, 2016 at 03:59:31PM +0200, Michael Niedermayer wrote:
> On Thu, Oct 06, 2016 at 02:31:57PM +0200, Matthieu Bouron wrote:
> > From: Matthieu Bouron <matthieu.bouron@stupeflix.com>
> > 
> > Fixes incompatible pointer type warning on 64-bit systems.
> > ---
> >  libavcodec/mediacodecdec.c | 1 -
> >  1 file changed, 1 deletion(-)
> 
> LGTM

Pushed. Thanks.

Matthieu

[...]
diff mbox

Patch

diff --git a/libavcodec/mediacodecdec.c b/libavcodec/mediacodecdec.c
index 3d519ca..126de17 100644
--- a/libavcodec/mediacodecdec.c
+++ b/libavcodec/mediacodecdec.c
@@ -561,7 +561,6 @@  int ff_mediacodec_dec_decode(AVCodecContext *avctx, MediaCodecDecContext *s,
     }
 
     while (offset < pkt->size || (need_draining && !s->draining)) {
-        int size;
 
         index = ff_AMediaCodec_dequeueInputBuffer(codec, input_dequeue_timeout_us);
         if (ff_AMediaCodec_infoTryAgainLater(codec, index)) {