[#145] addressed review comments

diff --git a/src/lib/dhcpsrv/cfg_subnets6.cc b/src/lib/dhcpsrv/cfg_subnets6.cc
index 9521a573df751423e889dc6b6ac1024a98744a3c..97b94eef30ab5d0b7fd69bd51d4871fde7d096a3 100644 (file)
--- a/src/lib/dhcpsrv/cfg_subnets6.cc
+++ b/src/lib/dhcpsrv/cfg_subnets6.cc
@@ -528,7 +528,7 @@ CfgSubnets6::updateStatistics() {
             stats_mgr.setValue(StatsMgr::generateName("subnet", subnet_id,
                                                       StatsMgr::generateName("pool", pool->getID(),
                                                                              "total-nas")),
-                               static_cast<int64_t>(pool->getCapacity()));
+                               pool->getCapacity());
 
             const std::string& name_nas =
                 StatsMgr::generateName("subnet", subnet_id,
@@ -543,7 +543,7 @@ CfgSubnets6::updateStatistics() {
             stats_mgr.setValue(StatsMgr::generateName("subnet", subnet_id,
                                                       StatsMgr::generateName("pd-pool", pool->getID(),
                                                                              "total-pds")),
-                               static_cast<int64_t>(pool->getCapacity()));
+                               pool->getCapacity());
 
             const std::string& name_nas =
                 StatsMgr::generateName("subnet", subnet_id,

diff --git a/src/lib/dhcpsrv/parsers/dhcp_parsers.cc b/src/lib/dhcpsrv/parsers/dhcp_parsers.cc
index 5a7b08485e4d28030c91b6db9de1ef456dec45ed..20df945fa0691980a26613372547a7f680e16344 100644 (file)
--- a/src/lib/dhcpsrv/parsers/dhcp_parsers.cc
+++ b/src/lib/dhcpsrv/parsers/dhcp_parsers.cc
@@ -630,9 +630,9 @@ SubnetConfigParser::createSubnet(ConstElementPtr params) {
     initSubnet(params, addr, len);
 
     // Add pools to it.
-    for (const auto& it : *pools_) {
+    for (const auto& pool : *pools_) {
         try {
-            subnet_->addPool(it);
+            subnet_->addPool(pool);
         } catch (const BadValue& ex) {
             // addPool() can throw BadValue if the pool is overlapping or
             // is out of bounds for the subnet.