Delete some unused code found by -Wunused-function

remoting/jingle_glue/xmpp_signal_strategy.cc

Index: remoting/jingle_glue/xmpp_signal_strategy.cc
diff --git a/remoting/jingle_glue/xmpp_signal_strategy.cc b/remoting/jingle_glue/xmpp_signal_strategy.cc
index 9a04f7b789095968f3f28f88fdc57903d5fa9176..03adcb8135ac8b6d5331580767da07478929ea16 100644
--- a/remoting/jingle_glue/xmpp_signal_strategy.cc
+++ b/remoting/jingle_glue/xmpp_signal_strategy.cc
@@ -21,8 +21,6 @@
 #include "third_party/libjingle/source/talk/xmpp/prexmppauth.h"
 #include "third_party/libjingle/source/talk/xmpp/saslcookiemechanism.h"
 
-namespace {
-
 const char kDefaultResourceName[] = "chromoting";
 
 // Use 58 seconds keep-alive interval, in case routers terminate
@@ -34,12 +32,6 @@ const int kKeepAliveIntervalSeconds = 50;
 const size_t kReadBufferSize = 4096;
 const size_t kWriteBufferSize = 4096;
 
-void DisconnectXmppClient(buzz::XmppClient* client) {
-  client->Disconnect();
-}
-
-}  // namespace

[Wez, 2012/09/28 17:56:50]

We want these constants to be static, so we don't want to lose this namespace.

[Nico, 2012/09/28 17:59:37]

const has implicit static linkage. the namespace is not necessary.

-
 namespace remoting {
 
 XmppSignalStrategy::XmppSignalStrategy(