On Fri, Mar 27, 2020 at 8:30 PM Carl Eugen Hoyos <ceffm...@gmail.com> wrote: > > Am Fr., 27. März 2020 um 15:58 Uhr schrieb Gautam Ramakrishnan > <gautamr...@gmail.com>: > > > 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? > > Not sure what the right solution is but there should be no (new) > compilation warning. > > If you have split a function, then this should be a separate patch > that makes the reviews easier. I'll make a patch series and send a v2 > (First patch: Splitting a function without functional improvement, > second patch: Functional improvement) > > Please find out what top-posting means and avoid it here. > Got it > 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".
-- ------------- Gautam | _______________________________________________ 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".
