|
Rockbox mail archiveSubject: Re: [RaaA] Weekly status reportRe: [RaaA] Weekly status report
From: Thomas Martitz <thomas.martitz_at_student.htw-berlin.de>
Date: Tue, 01 Jun 2010 19:08:02 +0200 Hello folks, this week I started (and mostly finished) the conversion of #ifdef SIMULATOR to #if (CONFIG_PLATFORM & PLATFORM_HOSTED) (and the like). I was quite surprised how little individual #ifdefs I actually converted. In fact, I could ignore most because they are not relevant for RaaA (on a related note, some of them are really not understandable and/or seem stupid and should be fixed up, refer to the "Ideas for hacking" item on the DevConEuro2010 page). I considered them not relevant if they met one of the following points: * when used in conjuction with a define RaaA will most certainly not define, e.g. CONFIG_CODEC != SWCODEC, HAVE_RECORDING, SANSA_E200, !USB_NONE, HAVE_LCD_CHARCELL * enables debug-only code parts * when it's clearly simulator specific (for another reason than point 1) I want to emphasize that I considered each #ifdef separately and did no sed action. It's possible that I overlooked some relevant #ifdef. However, I think I am more successful in spotting them when I know begin to create an application target without defining SIMULATOR. And this is my plan for the next week or 2. In case you feel my above points are not sufficient and/or I ignored too much, please drop a comment. I'll upload a diff to FS#11234 and commit to my git repo. Best regards. Received on 2010-06-01 Page template was last modified "Tue Sep 7 00:00:02 2021" The Rockbox Crew -- Privacy Policy |