[svn] commit: r1377 - /trunk/src/bin/auth/main.cc

BIND 10 source code commits bind10-changes at lists.isc.org
Fri Mar 12 23:00:15 UTC 2010


Author: jinmei
Date: Fri Mar 12 23:00:15 2010
New Revision: 1377

Log:
cleanup:
  style guideline conformance
  deprecated file-static, and put everything except main() in an anonymous
  namespace.

Modified:
    trunk/src/bin/auth/main.cc

Modified: trunk/src/bin/auth/main.cc
==============================================================================
--- trunk/src/bin/auth/main.cc (original)
+++ trunk/src/bin/auth/main.cc Fri Mar 12 23:00:15 2010
@@ -31,7 +31,7 @@
 #ifdef HAVE_BOOSTLIB
 #include <boost/bind.hpp>
 #include <boost/asio.hpp>
-#endif
+#endif  // HAVE_BOOSTLIB
 
 #include <exceptions/exceptions.h>
 
@@ -53,7 +53,7 @@
 using namespace boost::asio;
 using ip::udp;
 using ip::tcp;
-#endif
+#endif  // HAVE_BOOSTLIB
 
 using namespace isc::data;
 using namespace isc::cc;
@@ -61,18 +61,15 @@
 using namespace isc::dns;
 
 namespace {
-static bool verbose_mode = false;
-}
-
-namespace {
+
+bool verbose_mode = false;
+
 const string PROGRAM = "Auth";
 const char* DNSPORT = "5300";
-}
 
 /* need global var for config/command handlers.
  * todo: turn this around, and put handlers in the authserver
  * class itself? */
-namespace {
 AuthSrv *auth_server;
 #ifdef HAVE_BOOSTLIB
 // TODO: this should be a property of AuthSrv, and AuthSrv needs
@@ -80,15 +77,14 @@
 boost::asio::io_service io_service_;
 #else
 bool running;
-#endif
-}
-
-static ElementPtr
+#endif  // HAVE_BOOSTLIB
+
+ElementPtr
 my_config_handler(ElementPtr new_config) {
     return auth_server->updateConfig(new_config);
 }
 
-static ElementPtr
+ElementPtr
 my_command_handler(const string& command, const ElementPtr args) {
     ElementPtr answer = createAnswer();
 
@@ -101,7 +97,7 @@
         io_service_.stop();
 #else
         running = false;
-#endif
+#endif  // HAVE_BOOSTLIB
     }
     
     return answer;
@@ -111,7 +107,6 @@
 //
 // Helper classes for asynchronous I/O using boost::asio
 //
-namespace {
 class TCPClient {
 public:
     TCPClient(io_service& io_service) :
@@ -171,8 +166,7 @@
         }
     }
 
-    void responseWrite(const boost::system::error_code& error)
-    {
+    void responseWrite(const boost::system::error_code& error) {
         if (!error) {
                 async_write(socket_,
                             boost::asio::buffer(response_buffer_.getData(),
@@ -184,8 +178,7 @@
         }
     }
 
-    void handleWrite(const boost::system::error_code& error)
-    {
+    void handleWrite(const boost::system::error_code& error) {
         if (!error) {
             start();            // handle next request, if any.
       } else {
@@ -204,9 +197,7 @@
     char data_[MAX_LENGTH];
 };
 
-class TCPServer
-{
-private:
+class TCPServer {
 public:
     TCPServer(io_service& io_service, int af, short port) :
         io_service_(io_service),
@@ -316,8 +307,7 @@
     ServerSet() : udp4_server(NULL), udp6_server(NULL),
                   tcp4_server(NULL), tcp6_server(NULL)
     {}
-    ~ServerSet()
-    {
+    ~ServerSet() {
         delete udp4_server;
         delete udp6_server;
         delete tcp4_server;
@@ -329,7 +319,7 @@
     TCPServer* tcp6_server;
 };
 
-static void
+void
 run_server(const char* port, const bool use_ipv4, const bool use_ipv6,
            const string& specfile)
 {
@@ -351,12 +341,10 @@
     cout << "Server started." << endl;
     io_service_.run();
 }
-}
 #else  // !HAVE_BOOSTLIB
 struct SocketSet {
     SocketSet() : ups4(-1), tps4(-1), ups6(-1), tps6(-1) {}
-    ~SocketSet()
-    {
+    ~SocketSet() {
         if (ups4 >= 0) {
             close(ups4);
         }
@@ -373,7 +361,7 @@
     int ups4, tps4, ups6, tps6;
 };
 
-static int
+int
 getUDPSocket(int af, const char* port) {
     struct addrinfo hints, *res;
 
@@ -408,7 +396,7 @@
     return (s);
 }
 
-static int
+int
 getTCPSocket(int af, const char* port) {
     struct addrinfo hints, *res;
 
@@ -447,7 +435,7 @@
     return (s);
 }
 
-static void
+void
 processMessageUDP(const int fd, Message& dns_message,
                   MessageRenderer& response_renderer)
 {
@@ -469,7 +457,7 @@
     }
 }
 
-static void
+void
 processMessageTCP(const int fd, Message& dns_message,
                   MessageRenderer& response_renderer)
 {
@@ -532,7 +520,7 @@
     close(ts);
 }
 
-static void
+void
 run_server(const char* port, const bool use_ipv4, const bool use_ipv6,
            const string& specfile)
 {
@@ -601,11 +589,12 @@
 }
 #endif // HAVE_BOOSTLIB
 
-static void
+void
 usage() {
     cerr << "Usage: b10-auth [-p port] [-4|-6]" << endl;
     exit(1);
 }
+} // end of anonymous namespace
 
 int
 main(int argc, char* argv[]) {




More information about the bind10-changes mailing list