[5533a] Reviewed comment changes

diff --git a/premium b/premium
index 02dfa841546e3ab44e5cc8d55c82e0e5af8cbb7a..a7fc174b6a6dc556fbbea79dc267f458bd7e3a79 160000 (submodule)
--- a/premium
+++ b/premium
@@ -1 +1 @@
-Subproject commit 02dfa841546e3ab44e5cc8d55c82e0e5af8cbb7a
+Subproject commit a7fc174b6a6dc556fbbea79dc267f458bd7e3a79

diff --git a/src/lib/dhcpsrv/host_data_source_factory.cc b/src/lib/dhcpsrv/host_data_source_factory.cc
index 08695f54276a26ace59d432d9a891203f06c518a..d393af61b1424085a59f9500ac342ca6e2652efd 100644 (file)
--- a/src/lib/dhcpsrv/host_data_source_factory.cc
+++ b/src/lib/dhcpsrv/host_data_source_factory.cc
@@ -75,7 +75,8 @@ HostDataSourceFactory::add(HostDataSourceList& sources,
 }
 
 bool
-HostDataSourceFactory::del(HostDataSourceList& sources, const string& db_type) {
+HostDataSourceFactory::del(HostDataSourceList& sources,
+                           const string& db_type) {
     for (auto it = sources.begin(); it != sources.end(); ++it) {
         if ((*it)->getType() != db_type) {
             continue;

diff --git a/src/lib/dhcpsrv/host_mgr.cc b/src/lib/dhcpsrv/host_mgr.cc
index f9783e9807c25e11596f535da5dfd62fb03d30bb..f1fabdd48b1834b6b79c3860832c36be9a96171b 100644 (file)
--- a/src/lib/dhcpsrv/host_mgr.cc
+++ b/src/lib/dhcpsrv/host_mgr.cc
@@ -172,7 +172,7 @@ HostMgr::get4Any(const SubnetID& subnet_id,
     ConstHostPtr host = getCfgHosts()->get4(subnet_id, identifier_type,
                                             identifier_begin, identifier_len);
 
-    // Found it the config file or there are no backends configured?
+    // Found it in the config file or there are no backends configured?
     // Then we're done here.
     if (host || alternate_sources_.empty()) {
         return (host);
@@ -212,6 +212,7 @@ HostMgr::get4Any(const SubnetID& subnet_id,
         .arg(subnet_id)
         .arg(Host::getIdentifierAsText(identifier_type, identifier_begin,
                                        identifier_len));
+    // @todo: remove this
     if (negative_caching_) {
         cacheNegative(subnet_id, SubnetID(0),
                       identifier_type, identifier_begin, identifier_len);
@@ -357,6 +358,7 @@ HostMgr::get6Any(const SubnetID& subnet_id,
         .arg(Host::getIdentifierAsText(identifier_type, identifier_begin,
                                        identifier_len));
 
+    // @todo: remove this
     if (negative_caching_) {
         cacheNegative(SubnetID(0), subnet_id,
                       identifier_type, identifier_begin, identifier_len);

diff --git a/src/lib/dhcpsrv/host_mgr.h b/src/lib/dhcpsrv/host_mgr.h
index 1727bf20e4600a5f416e870f03369f99b415ccae..b47944aef3782ca1621a934609a87e3654f25ecf 100644 (file)
--- a/src/lib/dhcpsrv/host_mgr.h
+++ b/src/lib/dhcpsrv/host_mgr.h
@@ -393,9 +393,9 @@ public:
 protected:
     /// @brief The negative caching flag.
     ///
-    /// When true and the first cache source will be also
-    /// caching negative answers. This works for get[46]
-    /// for a subnet and an identifier.
+    /// When true and the first backend is a cache
+    /// negative answers are inserted in the cache.
+    /// This works for get[46] for a subnet and an identifier.
     bool negative_caching_;
 
     /// @brief Cache an answer.