Re: [FFmpeg-devel] [PATCH] Clean up support for openjpeg encoder

2017-03-06 Thread Aaron Boxer
Sorry, another change: switched irreversible default to 0, which matches the OpenJPEG default (i.e. lossless is the default) On Mon, Mar 6, 2017 at 11:10 AM, Aaron Boxer wrote: > One last change: numresolution default setting changed to 6, to match > OpenJPEG default. > > On Mon, Mar 6, 2017 at

Re: [FFmpeg-devel] [PATCH] Clean up support for openjpeg encoder

2017-03-06 Thread Aaron Boxer
One last change: numresolution default setting changed to 6, to match OpenJPEG default. On Mon, Mar 6, 2017 at 10:49 AM, Aaron Boxer wrote: > >> >> >> This was vertically algined. Now its not any more. >> >> >> > >> > >> > How is this? Feels like doing ASCII artwork to format this. >> >> The irr

Re: [FFmpeg-devel] [PATCH] Clean up support for openjpeg encoder

2017-03-06 Thread Aaron Boxer
> > >> > >> This was vertically algined. Now its not any more. > >> > > > > > > How is this? Feels like doing ASCII artwork to format this. > > The irreversible thing is not used at all, its just blank option. > Yes, quite right. Here is an updated patch. > ___

Re: [FFmpeg-devel] [PATCH] Clean up support for openjpeg encoder

2017-03-06 Thread Paul B Mahol
On 3/6/17, Aaron Boxer wrote: > On Mon, Mar 6, 2017 at 10:04 AM, Paul B Mahol wrote: > >> On 3/6/17, Aaron Boxer wrote: >> > I limited encoder to single quality layer, since there is currently no >> way >> > of specifying distortion for multiple layers. If there is any interest >> > in >> > this

Re: [FFmpeg-devel] [PATCH] Clean up support for openjpeg encoder

2017-03-06 Thread Aaron Boxer
On Mon, Mar 6, 2017 at 10:04 AM, Paul B Mahol wrote: > On 3/6/17, Aaron Boxer wrote: > > I limited encoder to single quality layer, since there is currently no > way > > of specifying distortion for multiple layers. If there is any interest in > > this, > > I can add support. Someone just needs

Re: [FFmpeg-devel] [PATCH] Clean up support for openjpeg encoder

2017-03-06 Thread Paul B Mahol
On 3/6/17, Aaron Boxer wrote: > I limited encoder to single quality layer, since there is currently no way > of specifying distortion for multiple layers. If there is any interest in > this, > I can add support. Someone just needs to tell me how to create an > AVOption which takes a comma separate