[FFmpeg-devel] [PATCH] avcodec/jpeg2000dec Support for PPT marker

Gautam Ramakrishnan gautamramk at gmail.com
Fri Mar 27 17:06:01 EET 2020


On Fri, Mar 27, 2020 at 8:30 PM Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
>
> Am Fr., 27. März 2020 um 15:58 Uhr schrieb Gautam Ramakrishnan
> <gautamramk at 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 at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".



-- 
-------------
Gautam |


More information about the ffmpeg-devel mailing list