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

Matthieu Bouron matthieu.bouron at gmail.com
Mon Jul 22 10:02:15 EEST 2019


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 ?

Thanks.

-- 
Matthieu B.


More information about the ffmpeg-devel mailing list