Re: [FFmpeg-devel] [PATCH] swr: do not reject channel layouts that use channel 63

2015-10-28 Thread wm4 
On Mon, 26 Oct 2015 20:31:00 +0100
Michael Niedermayer  wrote:

> On Sun, Oct 25, 2015 at 06:31:02PM +0100, wm4 wrote:
> > Channel layouts are essentially uint64_t, and every value is valid.
> > ---
> >  libswresample/options.c | 8 
> >  1 file changed, 4 insertions(+), 4 deletions(-)  
> 
> this LGTM if no unsiged int64 support is added to avoptions

Pushed. Not very ideal, but I see no better way to do this right now
(plus libavresample also does it this way).
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] swr: do not reject channel layouts that use channel 63

2015-10-26 Thread Michael Niedermayer 
On Sun, Oct 25, 2015 at 06:31:02PM +0100, wm4 wrote:
> Channel layouts are essentially uint64_t, and every value is valid.
> ---
>  libswresample/options.c | 8 
>  1 file changed, 4 insertions(+), 4 deletions(-)

this LGTM if no unsiged int64 support is added to avoptions

[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

What does censorship reveal? It reveals fear. -- Julian Assange


signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] swr: do not reject channel layouts that use channel 63

2015-10-26 Thread wm4 
On Mon, 26 Oct 2015 20:31:00 +0100
Michael Niedermayer  wrote:

> On Sun, Oct 25, 2015 at 06:31:02PM +0100, wm4 wrote:
> > Channel layouts are essentially uint64_t, and every value is valid.
> > ---
> >  libswresample/options.c | 8 
> >  1 file changed, 4 insertions(+), 4 deletions(-)  
> 
> this LGTM if no unsiged int64 support is added to avoptions

When I wanted to give a try, I noticed that the min/max fields are
double, and that double doesn't have enough mantissa bits to cover the
full range. So adding a uint64 type for this purpose would be just as
broken.

I'm wonder whether setting min/max to -inf/inf would make sense. In a
way it'd be better than INT64_MAX (which is a lie, because after
rounding it's INT64_MAX+1), but in the end it's the same thing.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] swr: do not reject channel layouts that use channel 63

2015-10-26 Thread Derek Buitenhuis 
On 10/25/2015 5:31 PM, wm4 wrote:
> Channel layouts are essentially uint64_t, and every value is valid.
> ---
>  libswresample/options.c | 8 
>  1 file changed, 4 insertions(+), 4 deletions(-)

Probably needs a micro version bump.

- Derek
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel