[5107] Changes after review:

 - typos corrected
 - copyright years updated
 - added comments

diff --git a/src/bin/agent/ctrl_agent_controller.cc b/src/bin/agent/ctrl_agent_controller.cc
index 281fae856bc5ee6a32c8154a2e92b34e403603f2..04845686a33352863d7eae27bf75b282bafd1830 100644 (file)
--- a/src/bin/agent/ctrl_agent_controller.cc
+++ b/src/bin/agent/ctrl_agent_controller.cc
@@ -1,4 +1,4 @@
-// Copyright (C) 2016 Internet Systems Consortium, Inc. ("ISC")
+// Copyright (C) 2016-2017 Internet Systems Consortium, Inc. ("ISC")
 //
 // This Source Code Form is subject to the terms of the Mozilla Public
 // License, v. 2.0. If a copy of the MPL was not distributed with this

diff --git a/src/bin/agent/ctrl_agent_process.cc b/src/bin/agent/ctrl_agent_process.cc
index 4740ce1292e924311c2682475a5c4d612a93e589..651e670151d63c67ac3aad3fe642db5debabb3ab 100644 (file)
--- a/src/bin/agent/ctrl_agent_process.cc
+++ b/src/bin/agent/ctrl_agent_process.cc
@@ -18,6 +18,7 @@ using namespace isc::http;
 using namespace isc::process;
 
 // Temporarily hardcoded configuration.
+/// @todo: remove once 5134 is merged.
 namespace {
 
 const IOAddress SERVER_ADDRESS("127.0.0.1");
@@ -47,14 +48,26 @@ CtrlAgentProcess::run() {
 
     try {
 
+        // Create response creator factory first. It will be used to generate
+        // response creators. Each response creator will be used to generate
+        // answer to specific request.
         HttpResponseCreatorFactoryPtr rcf(new CtrlAgentResponseCreatorFactory());
+
+        // Create http listener. It will open up a TCP socket and be prepared
+        // to accept incoming connection.
         HttpListener http_listener(*getIoService(), SERVER_ADDRESS,
                                    SERVER_PORT, rcf, REQUEST_TIMEOUT);
+
+        // Instruct the http listener to actually open socket, install callback
+        // and start listening.
         http_listener.start();
 
+        // Ok, seems we're good to go.
         LOG_INFO(agent_logger, CTRL_AGENT_HTTP_SERVICE_STARTED)
             .arg(SERVER_ADDRESS.toText()).arg(SERVER_PORT);
 
+        // Let's process incoming data or expiring timers in a loop until
+        // shutdown condition is detected.
         while (!shouldShutdown()) {
             getIoService()->run_one();
         }

diff --git a/src/bin/agent/tests/ctrl_agent_response_creator_unittest.cc b/src/bin/agent/tests/ctrl_agent_response_creator_unittest.cc
index 8fa6c04b88c392aba0c2229fa2e30cc8b66d5efd..ca020feb4329cf67bb604b291e6bb194b55835b2 100644 (file)
--- a/src/bin/agent/tests/ctrl_agent_response_creator_unittest.cc
+++ b/src/bin/agent/tests/ctrl_agent_response_creator_unittest.cc
@@ -104,8 +104,8 @@ public:
     /// @param command_arguments Command arguments (empty).
     ///
     /// @return Returns response with a single string "bar".
-    ConstElementPtr fooCommandHandler(const std::string& command_name,
-                                      const ConstElementPtr& command_arguments) {
+    ConstElementPtr fooCommandHandler(const std::string& /*command_name*/,
+                                      const ConstElementPtr& /*command_arguments*/) {
         ElementPtr arguments = Element::createList();
         arguments->add(Element::create("bar"));
         return (createAnswer(CONTROL_RESULT_SUCCESS, arguments));
@@ -122,7 +122,7 @@ public:
 
 };
 
-// This test verifies that the created "empty" reuqest has valid type.
+// This test verifies that the created "empty" request has valid type.
 TEST_F(CtrlAgentResponseCreatorTest, createNewHttpRequest) {
     // The request must be of PostHttpRequestJson type.
     PostHttpRequestJsonPtr request_json = boost::dynamic_pointer_cast<
@@ -144,7 +144,7 @@ TEST_F(CtrlAgentResponseCreatorTest, createStockHttpResponseHighVersion) {
                       "HTTP/1.0 408 Request Timeout");
 }
 
-// Test that the server responsds with version 1.1 if request version is 1.1.
+// Test that the server responds with version 1.1 if request version is 1.1.
 TEST_F(CtrlAgentResponseCreatorTest, createStockHttpResponseCorrectVersion) {
     request_->context()->http_version_major_ = 1;
     request_->context()->http_version_minor_ = 1;

diff --git a/src/lib/http/connection.h b/src/lib/http/connection.h
index f0854358ab9b8e02beac85b9092862b2e6382d96..5ae0982c6909806c615ab59b16835839207b3b77 100644 (file)
--- a/src/lib/http/connection.h
+++ b/src/lib/http/connection.h
@@ -175,6 +175,7 @@ private:
     /// @brief Stops current connection.
     void stopThisConnection();
 
+    /// @brief returns remote address in textual form
     std::string getRemoteEndpointAddressAsText() const;
 
     /// @brief Timer used to detect Request Timeout.

diff --git a/src/lib/http/tests/response_unittests.cc b/src/lib/http/tests/response_unittests.cc
index 03b8dbb0f18ba1004b3ca7a4d151a4f33a65b048..90076ea72124e259f6a3880a7f29fb370b5d04d5 100644 (file)
--- a/src/lib/http/tests/response_unittests.cc
+++ b/src/lib/http/tests/response_unittests.cc
@@ -1,4 +1,4 @@
-// Copyright (C) 2016 Internet Systems Consortium, Inc. ("ISC")
+// Copyright (C) 2016-2017 Internet Systems Consortium, Inc. ("ISC")
 //
 // This Source Code Form is subject to the terms of the Mozilla Public
 // License, v. 2.0. If a copy of the MPL was not distributed with this