From a9d6e0a2e6ad09b20dfb0d8f6ed565654007c94b Mon Sep 17 00:00:00 2001
From: Mike Brady <mikebrady@eircom.net>
Date: Wed, 11 Apr 2018 10:34:09 +0100
Subject: [PATCH] Remove some debug messages

---
 dacp.c   | 10 +---------
 player.c |  4 ++--
 2 files changed, 3 insertions(+), 11 deletions(-)

diff --git a/dacp.c b/dacp.c
index 8f9f8fdc..63001113 100644
--- a/dacp.c
+++ b/dacp.c
@@ -304,18 +304,13 @@ void relinquish_dacp_server_information(rtsp_conn_info *conn) {
 // the conversation number
 // Thus, we can keep the DACP port that might have previously been discovered
 void set_dacp_server_information(rtsp_conn_info *conn) {
-  debug(1, "set_dacp_server_information");
+  // debug(1, "set_dacp_server_information");
   sps_pthread_mutex_timedlock(
       &dacp_server_information_lock, 500000,
       "set_dacp_server_information couldn't get DACP server information lock in 0.5 second!.", 2);
-  debug(1, "got lock");
   dacp_server.players_connection_thread_index = conn->connection_number;
-  debug(1, "do comparison");
-  debug(1, "conn->dacp_id is \"%s\"", conn->dacp_id);
-  debug(1, "dacp_server.dacp_id is \"%s\"", dacp_server.dacp_id);
 
   if ((conn->dacp_id == NULL) || (strcmp(conn->dacp_id, dacp_server.dacp_id) != 0)) {
-    debug(1, "comparison failed");
     if (conn->dacp_id)
       strncpy(dacp_server.dacp_id, conn->dacp_id, sizeof(dacp_server.dacp_id));
     else
@@ -347,7 +342,6 @@ void set_dacp_server_information(rtsp_conn_info *conn) {
     }
     metadata_hub_modify_epilog(ch);
   } else {
-    debug(1, "comparison succeeded");
     if (dacp_server.port) {
       // debug(1, "Re-enable scanning.");
       dacp_server.scan_enable = 1;
@@ -357,14 +351,12 @@ void set_dacp_server_information(rtsp_conn_info *conn) {
       //      metadata_hub_modify_epilog(ch);
     }
   }
-  debug(1, "doing something else");
   dacp_server.active_remote_id = conn->dacp_active_remote; // even if the dacp_id remains the same,
                                                            // the active remote will change.
   debug(2, "set_dacp_server_information set active-remote id to %" PRIu32 ".",
         dacp_server.active_remote_id);
   pthread_cond_signal(&dacp_server_information_cv);
   pthread_mutex_unlock(&dacp_server_information_lock);
-  debug(1, "Done gettin' Ready");
 }
 
 void dacp_monitor_port_update_callback(char *dacp_id, uint16_t port) {
diff --git a/player.c b/player.c
index c3b74267..3552410a 100644
--- a/player.c
+++ b/player.c
@@ -1686,7 +1686,7 @@ static void *player_thread_func(void *arg) {
           config.output->play(silence, conn->max_frames_per_packet * conn->output_sample_ratio);
         } else if (frames_to_drop) {
           if (frames_to_drop > 3 * config.output_rate)
-            warn("Very large number of frames to drop: %" PRId64 ".");
+            warn("Very large number of frames to drop: %" PRId64 ".",frames_to_drop);
           debug(3, "%" PRId64 " frames to drop.", frames_to_drop);
           frames_to_drop -= inframe->length;
           if (frames_to_drop < 0)
@@ -1888,7 +1888,7 @@ static void *player_thread_func(void *arg) {
                 (!conn->player_thread_please_stop) && (config.resyncthreshold > 0.0) &&
                 (abs_sync_error > config.resyncthreshold * config.output_rate)) {
               if (abs_sync_error > 3 * config.output_rate) {
-                warn("The sync error is very large: %" PRId64 " frames.");
+                warn("Very large sync error: %" PRId64 " frames, with a delay of %" PRId64 "frames.",sync_error,delay);
               }
               sync_error_out_of_bounds++;
             } else {