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

Alex Converse alex.converse
Sat Feb 26 03:46:19 CET 2011


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



More information about the ffmpeg-devel mailing list