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 <[email protected]> wrote: > > Am Fr., 27. März 2020 um 15:29 Uhr schrieb Carl Eugen Hoyos > <[email protected]>: > > > > Am Fr., 27. März 2020 um 14:47 Uhr schrieb <[email protected]>: > > > > > > From: Gautam Ramakrishnan <[email protected]> > > > > > > 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 > [email protected] > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > [email protected] with subject "unsubscribe". -- ------------- Gautam | _______________________________________________ ffmpeg-devel mailing list [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
