On Wed, Apr 14, 2021 at 11:21:40PM +0200, Andreas Rheinhardt wrote: > Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@outlook.com> > --- > libavcodec/jpeglsdec.c | 16 ++++++++-------- > 1 file changed, 8 insertions(+), 8 deletions(-) [...] > @@ -548,7 +548,7 @@ AVCodec ff_jpegls_decoder = { > .long_name = NULL_IF_CONFIG_SMALL("JPEG-LS"), > .type = AVMEDIA_TYPE_VIDEO, > .id = AV_CODEC_ID_JPEGLS, > - .priv_data_size = sizeof(MJpegDecodeContext), > + .priv_data_size = sizeof(JpegLSDecodeContext), > .init = ff_mjpeg_decode_init, > .close = ff_mjpeg_decode_end,
Sorry for not spotting this before it was applied but this feels problematic a mjpeg could contain jpeg-ls in fact iam pretty sure you can mix lossless and lossy coding in a single jpeg image. Not sure if there is something in jpeg-ls that would prevent it from being part of the mix. I think its easier/more robust if the same context type would be used Thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Let us carefully observe those good qualities wherein our enemies excel us and endeavor to excel them, by avoiding what is faulty, and imitating what is excellent in them. -- Plutarch
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".