Rockbox

Tasklist

FS#5782 - remove the need to #ifdef show_*_backdrop();

Attached to Project: Rockbox
Opened by Jonathan Gordon (jdgordon) - Wednesday, 09 August 2006, 09:20 GMT
Last edited by Jonathan Gordon (jdgordon) - Sunday, 20 May 2007, 09:01 GMT
Task Type Patches
Category Operating System/Drivers
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

this #defines show_*_backdrop() so we dont need to put it inside #ifdef HAVE_LCD_COLOR
This task depends upon

Closed by  Jonathan Gordon (jdgordon)
Sunday, 20 May 2007, 09:01 GMT
Reason for closing:  Rejected
Additional comments about closing:  macros suck
Comment by Jonathan Gordon (jdgordon) - Wednesday, 09 August 2006, 09:25 GMT
grrr... Bj�n's (c) notice is annoying :p
Comment by Matthias Mohr (aka Massa) (mmohr) - Tuesday, 22 August 2006, 10:36 GMT
In principle it works very well.

But I don't know if we really want to hide functions via #define.
If we start this, we would have to do it for every possible function/target.

If you then have a look at the sourcecode you'll see apparent functions which
are not relevant for your target...
I think it's more confusing than having #ifdef around them.

(btw, the #define methos is only possible for functions without parameters -
which makes it somehow limited... ;)

my two cents...

Loading...