Re: [FFmpeg-devel] [PATCH 01/27] avcodec: add color_range to AVCodec struct and use it

2017-12-12 Thread Paul B Mahol
On 12/11/17, Hendrik Leppkes wrote: > On Mon, Dec 11, 2017 at 3:36 PM, James Almer wrote: >> On 12/11/2017 11:28 AM, Hendrik Leppkes wrote: >>> On Mon, Dec 11, 2017 at 2:22 PM, Paul B Mahol wrote: On 12/11/17, Hendrik Leppkes wrote: > On Mon, Dec 11, 2017 at 12:07 PM, Paul B Mahol >>>

Re: [FFmpeg-devel] [PATCH 01/27] avcodec: add color_range to AVCodec struct and use it

2017-12-11 Thread Paul B Mahol
On 12/11/17, Paul B Mahol wrote: > On 12/11/17, Hendrik Leppkes wrote: >> On Mon, Dec 11, 2017 at 3:36 PM, James Almer wrote: >>> On 12/11/2017 11:28 AM, Hendrik Leppkes wrote: On Mon, Dec 11, 2017 at 2:22 PM, Paul B Mahol wrote: > On 12/11/17, Hendrik Leppkes wrote: >> On Mon, De

Re: [FFmpeg-devel] [PATCH 01/27] avcodec: add color_range to AVCodec struct and use it

2017-12-11 Thread Vittorio Giovara
>* On Sat, Dec 9, 2017 at 10:37 AM, Paul B Mahol > wrote:* > Endless bikeshed... Being dismissive to valid points is not good for project health. > I hate how people can be so ignorant these days. Can you tone it down a notch maybe? I can't envisio

Re: [FFmpeg-devel] [PATCH 01/27] avcodec: add color_range to AVCodec struct and use it

2017-12-11 Thread Paul B Mahol
On 12/11/17, Hendrik Leppkes wrote: > On Mon, Dec 11, 2017 at 3:36 PM, James Almer wrote: >> On 12/11/2017 11:28 AM, Hendrik Leppkes wrote: >>> On Mon, Dec 11, 2017 at 2:22 PM, Paul B Mahol wrote: On 12/11/17, Hendrik Leppkes wrote: > On Mon, Dec 11, 2017 at 12:07 PM, Paul B Mahol >>>

Re: [FFmpeg-devel] [PATCH 01/27] avcodec: add color_range to AVCodec struct and use it

2017-12-11 Thread Hendrik Leppkes
On Mon, Dec 11, 2017 at 3:36 PM, James Almer wrote: > On 12/11/2017 11:28 AM, Hendrik Leppkes wrote: >> On Mon, Dec 11, 2017 at 2:22 PM, Paul B Mahol wrote: >>> On 12/11/17, Hendrik Leppkes wrote: On Mon, Dec 11, 2017 at 12:07 PM, Paul B Mahol wrote: >> >> Fine, but it's inevitable

Re: [FFmpeg-devel] [PATCH 01/27] avcodec: add color_range to AVCodec struct and use it

2017-12-11 Thread wm4
On Mon, 11 Dec 2017 11:36:52 -0300 James Almer wrote: > On 12/11/2017 11:28 AM, Hendrik Leppkes wrote: > > On Mon, Dec 11, 2017 at 2:22 PM, Paul B Mahol wrote: > >> On 12/11/17, Hendrik Leppkes wrote: > >>> On Mon, Dec 11, 2017 at 12:07 PM, Paul B Mahol wrote: > > > > Fine, but i

Re: [FFmpeg-devel] [PATCH 01/27] avcodec: add color_range to AVCodec struct and use it

2017-12-11 Thread James Almer
On 12/11/2017 11:28 AM, Hendrik Leppkes wrote: > On Mon, Dec 11, 2017 at 2:22 PM, Paul B Mahol wrote: >> On 12/11/17, Hendrik Leppkes wrote: >>> On Mon, Dec 11, 2017 at 12:07 PM, Paul B Mahol wrote: > > Fine, but it's inevitable that the encoder supports the J formats still > for a w

Re: [FFmpeg-devel] [PATCH 01/27] avcodec: add color_range to AVCodec struct and use it

2017-12-11 Thread wm4
On Mon, 11 Dec 2017 15:28:31 +0100 Hendrik Leppkes wrote: > On Mon, Dec 11, 2017 at 2:22 PM, Paul B Mahol wrote: > > On 12/11/17, Hendrik Leppkes wrote: > >> On Mon, Dec 11, 2017 at 12:07 PM, Paul B Mahol wrote: > > Fine, but it's inevitable that the encoder supports the J format

Re: [FFmpeg-devel] [PATCH 01/27] avcodec: add color_range to AVCodec struct and use it

2017-12-11 Thread Nicolas George
Paul B Mahol (2017-12-11): > Endless bikeshed... It is not bikeshedding, it is preparing for the future. You should try maintaining part of the framework, for a change. That is not as simple as porting a filter or implementing a demuxer. If the API is not well designed, it will need to be changed

Re: [FFmpeg-devel] [PATCH 01/27] avcodec: add color_range to AVCodec struct and use it

2017-12-11 Thread Hendrik Leppkes
On Mon, Dec 11, 2017 at 2:22 PM, Paul B Mahol wrote: > On 12/11/17, Hendrik Leppkes wrote: >> On Mon, Dec 11, 2017 at 12:07 PM, Paul B Mahol wrote: Fine, but it's inevitable that the encoder supports the J formats still for a while. >>> >>> >>> Why are you all dismissive about thi

Re: [FFmpeg-devel] [PATCH 01/27] avcodec: add color_range to AVCodec struct and use it

2017-12-11 Thread Tobias Rapp
On 11.12.2017 14:22, Paul B Mahol wrote: On 12/11/17, Hendrik Leppkes wrote: On Mon, Dec 11, 2017 at 12:07 PM, Paul B Mahol wrote: Fine, but it's inevitable that the encoder supports the J formats still for a while. Why are you all dismissive about this? Because we have an established

Re: [FFmpeg-devel] [PATCH 01/27] avcodec: add color_range to AVCodec struct and use it

2017-12-11 Thread wm4
On Mon, 11 Dec 2017 08:32:52 -0500 "Ronald S. Bultje" wrote: > Hi, > > On Sat, Dec 9, 2017 at 10:37 AM, Paul B Mahol wrote: > > > @@ -3376,6 +3376,7 @@ typedef struct AVCodec { > > uint8_t max_lowres; ///< maximum value for lowres > > supported by the decoder > >

Re: [FFmpeg-devel] [PATCH 01/27] avcodec: add color_range to AVCodec struct and use it

2017-12-11 Thread wm4
On Mon, 11 Dec 2017 14:22:43 +0100 Paul B Mahol wrote: > On 12/11/17, Hendrik Leppkes wrote: > > On Mon, Dec 11, 2017 at 12:07 PM, Paul B Mahol wrote: > >>> > >>> Fine, but it's inevitable that the encoder supports the J formats still > >>> for a while. > >> > >> > >> Why are you all dismis

Re: [FFmpeg-devel] [PATCH 01/27] avcodec: add color_range to AVCodec struct and use it

2017-12-11 Thread Paul B Mahol
On 12/11/17, Ronald S. Bultje wrote: > Hi, > > On Sat, Dec 9, 2017 at 10:37 AM, Paul B Mahol wrote: > >> @@ -3376,6 +3376,7 @@ typedef struct AVCodec { >> uint8_t max_lowres; ///< maximum value for lowres >> supported by the decoder >> const AVClass *priv_class;

Re: [FFmpeg-devel] [PATCH 01/27] avcodec: add color_range to AVCodec struct and use it

2017-12-11 Thread Ronald S. Bultje
Hi, On Sat, Dec 9, 2017 at 10:37 AM, Paul B Mahol wrote: > @@ -3376,6 +3376,7 @@ typedef struct AVCodec { > uint8_t max_lowres; ///< maximum value for lowres > supported by the decoder > const AVClass *priv_class; ///< AVClass for the private > context

Re: [FFmpeg-devel] [PATCH 01/27] avcodec: add color_range to AVCodec struct and use it

2017-12-11 Thread Paul B Mahol
On 12/11/17, Hendrik Leppkes wrote: > On Mon, Dec 11, 2017 at 12:07 PM, Paul B Mahol wrote: >>> >>> Fine, but it's inevitable that the encoder supports the J formats still >>> for a while. >> >> >> Why are you all dismissive about this? > > > Because we have an established way to remove things li

Re: [FFmpeg-devel] [PATCH 01/27] avcodec: add color_range to AVCodec struct and use it

2017-12-11 Thread Hendrik Leppkes
On Mon, Dec 11, 2017 at 12:07 PM, Paul B Mahol wrote: >> >> Fine, but it's inevitable that the encoder supports the J formats still >> for a while. > > > Why are you all dismissive about this? Because we have an established way to remove things like this, and that doesn't happen over night. Its

Re: [FFmpeg-devel] [PATCH 01/27] avcodec: add color_range to AVCodec struct and use it

2017-12-11 Thread wm4
On Mon, 11 Dec 2017 12:07:34 +0100 Paul B Mahol wrote: > On 12/11/17, wm4 wrote: > > On Sat, 9 Dec 2017 16:37:53 +0100 > > Paul B Mahol wrote: > > > >> Signed-off-by: Paul B Mahol > >> --- > >> libavcodec/avcodec.h | 1 + > >> libavcodec/utils.c | 2 ++ > >> 2 files changed, 3 insertions

Re: [FFmpeg-devel] [PATCH 01/27] avcodec: add color_range to AVCodec struct and use it

2017-12-11 Thread Paul B Mahol
On 12/11/17, wm4 wrote: > On Sat, 9 Dec 2017 16:37:53 +0100 > Paul B Mahol wrote: > >> Signed-off-by: Paul B Mahol >> --- >> libavcodec/avcodec.h | 1 + >> libavcodec/utils.c | 2 ++ >> 2 files changed, 3 insertions(+) >> >> diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h >> index 5

Re: [FFmpeg-devel] [PATCH 01/27] avcodec: add color_range to AVCodec struct and use it

2017-12-11 Thread wm4
On Sat, 9 Dec 2017 16:37:53 +0100 Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavcodec/avcodec.h | 1 + > libavcodec/utils.c | 2 ++ > 2 files changed, 3 insertions(+) > > diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h > index 5db6a81320..e5de4797c8 100644 > --- a/