GitLab

 - check for address being :: updated

    HooksManager::loadLibraries() call now requires
    LibraryCollection as parameter

    Merge branch 'trac4492'

src/bin/dhcp4/tests/hooks_unittest.cc
src/bin/dhcp6/tests/hooks_unittest.cc
    Fixed includes, copyright, and removed unnecessary calls to
    IfaceMgr::instance().deleteAllExternalSockets();

src/bin/dhcp4/tests/hooks_unittest.cc
    TEST_F(LoadUnloadDhcpv4SrvTest, unloadLibaries)  - new test that
    verifies Hooks libraries are unloaded by the Dhcpv4Srv destructor

src/bin/dhcp6/tests/hooks_unittest.cc
    TEST_F(LoadUnloadDhcpv6SrvTest, unloadLibaries)  - new test that
    verifies Hooks libraries are unloaded by the Dhcpv4Srv destructor

 Thanks to wpk for reporting this issue.

Presented rate is now the number of ACK/Reply messages,
rather than DO/SA.

The extended test verifies that the stats manager is
still usable after some packets have been collected.

When the perfdhcp performs unordered packet lookup within
a bucket of sent packets, it will remove all timed out
transactions.

This is to eliminate some of the recently found covertity isssues.
Reviewed on jabber.

A call to HooksManager::getHooksManager().unloadLibraries() was added to
the destructors Dhcpv4Srv::~Dhcpv4Srv() and Dhcpv6Srv::~Dhcpv6Srv(). This
ensures hook libraries are unloaded prior to static objects.

 - returnining different values in callout_params_library.cc
 - returned values are now described for getParameter
 - corrected index syntax comment in getParameter
 - added missing include
 - hooks_user.dox updated
 - hooks unit test moved, so it is now next to other hooks tests

- No additional check for HR mode in findReservations
- Additional commentary in the loop collecting host ids.

    Merges in trac4239.