From 7f5dce4116bf4e62e0cd3ef16730157e17625e1c Mon Sep 17 00:00:00 2001 From: Marcin Bukat Date: Thu, 16 Jan 2014 08:12:29 +0100 Subject: [PATCH] ipods: fix DEBUG builds iap-core.h defines LOGF_ENABLE unconditionally but hexstring() helper function was not compiled if logf support was not selected in configure. The regular builds worked by accident because logf() lines where discarded by preprocessor. Change-Id: Id76af478e7c760d81168ebf9bf3eb973a679987c --- apps/iap/iap-core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/apps/iap/iap-core.c b/apps/iap/iap-core.c index ddcb22853a..7226a908c5 100644 --- a/apps/iap/iap-core.c +++ b/apps/iap/iap-core.c @@ -511,7 +511,7 @@ void iap_send_tx(void) } *(iap_txnext) = 0x100 - (chksum & 0xFF); -#ifdef LOGF_ENABLE +#if defined(LOGF_ENABLE) && defined(ROCKBOX_HAS_LOGF) logf("T: %s", hexstring(txstart+3, (iap_txnext - txstart)-3)); #endif for (i=0; i <= (iap_txnext - txstart); i++) @@ -1253,7 +1253,7 @@ void iap_handlepkt(void) /* handle command by mode */ length = get_u16(iap_rxstart); -#ifdef LOGF_ENABLE +#if defined(LOGF_ENABLE) && defined(ROCKBOX_HAS_LOGF) logf("R: %s", hexstring(iap_rxstart+2, (length))); #endif