Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-04-14 Thread Michael Niedermayer
gt; Sent: Monday, 6 April, 2015 7:25:16 AM > > Subject: Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression > > (drc_scale) is now 0 by default > > > > > > > > has someone contacted justin ? > > > > > > > > > >

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-04-14 Thread Kieran Kunhya
ay, 6 April, 2015 7:25:16 AM >> Subject: Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression >> (drc_scale) is now 0 by default >> >> > >> > has someone contacted justin ? >> > >> > >> >> I did now. >> >> S

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-04-14 Thread Wiebe Cazemier
- Original Message - > From: "Wiebe Cazemier" > To: "Michael Niedermayer" > Cc: "FFmpeg development discussions and patches" , > "madshi" > Sent: Monday, 6 April, 2015 7:25:16 AM > Subject: Re: [FFmpeg-devel] [PATCH] Dolby Di

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-04-05 Thread Wiebe Cazemier
- Original Message - > From: "Michael Niedermayer" > To: "FFmpeg development discussions and patches" > Cc: "Wiebe Cazemier" , "madshi" > Sent: Monday, 6 April, 2015 3:20:46 AM > Subject: Re: [FFmpeg-devel] [PATCH] Dolby Digit

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-04-05 Thread Michael Niedermayer
y, 30 March, 2015 10:47:49 AM > > > Subject: Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range > > > compression (drc_scale) is now 0 by default > > > >> > > > >> It is not an option and I quote ETSI 102 366: > > > >> > > &

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-03-30 Thread Michael Niedermayer
On Mon, Mar 30, 2015 at 11:37:10AM +0200, Wiebe Cazemier wrote: > - Original Message - > > From: "Kieran Kunhya" > > To: "Wiebe Cazemier" > > Sent: Monday, 30 March, 2015 10:47:49 AM > > Subject: Re: [FFmpeg-devel] [PATCH] Dolby Digital dyna

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-03-30 Thread Andy Furniss
Michael Niedermayer wrote: Apologies for the slightly out of place response. I've only just subscribed after reading the rest of this thread via the archive. I'm with KieranK the exception should be full range for those that can stand it. Maybe I am not best placed to comment as I mixdown to 2c

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-03-30 Thread Michael Niedermayer
On Sun, Mar 29, 2015 at 07:01:08PM +0200, Wiebe Cazemier wrote: > Signed-off-by: Wiebe Cazemier > --- > Changelog | 1 + > doc/decoders.texi | 2 +- > libavcodec/ac3dec_fixed.c | 2 +- > libavcodec/ac3dec_float.c | 2 +- > libavutil/version.h | 2 +- > 5 files change

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-03-30 Thread Michael Niedermayer
On Mon, Mar 30, 2015 at 11:28:29AM +0100, Kieran Kunhya wrote: > On 30 March 2015 at 11:24, madshi wrote: > > 2015-03-30 12:10 GMT+02:00 Kieran Kunhya : > >> We should have DRC enabled in AAC and other codecs > >> if it's considered a normative part of the spec. > > > > So basically you do what th

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-03-30 Thread Nicolas George
Le decadi 10 germinal, an CCXXIII, madshi a écrit : > What do you think would happen if we started a vote like the following: > > 1) Should ffmpeg output reduced (range compressed) audio quality by > default, because the spec says so? > 2) Should ffmpeg output full (range) quality by default, unle

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-03-30 Thread madshi
2015-03-30 12:28 GMT+02:00 Kieran Kunhya : > Yes, and we let power users change things from the API - > I don't see why that's so hard. It's not hard. The only problem is that there are many many applications out there using ffmpeg, and many (most?) of them are not aware of drc_scale. So they deco

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-03-30 Thread Kieran Kunhya
On 30 March 2015 at 11:24, madshi wrote: > 2015-03-30 12:10 GMT+02:00 Kieran Kunhya : >> We should have DRC enabled in AAC and other codecs >> if it's considered a normative part of the spec. > > So basically you do what the spec says regardless of whether it makes sense > or not? Yes, and we let

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-03-30 Thread madshi
2015-03-30 12:10 GMT+02:00 Kieran Kunhya : > We should have DRC enabled in AAC and other codecs > if it's considered a normative part of the spec. So basically you do what the spec says regardless of whether it makes sense or not? Best regards, Mathias. ___

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-03-30 Thread Kieran Kunhya
> And you didn't answer the question I asked before: Why would only AC3 have > range compression on by default? Why not any other codec? What is the sense > in that?? Are you saying that laptop speakers are not able to play full > range AC3 tracks, but that they *are* able to play full range TrueHD

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-03-30 Thread madshi
> From: "Kieran Kunhya" > To: "Wiebe Cazemier" > Sent: Monday, 30 March, 2015 3:43:42 AM > Subject: Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default > It is not an option and I quote ETSI 102 366: > "Th

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-03-30 Thread Wiebe Cazemier
- Original Message - > From: "Kieran Kunhya" > To: "Wiebe Cazemier" > Sent: Monday, 30 March, 2015 10:47:49 AM > Subject: Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression > (drc_scale) is now 0 by default > >> >

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-03-30 Thread Wiebe Cazemier
- Original Message - > From: "Kieran Kunhya" > To: "Wiebe Cazemier" > Sent: Monday, 30 March, 2015 3:43:42 AM > Subject: Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression > (drc_scale) is now 0 by default > > > It's

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-03-29 Thread Wiebe Cazemier
- Original Message - > From: "Kieran Kunhya" > To: "Wiebe Cazemier" > Cc: "Kieran Kunhya" > Sent: Sunday, 29 March, 2015 10:03:05 PM > Subject: Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression > (drc_scale) is now 0 by

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-03-29 Thread madshi
2015-03-29 20:06 GMT+02:00 Kieran Kunhya : > > On 29 March 2015 at 18:01, Wiebe Cazemier wrote: > > Signed-off-by: Wiebe Cazemier > Why can't you just set this in your application? > Not everyone is listening on a full setup where they can hear the dynamic range. There are 2 possible situations:

Re: [FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-03-29 Thread Kieran Kunhya
On 29 March 2015 at 18:01, Wiebe Cazemier wrote: > Signed-off-by: Wiebe Cazemier Why can't you just set this in your application? Not everyone is listening on a full setup where they can hear the dynamic range. Kieran ___ ffmpeg-devel mailing list ffm

[FFmpeg-devel] [PATCH] Dolby Digital dynamic range compression (drc_scale) is now 0 by default

2015-03-29 Thread Wiebe Cazemier
Signed-off-by: Wiebe Cazemier --- Changelog | 1 + doc/decoders.texi | 2 +- libavcodec/ac3dec_fixed.c | 2 +- libavcodec/ac3dec_float.c | 2 +- libavutil/version.h | 2 +- 5 files changed, 5 insertions(+), 4 deletions(-) diff --git a/Changelog b/Changelog index 109