[fdunparse2] Finished agent and added some @todos

diff --git a/src/bin/agent/tests/testdata/get_config.json b/src/bin/agent/tests/testdata/get_config.json
index b52f7910bf8ec3c9bbcee70373733345da190856..3c264b0354deff8201d667f8789d01e9bbad4958 100644 (file)
--- a/src/bin/agent/tests/testdata/get_config.json
+++ b/src/bin/agent/tests/testdata/get_config.json
@@ -16,7 +16,7 @@
         },
         "hooks-libraries": [
             {
-                "library": "/tmp/kea/src/bin/agent/tests/.libs/libbasic.so",
+                "library": "/opt/local/control-agent-commands.so",
                 "parameters": {
                     "param1": "foo"
                 }

diff --git a/src/lib/dhcpsrv/cfg_mac_source.cc b/src/lib/dhcpsrv/cfg_mac_source.cc
index 9475f6e14de30d0f75cc170f1a3574dbbd2e6791..7a0db260b38dc73b34c012d47b2a70796c0b2ab3 100644 (file)
--- a/src/lib/dhcpsrv/cfg_mac_source.cc
+++ b/src/lib/dhcpsrv/cfg_mac_source.cc
@@ -81,6 +81,7 @@ ElementPtr CfgMACSource::toElement() const {
         }
         result->add(Element::create(name));
     }
+    // @todo check if the list is empty (including a new unit test)
     return (result);
 }
 

diff --git a/src/lib/dhcpsrv/cfg_subnets4.cc b/src/lib/dhcpsrv/cfg_subnets4.cc
index 425c5faa0b43ea887d62c040155f31e764c23898..14f1c15be674cedb13bfc09c9d31ae158318c78f 100644 (file)
--- a/src/lib/dhcpsrv/cfg_subnets4.cc
+++ b/src/lib/dhcpsrv/cfg_subnets4.cc
@@ -313,7 +313,7 @@ CfgSubnets4::toElement() const {
         ElementPtr pool_list = Element::createList();
         for (PoolCollection::const_iterator pool = pools.cbegin();
              pool != pools.cend(); ++pool) {
-            // Prepare the map for a pool
+            // Prepare the map for a pool (@todo move this code to pool.cc)
             ElementPtr pool_map = Element::createMap();
             // Set pool
             const IOAddress& first = (*pool)->getFirstAddress();

diff --git a/src/lib/dhcpsrv/cfg_subnets6.cc b/src/lib/dhcpsrv/cfg_subnets6.cc
index 2407976bdb1b91728a2f9b2b74efd6eff4382450..c2c8c5f16546a1f6375328018209fd36c356ea4a 100644 (file)
--- a/src/lib/dhcpsrv/cfg_subnets6.cc
+++ b/src/lib/dhcpsrv/cfg_subnets6.cc
@@ -289,7 +289,7 @@ CfgSubnets6::toElement() const {
         ElementPtr pool_list = Element::createList();
         for (PoolCollection::const_iterator pool = pools.cbegin();
              pool != pools.cend(); ++pool) {
-            // Prepare the map for a pool
+            // Prepare the map for a pool (@todo move this code to pool.cc)
             ElementPtr pool_map = Element::createMap();
             // Set pool
             const IOAddress& first = (*pool)->getFirstAddress();
@@ -320,7 +320,7 @@ CfgSubnets6::toElement() const {
         ElementPtr pdpool_list = Element::createList();
         for (PoolCollection::const_iterator pool = pdpools.cbegin();
              pool != pdpools.cend(); ++pool) {
-            // Get it as a Pool6
+            // Get it as a Pool6 (@todo move this code to pool.cc)
             const Pool6* pdpool = dynamic_cast<Pool6*>(pool->get());
             if (!pdpool) {
                 isc_throw(ToElementError, "invalid pd-pool pointer");