diff options
| author | Denys Vlasenko <vda.linux@googlemail.com> | 2009-09-08 17:16:49 +0200 |
|---|---|---|
| committer | Denys Vlasenko <vda.linux@googlemail.com> | 2009-09-08 17:16:49 +0200 |
| commit | c02dc38e2f27542a78500b5a04ec787495b8f0f1 (patch) | |
| tree | 4b568838e0585b494e2df6401365c06c79c6595f /lib/CommLayer/CommLayerInner.cpp | |
| parent | ff2627e7c597e50025ca4d91ff8168eec80f9054 (diff) | |
| download | abrt-c02dc38e2f27542a78500b5a04ec787495b8f0f1.tar.gz abrt-c02dc38e2f27542a78500b5a04ec787495b8f0f1.tar.xz abrt-c02dc38e2f27542a78500b5a04ec787495b8f0f1.zip | |
remove unused Error() methods and dbus signals.
I guess it turned out Warning() works just fine.
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'lib/CommLayer/CommLayerInner.cpp')
| -rw-r--r-- | lib/CommLayer/CommLayerInner.cpp | 10 |
1 files changed, 8 insertions, 2 deletions
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()); } |