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".
