You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by xi...@apache.org on 2020/11/27 08:43:40 UTC

[incubator-nuttx-apps] 02/05: system/nxplayer/nxplayer.c: Don't assume debug macro expansion

This is an automated email from the ASF dual-hosted git repository.

xiaoxiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-nuttx-apps.git

commit db07ae43eeae7a5b467b562fc94cb184f218c0de
Author: YAMAMOTO Takashi <ya...@midokura.com>
AuthorDate: Fri Nov 27 14:11:17 2020 +0900

    system/nxplayer/nxplayer.c: Don't assume debug macro expansion
---
 system/nxplayer/nxplayer.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/system/nxplayer/nxplayer.c b/system/nxplayer/nxplayer.c
index bb178f3..0ab7fe9 100644
--- a/system/nxplayer/nxplayer.c
+++ b/system/nxplayer/nxplayer.c
@@ -1082,7 +1082,9 @@ static void *nxplayer_playthread(pthread_addr_t pvarg)
 
             /* Send a stop message to the device */
 
+#ifdef CONFIG_DEBUG_FEATURES
             audinfo("Stopping! outstanding=%d\n", outstanding);
+#endif
 
 #ifdef CONFIG_AUDIO_MULTI_SESSION
             ioctl(pplayer->dev_fd, AUDIOIOC_STOP,
@@ -1101,7 +1103,9 @@ static void *nxplayer_playthread(pthread_addr_t pvarg)
           /* Message indicating the playback is complete */
 
           case AUDIO_MSG_COMPLETE:
+#ifdef CONFIG_DEBUG_FEATURES
             audinfo("Play complete.  outstanding=%d\n", outstanding);
+#endif
             DEBUGASSERT(outstanding == 0);
             running = false;
             break;