[FFmpeg-devel] [PATCH] avcodec/jpeg2000dec Support for PPT marker

[Gautam Ramakrishnan]

Hi,

I was a bit apprehensive about removing the jpeg2000_decode_packet()
function. I had basically split that function into 2 separate ones.
Should I keep it till we add more changes to the decoder or do I
remove it now itself?

On Fri, Mar 27, 2020 at 8:03 PM Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
>
> Am Fr., 27. März 2020 um 15:29 Uhr schrieb Carl Eugen Hoyos
> <ceffmpeg at gmail.com>:
> >
> > Am Fr., 27. März 2020 um 14:47 Uhr schrieb <gautamramk at gmail.com>:
> > >
> > > From: Gautam Ramakrishnan <gautamramk at gmail.com>
> > >
> > > This patch adds support for the JPEG2000 PPT marker. This patch hence,
> > > fixes #4610. The patch was tested on the samples posted along with
> > > ticket #4610.
> >
> > Your patch adds a compile-time warning here, please fix that.
> >
> > @Michael: Both samples from ticket #4610 are decoded visually correct,
> > the sample from the testsrc (p1_02.j2k) is not lossless afaict (jasper and
> > openjpeg produce different output), the samma sample is lossless but
> > since it is yuv, it will take a little time to test for bit-exactness.
>
> Not so difficult: The samma sample produces bit-exact output with the patch.
>
> Carl Eugen
> _______________________________________________
> 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 |