Am Fr., 27. März 2020 um 17:21 Uhr schrieb <gautamr...@gmail.com>: > > From: Gautam Ramakrishnan <gautamr...@gmail.com> > > This patch modifies the Jpeg2000Tile struct and breaks the > jpeg2000_decode_packet() function into two to decode packet > header and data separately.
The solution is also not to add functions that are unused at first. I don't know if it is good to split the patch but you mentioned that you had to split a particular function. That makes it sound as if you should - with the first patch - split the function without changing FFmpeg's behaviour, and then with a second patch change FFmpeg's behaviour by adding the new functionality. As said, I don't know if that simplifies the review in this specific case. Did you already look at the sample file I pointed out? Carl Eugen _______________________________________________ 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".