[FFmpeg-devel] [PATCH 4/4] avcodec/mediacodec_wrapper: remove unused local variables in ff_AMediaCodec_getCodecNameByType()

[Matthieu Bouron]

On Tue, Jul 23, 2019 at 01:09:41AM +0200, Michael Niedermayer wrote:
> On Mon, Jul 22, 2019 at 09:02:15AM +0200, Matthieu Bouron wrote:
> > On Fri, Jul 19, 2019 at 09:40:52AM +0200, Matthieu Bouron wrote:
> > > On Sun, Jul 14, 2019 at 08:17:03PM +0200, Matthieu Bouron wrote:
> > > > On Thu, Jul 04, 2019 at 03:43:48PM +0200, Matthieu Bouron wrote:
> > > > > ---
> > > > >  libavcodec/mediacodec_wrapper.c | 10 ----------
> > > > >  1 file changed, 10 deletions(-)
> > > > > 
> > > > > diff --git a/libavcodec/mediacodec_wrapper.c b/libavcodec/mediacodec_wrapper.c
> > > > > index 70e1e7cae1..5213cf640a 100644
> > > > > --- a/libavcodec/mediacodec_wrapper.c
> > > > > +++ b/libavcodec/mediacodec_wrapper.c
> > > > > @@ -392,8 +392,6 @@ char *ff_AMediaCodecList_getCodecNameByType(const char *mime, int profile, int e
> > > > >      struct JNIAMediaCodecListFields jfields = { 0 };
> > > > >      struct JNIAMediaFormatFields mediaformat_jfields = { 0 };
> > > > >  
> > > > > -    jobject format = NULL;
> > > > > -    jobject codec = NULL;
> > > > >      jobject codec_name = NULL;
> > > > >  
> > > > >      jobject info = NULL;
> > > > > @@ -571,14 +569,6 @@ done_with_info:
> > > > >      }
> > > > >  
> > > > >  done:
> > > > > -    if (format) {
> > > > > -        (*env)->DeleteLocalRef(env, format);
> > > > > -    }
> > > > > -
> > > > > -    if (codec) {
> > > > > -        (*env)->DeleteLocalRef(env, codec);
> > > > > -    }
> > > > > -
> > > > >      if (codec_name) {
> > > > >          (*env)->DeleteLocalRef(env, codec_name);
> > > > >      }
> > > > > -- 
> > > > > 2.22.0
> > > > > 
> > > > 
> > > > Ping for the patch set.
> > > 
> > > If there is no objection, I will push the patchset in a few days.
> > > 
> > 
> > Pushed to master. Is it still on time to also be pushed to the 4.2 release
> > branch ?
> 
> i found no time today for working on the release so theres still time for
> any backports of bug fixes, yes

Pushed to the release/4.2 branch. Thanks.

-- 
Matthieu B.