[FFmpeg-devel] [RFC PATCH 3/4] libavcodec/jpeg2000: Modify cleanup

Gautam Ramakrishnan gautamramk at gmail.com
Thu Aug 20 17:32:32 EEST 2020


On Thu, Aug 20, 2020 at 3:02 PM Moritz Barsnick <barsnick at gmx.net> wrote:
>
> On Wed, Aug 19, 2020 at 17:51:01 +0530, gautamramk at gmail.com wrote:
> > From: Gautam Ramakrishnan <gautamramk at gmail.com>
> >
> > This patch makes the ff_jpeg2000_cleanup
> > function take in an extra parameter which
> > indicates whether it is called from the
> > encoder or decoder.
>
> I fail to see an explanation why, ...
>
> > -void ff_jpeg2000_cleanup(Jpeg2000Component *comp, Jpeg2000CodingStyle *codsty)
> > +void ff_jpeg2000_cleanup(Jpeg2000Component *comp, Jpeg2000CodingStyle *codsty, int isencoder)
> >  {
> >      int reslevelno, bandno, precno;
> >      for (reslevelno = 0;
> > diff --git a/libavcodec/jpeg2000.h b/libavcodec/jpeg2000.h
> > index ad58b1ae88..fee9607e86 100644
> > --- a/libavcodec/jpeg2000.h
> > +++ b/libavcodec/jpeg2000.h
>
> ... and what it does with this change. Looks like an NOP from the patch
> file alone.
It was done to provide functionality for the next patch in the series.
>
> Moritz
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".



-- 
-------------
Gautam |


More information about the ffmpeg-devel mailing list