[FFmpeg-devel] [PATCH] aacenc: remove the data arrays

Ronald S. Bultje rsbultje
Fri Feb 25 21:21:09 CET 2011


Hi Alex,

On Fri, Feb 18, 2011 at 5:26 AM, Alex Converse <alex.converse at gmail.com> wrote:
> On Thu, Feb 17, 2011 at 11:28 PM, Young Han Lee <cpumaker at gmail.com> wrote:
>> Hello
>>
>
> Hi
>
>> On Fri, Feb 18, 2011 at 1:32 PM, Nathan Caldwell <saintdev at gmail.com> wrote:
>>
>>> LGTM. Would like to hear what Alex has to say about it.
>>>
>
> I'm pretty neutral on this. I'm not particularly keen on prematurely
> obfuscating the code base unless there is some sort of speed up
> associated with it that could help with development.
>
> OTOH there doesn't seem to be any real harm here and it gets some crud
> out of the context.

You're going to have to make a final call here, yes or no? :-).

Ronald



More information about the ffmpeg-devel mailing list