[Ffmpeg-cvslog] r8244 - in trunk: ffmpeg.c libavcodec/utils.c tests/regression.sh

[Baptiste Coudurier]

Hi

Michael Niedermayer wrote:
> Hi
> 
> On Mon, Mar 05, 2007 at 01:23:30PM +0100, Baptiste Coudurier wrote:
>> Hi
>>
>> gpoirier wrote:
>>> Author: gpoirier
>>> Date: Mon Mar  5 11:04:53 2007
>>> New Revision: 8244
>>>
>>> Modified:
>>>    trunk/ffmpeg.c
>>>    trunk/libavcodec/utils.c
>>>    trunk/tests/regression.sh
>>>
>>> Log:
>>> change management of "ab" option in ffmpeg.c, so that it works as "b" option.
>>> patch by Benoit Fouet % benoit P fouet A purplelabs P com %
>>> Original thread:
>>> Subj: [Ffmpeg-devel] [PATCH] use new way to handle "ab" option
>>> Date: 03/03/2007 10:51 PM
>>>
>>>
>>> Modified: trunk/ffmpeg.c
>>> ==============================================================================
>>> --- trunk/ffmpeg.c	(original)
>>> +++ trunk/ffmpeg.c	Mon Mar  5 11:04:53 2007
>>> @@ -137,7 +137,6 @@ static int qp_hist = 0;
>>>  
>>>  static int intra_only = 0;
>>>  static int audio_sample_rate = 44100;
>>> -static int audio_bit_rate = 64000;
>>>
>>> [...]
>>>
>>>  
>>> -        audio_enc->bit_rate = audio_bit_rate;
>>>          if (audio_qscale > QSCALE_NONE) {
>> Can we just let default audio bitrate to 64k ? It is 200k now.
> 
> ive no objections if you know of a clean solution to set it to 64 and 200
> 

If we reintroduce those 2 lines, default bitrate stays at 64k, you can
then use -ab 200k to set it to 200 or am I missing something ?

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
SMARTJOG S.A.                                    http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
Phone: +33 1 49966312