[#3230] addressed review

src/lib/dhcp/pkt_filter_inet.cc

@@ -22,11 +22,11 @@ const size_t PktFilterInet::CONTROL_BUF_LEN = 512;
 bool
 PktFilterInet::isSocketReceivedTimeSupported() const {
 #ifdef SO_TIMESTAMP
-        return (true);
+    return (true);
 #else
-        return (false);
+    return (false);
 #endif
-} 
+}
 
 SocketInfo
 PktFilterInet::openSocket(Iface& iface,

src/lib/dhcp/pkt_filter_inet6.cc

@@ -25,9 +25,9 @@ const size_t PktFilterInet6::CONTROL_BUF_LEN = 512;
 bool
 PktFilterInet6::isSocketReceivedTimeSupported() const {
 #ifdef SO_TIMESTAMP
-        return (true);
+    return (true);
 #else
-        return (false);
+    return (false);
 #endif
 }
 

src/lib/dhcp/pkt_filter_lpf.cc

@@ -131,9 +131,9 @@ namespace dhcp {
 bool
 PktFilterLPF::isSocketReceivedTimeSupported() const {
 #ifdef SO_TIMESTAMP
-        return (true);
+    return (true);
 #else
-        return (false);
+    return (false);
 #endif
 }
 

src/lib/dhcp/tests/pkt_filter6_test_utils.h

@@ -76,8 +76,8 @@ class PktFilter6Test : public ::testing::Test {
     /// @brief Checks that a received message has the appropriate events
     /// in it's event stack.
     ///
-    /// @param rcvd_msg An instance of the message to be tested.
-    /// @param so_time_supported If true the event stack should have with
+    /// @param msg An instance of the message to be tested.
+    /// @param so_time_supported If true the event stack should have
     /// a SOCKET_RECEIVED event followed by a BUFFER_READ event, if false
     /// it should have only the latter.
     void testReceivedPktEvents(const PktPtr& msg, bool so_time_supported) const;

src/lib/dhcp/tests/pkt_filter_bpf_unittest.cc

@@ -205,7 +205,7 @@ TEST_F(PktFilterBPFTest, receive) {
     testRcvdMessageAddressPort(rcvd_pkt);
 
     // Verify that the packet event stack is as expected.
-    testReceivedPktEvents (rcvd_pkt, pkt_filter.isSocketReceivedTimeSupported());
+    testReceivedPktEvents(rcvd_pkt, pkt_filter.isSocketReceivedTimeSupported());
 }
 
 // This test verifies that if the packet is received over the raw

src/lib/dhcp/tests/pkt_filter_inet6_unittest.cc

@@ -145,7 +145,7 @@ TEST_F(PktFilterInet6Test, receive) {
     testRcvdMessage(rcvd_pkt);
 
     // Verify that the packet event stack is as expected.
-    testReceivedPktEvents (rcvd_pkt, pkt_filter.isSocketReceivedTimeSupported());
+    testReceivedPktEvents(rcvd_pkt, pkt_filter.isSocketReceivedTimeSupported());
 }
 
 } // anonymous namespace

src/lib/dhcp/tests/pkt_filter_inet_unittest.cc

@@ -160,7 +160,7 @@ TEST_F(PktFilterInetTest, receive) {
     testRcvdMessageAddressPort(rcvd_pkt);
 
     // Verify that the packet event stack is as expected.
-    testReceivedPktEvents (rcvd_pkt, pkt_filter.isSocketReceivedTimeSupported());
+    testReceivedPktEvents(rcvd_pkt, pkt_filter.isSocketReceivedTimeSupported());
 }
 
 } // anonymous namespace

src/lib/dhcp/tests/pkt_filter_lpf_unittest.cc

@@ -196,7 +196,7 @@ TEST_F(PktFilterLPFTest, receive) {
     testRcvdMessageAddressPort(rcvd_pkt);
 
     // Verify that the packet event stack is as expected.
-    testReceivedPktEvents (rcvd_pkt, pkt_filter.isSocketReceivedTimeSupported());
+    testReceivedPktEvents(rcvd_pkt, pkt_filter.isSocketReceivedTimeSupported());
 }
 
 // This test verifies that if the packet is received over the raw

src/lib/dhcp/tests/pkt_filter_test_utils.h

@@ -85,8 +85,8 @@ class PktFilterTest : public ::testing::Test {
     /// @brief Checks that a received message has the appropriate events
     /// in it's event stack.
     ///
-    /// @param rcvd_msg An instance of the message to be tested.
-    /// @param so_time_supported If true the event stack should have with
+    /// @param msg An instance of the message to be tested.
+    /// @param so_time_supported If true the event stack should have
     /// a SOCKET_RECEIVED event followed by a BUFFER_READ event, if false
     /// it should have only the latter.
     void testReceivedPktEvents(const PktPtr& msg, bool so_time_supported) const;

src/lib/dhcp/testutils/pkt_filter_test_stub.cc

@@ -27,9 +27,9 @@ PktFilterTestStub::isDirectResponseSupported() const {
 bool
 PktFilterTestStub::isSocketReceivedTimeSupported() const {
 #ifdef SO_TIMESTAMP
-    return(true);
+    return (true);
 #else
-    return(false);
+    return (false);
 #endif
 }