[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