dev builds
themes manual
device status forums
mailing lists
IRC bugs
dev guide

Rockbox mail archive

Subject: Why not empty defines?

Why not empty defines?

From: Burelli Luca <>
Date: 2005-11-17

Hello devs,
         I've just seen in one of the last commits something like

> cpu_boost(false);
> #endif

Question: why don't define cpu_boost to be an empty macro on targets that
don't support cpu frequencies? That would make the code much more
readable, IMHO. (This is just an example, there are several one-line,
binary #ifdefs that could be swept away).

Of course, in the same example, this doesn't mean that
HAVE_ADJUSTABLE_CPU_FREQ should be removed, but you could cut out all
these one-liners... just my $0.02!

Happy coding and thanks for your work!

Luca Burelli aka pillo

"I have seen things you people would not believe. Attack ships on fire off
the shoulder of Orion.  C-beams glittering in the dark near the Tannhäuser
Gate.  But... all those moments will be lost in time, like tears in rain."
                                      (Roy Batty, from Blade Runner, 1982)
ASCII ribbon campaign ( )
  - against HTML email  X
              & vCards / \
Received on Thu Nov 17 21:39:55 2005

Page was last modified "Jan 10 2012" The Rockbox Crew