[#42, !103] Additional review fixes

    Minor commentary changes.

diff --git a/src/lib/dhcp/iface_mgr.cc b/src/lib/dhcp/iface_mgr.cc
index a349e1d01b8052ce6258bdce97986c994c08f5d8..044fbb60813ea9c21c8d163e1dc503105ffb7ec0 100644 (file)
--- a/src/lib/dhcp/iface_mgr.cc
+++ b/src/lib/dhcp/iface_mgr.cc
@@ -300,12 +300,6 @@ void IfaceMgr::stopDHCPReceiver() {
     getPacketQueue4()->clear();
 }
 
-
-void
-IfaceMgr::closeSockets(const uint16_t) {
-    isc_throw(NotImplemented, "closeSockets(family) is obsolete");
-}
-
 IfaceMgr::~IfaceMgr() {
     // control_buf_ is deleted automatically (scoped_ptr)
     control_buf_len_ = 0;

diff --git a/src/lib/dhcp/iface_mgr.h b/src/lib/dhcp/iface_mgr.h
index 9a3ffdd568df0c57246c88f2fbbd4c25d857cc88..979e2667a7da9d7e401f02c27f2cd526423f90ad 100644 (file)
--- a/src/lib/dhcp/iface_mgr.h
+++ b/src/lib/dhcp/iface_mgr.h
@@ -919,12 +919,6 @@ public:
     /// Is used in destructor, but also from Dhcpv4Srv and Dhcpv6Srv classes.
     void closeSockets();
 
-    /// @brief Closes all IPv4 or IPv6 sockets.
-    ///
-    /// Obsolete!
-    /// @throw NotImplemented
-    void closeSockets(const uint16_t family);
-
     /// @brief Returns number of detected interfaces.
     ///
     /// @return number of detected interfaces

diff --git a/src/lib/dhcp/packet_queue.h b/src/lib/dhcp/packet_queue.h
index aa427aa0c13789f1f6ef6b40e53b170b78724c08..32e2c4a95c2f47c4558653918d00f57f521c4b55 100644 (file)
--- a/src/lib/dhcp/packet_queue.h
+++ b/src/lib/dhcp/packet_queue.h
@@ -47,7 +47,7 @@ public:
 
     /// @brief Constructor
     ///
-    /// @param queue_type name of this queue's implementation type.
+    /// @param queue_type logical name of the queue implementation
     /// Typically this is assigned by the factory that creates the
     /// queue.  It is the logical name used to register queue
     /// implementations.
@@ -233,7 +233,7 @@ public:
 
     /// @brief Constructor
     ///
-    /// @param queue_type logical name of the queue type
+    /// @param queue_type logical name of the queue implementation
     /// @param queue_capacity maximum number of packets the queue can hold
     PacketQueueRing(const std::string& queue_type, size_t capacity)
         : PacketQueue<PacketTypePtr>(queue_type) {
@@ -365,6 +365,7 @@ class PacketQueueRing4 : public PacketQueueRing<Pkt4Ptr> {
 public:
     /// @brief Constructor
     ///
+    /// @param queue_type logical name of the queue implementation
     /// @param capacity maximum number of packets the queue can hold
     PacketQueueRing4(const std::string& queue_type, size_t capacity)
         : PacketQueueRing(queue_type, capacity) {
@@ -384,6 +385,7 @@ class PacketQueueRing6 : public PacketQueueRing<Pkt6Ptr> {
 public:
     /// @brief Constructor
     ///
+    /// @param queue_type logical name of the queue implementation
     /// @param capacity maximum number of packets the queue can hold
     PacketQueueRing6(const std::string& queue_type, size_t capacity)
         : PacketQueueRing(queue_type, capacity) {

diff --git a/src/lib/dhcp/packet_queue_mgr4.cc b/src/lib/dhcp/packet_queue_mgr4.cc
index a982195b8ed0c8b886d548937194ae25e32d11f4..079c067de16a0885daa8726f005d4bd24a2a78e2 100644 (file)
--- a/src/lib/dhcp/packet_queue_mgr4.cc
+++ b/src/lib/dhcp/packet_queue_mgr4.cc
@@ -13,6 +13,9 @@ namespace isc {
 namespace dhcp {
 
 PacketQueueMgr4::PacketQueueMgr4() {
+    // @todo Please forgive magic strings and constants.  The default values,
+    // mechanisms will soon be reworked.
+
     // Register default queue factory
     registerPacketQueueFactory("kea-ring4", [](data::ConstElementPtr parameters)
                                           -> PacketQueue4Ptr {
@@ -28,7 +31,6 @@ PacketQueueMgr4::PacketQueueMgr4() {
             return (queue);
         });
 
-    /// @todo default comes from ? 
     data::ElementPtr parameters = data::Element::createMap();
     parameters->set("queue-type", data::Element::create("kea-ring4"));
     parameters->set("capacity", data::Element::create(static_cast<long int>(500)));

diff --git a/src/lib/dhcp/packet_queue_mgr6.cc b/src/lib/dhcp/packet_queue_mgr6.cc
index 77767ff162c5ad67eaf78b673a0dc6f33e18a8f6..b6c8ad9212f96d429242d4f0c837261dc795a737 100644 (file)
--- a/src/lib/dhcp/packet_queue_mgr6.cc
+++ b/src/lib/dhcp/packet_queue_mgr6.cc
@@ -13,6 +13,9 @@ namespace isc {
 namespace dhcp {
 
 PacketQueueMgr6::PacketQueueMgr6() {
+    // @todo Please forgive magic strings and constants.  The default values,
+    // mechanisms will soon be reworked.
+
     // Register default queue factory
     registerPacketQueueFactory("kea-ring6", [](data::ConstElementPtr parameters)
                                           -> PacketQueue6Ptr {
@@ -28,7 +31,6 @@ PacketQueueMgr6::PacketQueueMgr6() {
             return (queue);
         });
 
-    // @todo default comes from ?
     data::ElementPtr parameters = data::Element::createMap();
     parameters->set("queue-type", data::Element::create("kea-ring6"));
     parameters->set("capacity", data::Element::create(static_cast<long int>(500)));