[#1097] Addresses review comments

    Minor cleanups

diff --git a/src/lib/config/command_mgr.cc b/src/lib/config/command_mgr.cc
index 0ff2c5d850dda224cbe4d39ba63951810311e00d..945b198c245f3b2aae12da88735bafdf9d4c62cc 100644 (file)
--- a/src/lib/config/command_mgr.cc
+++ b/src/lib/config/command_mgr.cc
@@ -490,7 +490,7 @@ public:
 
     /// @brief Returns the lock file name
     std::string getLockName() {
-       return(std::string(socket_name_ + ".lock"));
+        return (std::string(socket_name_ + ".lock"));
     }
 
     /// @brief Pointer to the IO service used by the server process for running
@@ -549,7 +549,6 @@ CommandMgrImpl::openCommandSocket(const isc::data::ConstElementPtr& socket_info)
 
     // First let's open lock file.
     std::string lock_name = getLockName();
-    errno = 0;
     int lock_fd = open(lock_name.c_str(), O_RDONLY | O_CREAT, 0600);
     if (lock_fd == -1) {
         std::string errmsg = strerror(errno);
@@ -559,7 +558,6 @@ CommandMgrImpl::openCommandSocket(const isc::data::ConstElementPtr& socket_info)
 
     // Try to acquire lock. If we can't somebody else is actively
     // using it.
-    errno = 0;
     int ret = flock(lock_fd, LOCK_EX | LOCK_NB);
     if (ret != 0) {
         std::string errmsg = strerror(errno);
@@ -568,7 +566,7 @@ CommandMgrImpl::openCommandSocket(const isc::data::ConstElementPtr& socket_info)
     }
 
     // We have the lock, so let's remove the pre-existing socket
-    // file if it's exists.
+    // file if it exists.
     static_cast<void>(::remove(socket_name_.c_str()));
 
     LOG_INFO(command_logger, COMMAND_ACCEPTOR_START)

diff --git a/src/lib/config/tests/command_mgr_unittests.cc b/src/lib/config/tests/command_mgr_unittests.cc
index 419cdf3cfd093fecdd85a800915e6fd316917a7a..b607d3e4144ce275e043af1890205f8754366401 100644 (file)
--- a/src/lib/config/tests/command_mgr_unittests.cc
+++ b/src/lib/config/tests/command_mgr_unittests.cc
@@ -17,7 +17,6 @@
 #include <hooks/hooks_manager.h>
 #include <hooks/callout_handle.h>
 #include <hooks/library_handle.h>
-#include <testutils/gtest_utils.h>
 #include <string>
 #include <vector>