[FFmpeg-soc] [soc]: r1965 - aac/aac.c

Diego Biurrun diego at biurrun.de
Wed Mar 5 16:05:54 CET 2008


On Wed, Mar 05, 2008 at 02:01:27PM +0100, Andreas Öman wrote:
> Aurelien Jacobs wrote:
> > andoma wrote:
> > 
> >> Log:
> >> Move LTP_ROUND() close to where it's used. 
> >>
> >> Part of LTP conditional compilation.
> >>
> >> [...]
> >>  
> >> +
> >> +/**
> >> + * @todo: Replace this with float_to_int16()
> >> + */
> >> +static inline int16_t LTP_ROUND(float x) {
> > 
> > Not related to this commit, but all uppercase function names
> > are ugly.
> 
> Agree

Fix it :)  I'm sure it will not pass review with all uppercase function
names...

Diego



More information about the FFmpeg-soc mailing list