Rockbox.org home
release
dev builds
extras
themes manual
wiki
device status forums
mailing lists
IRC bugs
patches
dev guide



Rockbox mail archive

Subject: Re: christian: r15632 - in trunk/firmware: . export

Re: christian: r15632 - in trunk/firmware: . export

From: Linus Nielsen Feltzing <linus_at_haxx.se>
Date: Fri, 16 Nov 2007 08:01:32 +0100

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)
> _at__at_ -22,4 +22,98 _at__at_
> #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

Page template was last modified "Tue Sep 7 00:00:02 2021" The Rockbox Crew -- Privacy Policy