From 25820107c63cf55996aec21aee7384de3b5fbf70 Mon Sep 17 00:00:00 2001
From: Tristan Matthews <le.businessman@gmail.com>
Date: Sat, 28 Apr 2012 17:29:08 -0400
Subject: [PATCH] * #9832: audio: clean up logging

---
 daemon/src/audio/gaincontrol.cpp     |  2 +-
 daemon/src/audio/mainbuffer.cpp      | 18 +++++++++---------
 daemon/src/audio/recordable.cpp      |  2 +-
 daemon/src/audio/ringbuffer.cpp      |  2 +-
 daemon/src/audio/speexechocancel.cpp |  2 +-
 5 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/daemon/src/audio/gaincontrol.cpp b/daemon/src/audio/gaincontrol.cpp
index 5e0f282fd5..297b1fd0e0 100644
--- a/daemon/src/audio/gaincontrol.cpp
+++ b/daemon/src/audio/gaincontrol.cpp
@@ -23,7 +23,7 @@ GainControl::GainControl(double sr, double target) : averager_(sr, SFL_GAIN_ATTA
     , maxIncreaseStep_(exp(0.11513 * 12. * 160 / 8000)) // Computed on 12 frames (240 ms)
     , maxDecreaseStep_(exp(-0.11513 * 40. * 160 / 8000))// Computed on 40 frames (800 ms)
 {
-    DEBUG("GainControl: Target gain %f dB (%f linear)", targetLeveldB_, targetLevelLinear_);
+    DEBUG("Target gain %f dB (%f linear)", targetLeveldB_, targetLevelLinear_);
 }
 
 void GainControl::process(SFLDataFormat *buf, int samples)
diff --git a/daemon/src/audio/mainbuffer.cpp b/daemon/src/audio/mainbuffer.cpp
index 6b19f0b100..6925bf0f43 100644
--- a/daemon/src/audio/mainbuffer.cpp
+++ b/daemon/src/audio/mainbuffer.cpp
@@ -71,12 +71,12 @@ bool MainBuffer::removeCallIDSet(const std::string & set_id)
     CallIDSet* callid_set = getCallIDSet(set_id);
 
     if (!callid_set) {
-        DEBUG("removeCallIDSet error callid set %s does not exist!", set_id.c_str());
+        DEBUG("CallID set %s does not exist!", set_id.c_str());
         return false;
     }
 
     if (callIDMap_.erase(set_id) == 0) {
-        DEBUG("removeCallIDSet error while removing callid set %s!", set_id.c_str());
+        DEBUG("CallID set %s not found!", set_id.c_str());
         return false;
     }
 
@@ -96,9 +96,9 @@ void MainBuffer::removeCallIDfromSet(const std::string & set_id, const std::stri
     CallIDSet* callid_set = getCallIDSet(set_id);
 
     if (callid_set == NULL)
-        ERROR("removeCallIDfromSet error callid set %s does not exist!", set_id.c_str());
+        ERROR("CallIDSet %s does not exist!", set_id.c_str());
     else if (callid_set->erase(call_id) == 0)
-        ERROR("removeCallIDfromSet error while removing callid %s from set %s!", call_id.c_str(), set_id.c_str());
+        ERROR("Could not find callID %s in set %s!", call_id.c_str(), set_id.c_str());
 }
 
 RingBuffer* MainBuffer::getRingBuffer(const std::string & call_id)
@@ -123,11 +123,11 @@ bool MainBuffer::removeRingBuffer(const std::string & call_id)
             delete ring_buffer;
             return true;
         } else {
-            ERROR("BufferManager: Error: Fail to delete ringbuffer %s!", call_id.c_str());
+            ERROR("Fail to delete ringbuffer %s!", call_id.c_str());
             return false;
         }
     } else {
-        DEBUG("BufferManager: Error: Ringbuffer %s does not exist!", call_id.c_str());
+        DEBUG("Ringbuffer %s does not exist!", call_id.c_str());
         return true;
     }
 }
@@ -215,7 +215,7 @@ void MainBuffer::unBindHalfDuplexOut(const std::string & process_id, const std::
             removeRingBuffer(call_id);
         }
     } else {
-        DEBUG("Error: did not found ringbuffer %s", process_id.c_str());
+        DEBUG("did not found ringbuffer %s", process_id.c_str());
         removeCallIDSet(process_id);
     }
 
@@ -339,12 +339,12 @@ int MainBuffer::availForGetByID(const std::string &call_id,
                                 const std::string &reader_id)
 {
     if ((call_id != DEFAULT_ID) and (reader_id == call_id))
-        ERROR("MainBuffer: Error: RingBuffer has a readpointer on itself");
+        ERROR("RingBuffer has a readpointer on itself");
 
     RingBuffer* ringbuffer = getRingBuffer(call_id);
 
     if (ringbuffer == NULL) {
-        ERROR("MainBuffer: Error: RingBuffer does not exist");
+        ERROR("RingBuffer does not exist");
         return 0;
     } else
         return ringbuffer->AvailForGet(reader_id);
diff --git a/daemon/src/audio/recordable.cpp b/daemon/src/audio/recordable.cpp
index 7eda5bd661..6915c1e6c0 100644
--- a/daemon/src/audio/recordable.cpp
+++ b/daemon/src/audio/recordable.cpp
@@ -33,7 +33,7 @@
 
 Recordable::Recordable() : recAudio_(), recorder_(&recAudio_, Manager::instance().getMainBuffer())
 {
-    DEBUG("=================== Set recornding options: %s", Manager::instance().audioPreference.getRecordpath().c_str());
+    DEBUG("Set recording options: %s", Manager::instance().audioPreference.getRecordpath().c_str());
     recAudio_.setRecordingOption(AudioRecord::FILE_WAV, 8000, Manager::instance().audioPreference.getRecordpath());
 }
 
diff --git a/daemon/src/audio/ringbuffer.cpp b/daemon/src/audio/ringbuffer.cpp
index fe8547d06f..51491fcf6e 100644
--- a/daemon/src/audio/ringbuffer.cpp
+++ b/daemon/src/audio/ringbuffer.cpp
@@ -132,7 +132,7 @@ RingBuffer::storeReadPointer(int pointer_value, const std::string &call_id)
     if (iter != readpointer_.end())
         iter->second = pointer_value;
     else
-        DEBUG("storeReadPointer: Cannot find \"%s\" readPointer in \"%s\" ringbuffer", call_id.c_str(), buffer_id_.c_str());
+        DEBUG("Cannot find \"%s\" readPointer in \"%s\" ringbuffer", call_id.c_str(), buffer_id_.c_str());
 }
 
 
diff --git a/daemon/src/audio/speexechocancel.cpp b/daemon/src/audio/speexechocancel.cpp
index f51989ed5d..4fc2531843 100644
--- a/daemon/src/audio/speexechocancel.cpp
+++ b/daemon/src/audio/speexechocancel.cpp
@@ -48,7 +48,7 @@ SpeexEchoCancel::SpeexEchoCancel() :
     tmpMic_(),
     tmpOut_()
 {
-    DEBUG("EchoCancel: Initializing echo canceller with delay: %d, filter "
+    DEBUG("Initializing echo canceller with delay: %d, filter "
           "length: %d, frame size: %d and samplerate %d", echoDelay_,
           echoTailLength_, EC_FRAME_SIZE, SPEEX_SAMPLE_RATE);
 
-- 
GitLab