[FFmpeg-devel] [PATCH] updated LGPL AC-3 decoder
Justin Ruggles
justinruggles
Sun Jun 3 20:46:20 CEST 2007
Michael Niedermayer wrote:
> Hi
>
> On Mon, May 07, 2007 at 08:50:00PM -0400, Justin Ruggles wrote:
>
>>Michael Niedermayer wrote:
>>
>>[...]
>>>and patch looks ok
>>
>>great. now...
>>
>>Should I apply it all at once or try to go back to the original SoC code
>>and do incremental changes? I looked back at the original, and my best
>>guess is that I changed about 75% of the code. I could do it either way
>>really.
>
>
> incremental is better if its easy, but if you use some script to
> automatically check all the revissions in then test it on a dummy
> repository first and ensure it can deal with errors
> the precommit check script might reject changes due to whitespace/tab issues
> or there might be a conflicting commit in the middle (extreemly unlikely i
> know, but the script should stop if something unexpected happens ...)
I don't think I have the scriping skills to create such a thing. I also
don't really have a repo to test with. I noticed others are working on
last year's AAC decoder. Are there plans to merge that into FFmpeg with
history? If so, is there already a script out there to do this?
If others are willing to help with getting the AC-3 decoder from the SoC
branch to FFmpeg while preserving the history, I am willing to apply
incremental changes to the SoC code to bring it inline with the current
patch. Otherwise, I think I should just apply it as-is so I don't screw
anything up.
-Justin
More information about the ffmpeg-devel
mailing list