mailer_at_svn.rockbox.org wrote:
> Modified: trunk/firmware/export/mas35xx.h
> ===================================================================
> --- trunk/firmware/export/mas35xx.h 2007-11-15 21:20:29 UTC (rev 15631)
> +++ trunk/firmware/export/mas35xx.h 2007-11-15 23:32:56 UTC (rev 15632)
> @@ -22,4 +22,98 @@
> #ifndef _MAS35XX_H
> #define _MAS35XX_H
>
> +#include "config.h"
> +
> +#if CONFIG_CODEC == MAS3507D
> +static const unsigned int bass_table[] =
> +{

Ehum. I'm not sure I like to have header files that declare static
constant data. The same data will be instantiated in every file that
includes this header file, and that is certainly a waste of memory.

I'ts possible that the compiler/linker can see that the data isn't used
and remove it, but I still consider it very bad practice.

Linus
Received on 2007-11-16