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

Ronald S. Bultje rsbultje
Mon Mar 7 17:58:28 CET 2011


Hi,

On Fri, Feb 25, 2011 at 9:46 PM, Alex Converse <alex.converse at gmail.com> wrote:
> On Fri, Feb 25, 2011 at 12:21 PM, Ronald S. Bultje <rsbultje at gmail.com> wrote:
>>
>> 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
>
> Let's say yes

Queued.

Ronald



More information about the ffmpeg-devel mailing list