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)
>> @@ -33,13 +33,10 @@
>> #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.

Best regards.
Received on 2010-09-20