Re: [libav-devel] [PATCH] [post-bump] pixfmt: Always assume unspecified range as limited

2016-12-20 Thread Vittorio Giovara
On Tue, Dec 20, 2016 at 5:54 PM, Hendrik Leppkes wrote: > On Tue, Dec 20, 2016 at 5:48 PM, Vittorio Giovara >> >> No, as I said I can compromise in simply moving "undefined" state to >> value 2, so that if there is a remote case where this is useful it can >> be still

Re: [libav-devel] [PATCH] [post-bump] pixfmt: Always assume unspecified range as limited

2016-12-20 Thread Hendrik Leppkes
On Tue, Dec 20, 2016 at 5:48 PM, Vittorio Giovara wrote: > On Tue, Dec 20, 2016 at 5:14 PM, Hendrik Leppkes wrote: >> On Tue, Dec 20, 2016 at 4:59 PM, Vittorio Giovara >> wrote: >>> On Tue, Dec 20, 2016 at 4:10 PM, wm4

Re: [libav-devel] [PATCH] [post-bump] pixfmt: Always assume unspecified range as limited

2016-12-20 Thread Vittorio Giovara
On Tue, Dec 20, 2016 at 5:14 PM, Hendrik Leppkes wrote: > On Tue, Dec 20, 2016 at 4:59 PM, Vittorio Giovara > wrote: >> On Tue, Dec 20, 2016 at 4:10 PM, wm4 wrote: >>> On Tue, 20 Dec 2016 12:29:20 +0100 >>> Vittorio Giovara

Re: [libav-devel] [PATCH] [post-bump] pixfmt: Always assume unspecified range as limited

2016-12-20 Thread Hendrik Leppkes
On Tue, Dec 20, 2016 at 4:59 PM, Vittorio Giovara wrote: > On Tue, Dec 20, 2016 at 4:10 PM, wm4 wrote: >> On Tue, 20 Dec 2016 12:29:20 +0100 >> Vittorio Giovara wrote: >> >>> >>> Also h264/hevc are not the only video

Re: [libav-devel] [PATCH] [post-bump] pixfmt: Always assume unspecified range as limited

2016-12-20 Thread Vittorio Giovara
On Tue, Dec 20, 2016 at 4:10 PM, wm4 wrote: > On Tue, 20 Dec 2016 12:29:20 +0100 > Vittorio Giovara wrote: > >> >>> Also h264/hevc are not the only video codecs on the planet. Implicit >> >>> defaults in the data type is the wrong way to go

Re: [libav-devel] [PATCH] [post-bump] pixfmt: Always assume unspecified range as limited

2016-12-20 Thread Vittorio Giovara
On Tue, Dec 20, 2016 at 3:09 PM, Anton Khirnov wrote: > Quoting Vittorio Giovara (2016-12-20 12:29:20) >> mm would you be okay with moving unspecified to 2 so that at least the >> most common usecase can be mapped to a bool? > > I think the API should be designed in such a way

Re: [libav-devel] [PATCH] [post-bump] pixfmt: Always assume unspecified range as limited

2016-12-20 Thread wm4
On Tue, 20 Dec 2016 12:29:20 +0100 Vittorio Giovara wrote: > >>> Also h264/hevc are not the only video codecs on the planet. Implicit > >>> defaults in the data type is the wrong way to go about this, it takes > >>> away any kind of flexibility for no real gain. >

Re: [libav-devel] [PATCH] [post-bump] pixfmt: Always assume unspecified range as limited

2016-12-20 Thread Anton Khirnov
Quoting Vittorio Giovara (2016-12-20 12:29:20) > On Tue, Dec 20, 2016 at 12:22 PM, Hendrik Leppkes wrote: > > On Tue, Dec 20, 2016 at 12:18 PM, Vittorio Giovara > > wrote: > >> On Tue, Dec 20, 2016 at 12:12 PM, Hendrik Leppkes

Re: [libav-devel] [PATCH] [post-bump] pixfmt: Always assume unspecified range as limited

2016-12-20 Thread Vittorio Giovara
On Tue, Dec 20, 2016 at 12:22 PM, Hendrik Leppkes wrote: > On Tue, Dec 20, 2016 at 12:18 PM, Vittorio Giovara > wrote: >> On Tue, Dec 20, 2016 at 12:12 PM, Hendrik Leppkes >> wrote: >>> On Tue, Dec 20, 2016 at 12:07 PM,

Re: [libav-devel] [PATCH] [post-bump] pixfmt: Always assume unspecified range as limited

2016-12-20 Thread Hendrik Leppkes
On Tue, Dec 20, 2016 at 12:18 PM, Vittorio Giovara wrote: > On Tue, Dec 20, 2016 at 12:12 PM, Hendrik Leppkes wrote: >> On Tue, Dec 20, 2016 at 12:07 PM, Vittorio Giovara >> wrote: >>> On Tue, Dec 20, 2016 at 10:40 AM,

Re: [libav-devel] [PATCH] [post-bump] pixfmt: Always assume unspecified range as limited

2016-12-20 Thread Vittorio Giovara
On Tue, Dec 20, 2016 at 12:12 PM, Hendrik Leppkes wrote: > On Tue, Dec 20, 2016 at 12:07 PM, Vittorio Giovara > wrote: >> On Tue, Dec 20, 2016 at 10:40 AM, Hendrik Leppkes >> wrote: >>> On Tue, Dec 20, 2016 at 9:19 AM,

Re: [libav-devel] [PATCH] [post-bump] pixfmt: Always assume unspecified range as limited

2016-12-20 Thread Hendrik Leppkes
On Tue, Dec 20, 2016 at 12:07 PM, Vittorio Giovara wrote: > On Tue, Dec 20, 2016 at 10:40 AM, Hendrik Leppkes wrote: >> On Tue, Dec 20, 2016 at 9:19 AM, Vittorio Giovara >> wrote: >>> This is the assumption that is

Re: [libav-devel] [PATCH] [post-bump] pixfmt: Always assume unspecified range as limited

2016-12-20 Thread Vittorio Giovara
On Tue, Dec 20, 2016 at 10:40 AM, Hendrik Leppkes wrote: > On Tue, Dec 20, 2016 at 9:19 AM, Vittorio Giovara > wrote: >> This is the assumption that is made everywhere in the codebase >> and in several specifications too. The added benefit is that

Re: [libav-devel] [PATCH] [post-bump] pixfmt: Always assume unspecified range as limited

2016-12-20 Thread Anton Khirnov
Quoting Hendrik Leppkes (2016-12-20 10:40:41) > On Tue, Dec 20, 2016 at 9:19 AM, Vittorio Giovara > wrote: > > This is the assumption that is made everywhere in the codebase > > and in several specifications too. The added benefit is that any > > variable referencing

Re: [libav-devel] [PATCH] [post-bump] pixfmt: Always assume unspecified range as limited

2016-12-20 Thread Hendrik Leppkes
On Tue, Dec 20, 2016 at 9:19 AM, Vittorio Giovara wrote: > This is the assumption that is made everywhere in the codebase > and in several specifications too. The added benefit is that any > variable referencing range can now be simply considered a boolean. > >

[libav-devel] [PATCH] [post-bump] pixfmt: Always assume unspecified range as limited

2016-12-20 Thread Vittorio Giovara
This is the assumption that is made everywhere in the codebase and in several specifications too. The added benefit is that any variable referencing range can now be simply considered a boolean. Signed-off-by: Vittorio Giovara --- This is the first patch I had in mind