[FFmpeg-devel] [PATCH] extract bit rate calculation into separate function

Robert Krüger krueger
Fri Nov 13 23:42:30 CET 2009

On 13.11.2009, at 19:02, Michael Niedermayer wrote:

> On Fri, Nov 13, 2009 at 08:59:16AM +0100, Robert Kr?ger wrote:
>> On 12.11.2009, at 19:29, Michael Niedermayer wrote:
>>> On Thu, Nov 12, 2009 at 09:13:58AM +0100, Robert Kr?ger wrote:
>>>> Hi,
>>>> On 12.11.2009, at 01:42, Stefano Sabatini wrote:
>>>>> On date Wednesday 2009-11-11 12:41:33 +0100, Robert Kr?ger  
>>>>> encoded:
>>> [...]
>>>>>> +
>>>> hmm, I tried not to change the behaviour of the code I extracted  
>>>> as I
>>>> only
>>>> wanted to refactor and what you suggest would change the  
>>>> behaviour for
>>>> it would
>>>> no longer return 0 but return ctx->bit_rate. If that makes sense, I
>>>> cannot
>>>> judge. I'll change it accordingly, if you say it does.
>>> you could send 2 pathes
>>> first to move the code, second to simplify it
>> OK, then consider the first one submitted with my most recent email.
> your most recent mail adds code duplication, the previous patches are
Hmmm, the code duplication must refer to the fact that my most recent  
patch only adds the new function without removing the code from the  
function where I extracted it. That's how I understood Stefano I  
should do it, i.e. send the one patch with the new function, have it  
accepted, then send the one eliminating the duplicate code, which I  
thought I just did. Please tell me what I should do to do it right so  
that someone who can make that call will accept it.



More information about the ffmpeg-devel mailing list