Re: [FFmpeg-devel] [PATCH 2/2] libavcodec/mjpeg: remove YUVJ mentions

2017-12-12 Thread Paul B Mahol
On 12/11/17, Michael Niedermayer wrote: > On Mon, Dec 11, 2017 at 10:58:41PM +0100, Paul B Mahol wrote: >> On 12/11/17, Michael Niedermayer wrote: >> > On Mon, Dec 11, 2017 at 12:09:33PM -0500, Vittorio Giovara wrote: >> >> >* On 12/8/17, Paul B Mahol http://gmail.com>>* >> >> >> On 12/8/17, Vitt

Re: [FFmpeg-devel] [PATCH 2/2] libavcodec/mjpeg: remove YUVJ mentions

2017-12-11 Thread Michael Niedermayer
On Mon, Dec 11, 2017 at 10:58:41PM +0100, Paul B Mahol wrote: > On 12/11/17, Michael Niedermayer wrote: > > On Mon, Dec 11, 2017 at 12:09:33PM -0500, Vittorio Giovara wrote: > >> >* On 12/8/17, Paul B Mahol http://gmail.com>>* > >> >> On 12/8/17, Vittorio Giovara >> >>

Re: [FFmpeg-devel] [PATCH 2/2] libavcodec/mjpeg: remove YUVJ mentions

2017-12-11 Thread Paul B Mahol
On 12/11/17, Michael Niedermayer wrote: > On Mon, Dec 11, 2017 at 12:09:33PM -0500, Vittorio Giovara wrote: >> >* On 12/8/17, Paul B Mahol http://gmail.com>>* >> >> On 12/8/17, Vittorio Giovara > >> > wrote: >> >*> If we were to break this feature,

Re: [FFmpeg-devel] [PATCH 2/2] libavcodec/mjpeg: remove YUVJ mentions

2017-12-11 Thread Michael Niedermayer
On Mon, Dec 11, 2017 at 12:09:33PM -0500, Vittorio Giovara wrote: > >* On 12/8/17, Paul B Mahol http://gmail.com>>* > >> On 12/8/17, Vittorio Giovara >> > wrote: > >*> If we were to break this feature, I'd suggest going the full route of > *>*> addi

Re: [FFmpeg-devel] [PATCH 2/2] libavcodec/mjpeg: remove YUVJ mentions

2017-12-11 Thread Paul B Mahol
On 12/11/17, Vittorio Giovara wrote: >>* On 12/8/17, Paul B Mahol http://gmail.com>>* >>> On 12/8/17, Vittorio Giovara >> > wrote: >>*> If we were to break this feature, I'd suggest going the full route of > *>*> adding a PixelFormaton and work on a

Re: [FFmpeg-devel] [PATCH 2/2] libavcodec/mjpeg: remove YUVJ mentions

2017-12-11 Thread Vittorio Giovara
>* On 12/8/17, Paul B Mahol http://gmail.com>>* >> On 12/8/17, Vittorio Giovara > > wrote: >*> If we were to break this feature, I'd suggest going the full route of *>*> adding a PixelFormaton and work on a sws alternative (one is allowed to *>*> dre

Re: [FFmpeg-devel] [PATCH 2/2] libavcodec/mjpeg: remove YUVJ mentions

2017-12-08 Thread Michael Niedermayer
On Fri, Dec 08, 2017 at 12:12:58PM +0100, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavcodec/mjpegdec.c| 18 +- > libavcodec/mjpegenc.c| 6 -- > libavcodec/mjpegenc_common.c | 5 + > tests/fate/vcodec.mak| 21 ++--

Re: [FFmpeg-devel] [PATCH 2/2] libavcodec/mjpeg: remove YUVJ mentions

2017-12-08 Thread Paul B Mahol
On 12/8/17, Paul B Mahol wrote: > On 12/8/17, Vittorio Giovara wrote: >>> On 12/8/17, Paul B Mahole *>> >* wrote: >>>*> This will basically break everyone encoding mjpeg right now, since it >> *>*> suddenly only accepts different formats without an

Re: [FFmpeg-devel] [PATCH 2/2] libavcodec/mjpeg: remove YUVJ mentions

2017-12-08 Thread Paul B Mahol
On 12/8/17, Vittorio Giovara wrote: >> On 12/8/17, Paul B Mahole *> >* wrote: >>*> This will basically break everyone encoding mjpeg right now, since it > *>*> suddenly only accepts different formats without any common-ground > *>*> before/after. >

Re: [FFmpeg-devel] [PATCH 2/2] libavcodec/mjpeg: remove YUVJ mentions

2017-12-08 Thread Vittorio Giovara
> On 12/8/17, Paul B Mahole * >* wrote: >*> This will basically break everyone encoding mjpeg right now, since it *>*> suddenly only accepts different formats without any common-ground *>*> before/after. *>*> Furthermore, there is no replacement for

[FFmpeg-devel] [PATCH 2/2] libavcodec/mjpeg: remove YUVJ mentions

2017-12-08 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavcodec/mjpegdec.c| 18 +- libavcodec/mjpegenc.c| 6 -- libavcodec/mjpegenc_common.c | 5 + tests/fate/vcodec.mak| 21 ++--- 4 files changed, 28 insertions(+), 22 deletions(-) diff --git a/libavc

Re: [FFmpeg-devel] [PATCH 2/2] libavcodec/mjpeg: remove YUVJ mentions

2017-12-08 Thread Paul B Mahol
On 12/8/17, Hendrik Leppkes wrote: > On Fri, Dec 8, 2017 at 10:46 AM, Paul B Mahol wrote: >> Signed-off-by: Paul B Mahol >> --- >> libavcodec/mjpegdec.c| 18 +- >> libavcodec/mjpegenc.c| 4 ++-- >> libavcodec/mjpegenc_common.c | 5 + >> tests/fate/vcodec.ma

Re: [FFmpeg-devel] [PATCH 2/2] libavcodec/mjpeg: remove YUVJ mentions

2017-12-08 Thread Hendrik Leppkes
On Fri, Dec 8, 2017 at 10:46 AM, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavcodec/mjpegdec.c| 18 +- > libavcodec/mjpegenc.c| 4 ++-- > libavcodec/mjpegenc_common.c | 5 + > tests/fate/vcodec.mak| 21 ++--- > 4 fil

[FFmpeg-devel] [PATCH 2/2] libavcodec/mjpeg: remove YUVJ mentions

2017-12-08 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavcodec/mjpegdec.c| 18 +- libavcodec/mjpegenc.c| 4 ++-- libavcodec/mjpegenc_common.c | 5 + tests/fate/vcodec.mak| 21 ++--- 4 files changed, 26 insertions(+), 22 deletions(-) diff --git a/libavcod