From c02dc38e2f27542a78500b5a04ec787495b8f0f1 Mon Sep 17 00:00:00 2001 From: Denys Vlasenko Date: Tue, 8 Sep 2009 17:16:49 +0200 Subject: remove unused Error() methods and dbus signals. I guess it turned out Warning() works just fine. Signed-off-by: Denys Vlasenko --- lib/CommLayer/CommLayerInner.cpp | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) (limited to 'lib/CommLayer/CommLayerInner.cpp') diff --git a/lib/CommLayer/CommLayerInner.cpp b/lib/CommLayer/CommLayerInner.cpp index 5450b41..b5b8db7 100644 --- a/lib/CommLayer/CommLayerInner.cpp +++ b/lib/CommLayer/CommLayerInner.cpp @@ -44,7 +44,10 @@ void warn_client(const std::string& pMessage) const char* peer = (ki != s_mapClientID.end() ? ki->second.c_str() : NULL); pthread_mutex_unlock(&s_map_mutex); - s_pObs->Warning(pMessage, peer, key); + if (peer) + s_pObs->Warning(pMessage, peer, key); + else /* Bug: someone tries to warn_client() without set_client_name()!? */ + log("Hmm, stray %s: '%s'", __func__, pMessage.c_str()); } void update_client(const std::string& pMessage) @@ -59,5 +62,8 @@ void update_client(const std::string& pMessage) const char* peer = (ki != s_mapClientID.end() ? ki->second.c_str() : NULL); pthread_mutex_unlock(&s_map_mutex); - s_pObs->Status(pMessage, peer, key); + if (peer) + s_pObs->Status(pMessage, peer, key); + else + log("Hmm, stray %s: '%s'", __func__, pMessage.c_str()); } -- cgit