[fdunparse2] Finished agent and added some @todos

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"
                 }

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);
 }
 

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();

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");