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: Fwd: Re: kugel: r28125 - in trunk/firmware: . export target/hosted/android

Re: Fwd: Re: kugel: r28125 - in trunk/firmware: . export target/hosted/android

From: Maurus Cuelenaere <mcuelenaere_at_gmail.com>
Date: Mon, 20 Sep 2010 22:49:41 +0200

 Op 20-09-10 21:10, Thomas Martitz schreef:
> On 20.09.2010 20:58, Maurus Cuelenaere wrote:
>> Modified: trunk/firmware/export/debug.h
>>> ===================================================================
>>> --- trunk/firmware/export/debug.h 2010-09-20 17:09:55 UTC (rev 28124)
>>> +++ trunk/firmware/export/debug.h 2010-09-20 17:38:47 UTC (rev 28125)
>>> _at__at_ -33,13 +33,10 _at__at_
>>> #ifdef __GNUC__
>>>
>>> /* */
>>> -#if defined(SIMULATOR)&& !defined(__PCTOOL__)
>>> +#if (SIMULATOR)&& !defined(__PCTOOL__) \
>>> + || ((CONFIG_PLATFORM& PLATFORM_ANDROID)&& defined(DEBUG))
>>> #define DEBUGF debugf
>>> #define LDEBUGF(...) ldebugf(__FILE__, __LINE__, __VA_ARGS__)
>> Another way of doing this could be (not sure if we have a STRINGIFY macro):
>> #define LDEBUGF(fmt, ...) DEBUGF(__FILE__ ":" STRINGIFY(__LINE__) " " fmt, \
>> __VA_ARGS__)
>>
>> That way there's no need for the separate ldebugf() function and buffer.
> That doesn't work when fmt is not a string literal. But I don't feel
> strong, I've never used LDEBUGF myself and the only occurrences are in
> code Android doesn't compile.

True. I didn't check the usage of LDEBUGF but if you say it isn't used much then there's no point in changing this.

-- 
Maurus Cuelenaere
Received on 2010-09-20

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