[FFmpeg-cvslog] r15193 - in trunk/libavcodec: aac.c lpc.c lpc.h ra288.c

Diego Biurrun diego
Thu Sep 4 23:54:24 CEST 2008


On Thu, Sep 04, 2008 at 01:03:14PM +0200, vitor wrote:
> 
> Log:
> Avoid duplicating compute_lpc_coefs() function in both the RA288 and AAC decoders.
> 
> --- trunk/libavcodec/lpc.h	(original)
> +++ trunk/libavcodec/lpc.h	Thu Sep  4 13:03:14 2008
> @@ -45,4 +45,58 @@ int ff_lpc_calc_coefs(DSPContext *s,
>  
> +#ifdef LPC_USE_DOUBLE
> +#define LPC_type double
> +#else
> +#define LPC_type float
> +#endif

I think the convention is to use all-uppercase names for macros.  This
looks weird.

Diego




More information about the ffmpeg-cvslog mailing list