[2292] minor cleanups: constify, indentation

[2292] Update tests

They don't need the mutable versions to work, but they used them anyway.

[2292] Re-add version of find removed by accident

This one was not mutable, it just took Name as an argument.

[2292] Drop tests for InMemoryClient::add

As the method was removed.

[2292] Remove unused template parameters

The domain tree no longer supports the mutable find versions, so we
don't need the template parameters allowing for them.

[2292] Remove mutable find methods

They are currently used in test code only (so we'll need to update the
tests to compile). The real code seams clean now.

[2292] Remove the setZoneData method

Currently, the content of zone is passed to the addZone method. No way
to change it later, but the addZone overwrites the old value if called
again.

This should hopefully allow for removal of mutable find methods from the
tree.

[2292] Get rid of the const_cast

It was needed when extracting data from a domain tree chain. The chain
now can hold mutable pointers too, so we use that (and some amount of
template bureaucracy) to avoid the cast.

While the interface changed (on the core find function, it is not
possible to pass const node chain and have a mutable node get out), it
doesn't seem to influence the current code. Also, it is a private
interface anyway, so it should be safe.

[2292] Parametrize constness of the chain

[master] Set lockfile path for DHCP tests

[master] Update .gitignore files

[master] Remove empty GNU-style dist files

[master] define dbutil_test.sh as noinst_SCRIPTS to auto-regenerate on modify.

this will prevent surprising test failure on an incrementally updated dev
tree.  okayed on jabber.

[master] added changelog for #1870

[1870] avoid using "no" keyword as a configured value for "bind10-include".

[1870] removed unnecessary call to AX_BOOST_INCLUDE.

since AC_REQUIRE internally does it.

[1870] removed rrsetlist.h from the libdns___include_HEADERS

it will be cleaned up and should better be excluded from the publicly
visible list.

[1870] some more doc cleanups

[1870] provided top level README

[1870] separated BIND 10 common libs and DNS lib

so that we can eventually selectively choose module specific libraries.

[1870] removed src/bin/host/Makefile from toplevel configure.ac.

[1870] introduced separate m4 files for dependencies.

[1870] make sure install some more headers under dns/ and util/

there may be more, but at least this makes (b10-)host compile as a 3rd party
app.

[1870] moved src/bin/host under example/

(at least for now) there's no way to transform xml doc to other formats.
only xml is provided for reference.

[1870] created a separate dir to store example "3rd party" apps using BIND 10.

[2203] renamed datasrc_configurator to datasrc_config; it represents it better.

[2203] forotten cleanup: removed now-unused header file.

[2203] a piggy back fix: prevent redundant initial data configuration.

this addresses the issue described in #2291.  Still not really clean,
but thanks to boost::bind we can centralize all the code logic in the
callback, so I think it's now less likely that we forget cleaning it up
when the hack is not necessary.

[2203] changed configurator class to a simple function.

as it's now completely stateless and can work independently.
the common specialization for the main implementation is defined in a
new created .cc file.

[2203] pass server obj to configurator's reconfigure().

now the configurator class is completely stateless.

[2203] refactoring 2nd step: configurator can now be a separate object.

i.e., it's not a singleton any more.
testReconfigure() method isn't needed any more because it doesn't hold
CC session internally.
DatasrcConfiguratorTest.initialization test currently fails and is
disabled for now.  The plan is to make the class completely stateless,
at which point we don't even have to think about initialization or cleanup,
and then the test will be able to be removed.

[2203] changed the callback type of addRemoteConfig to boost::function.

this will make it more convenient, e.g., by allowing the caller to pass
boost::bind encapsulating a class object and a class method.

boost::function is upper compatible to function pointer, so it doesn't
ensure source-level compatibility.

the functor overhead shouldn't matter in this context, and since this module
already uses boost::function this change doesn't introduce additional
dependency.

[2203] refactoring 1st step: move session obj outside of datasrc configurator.

[1899] Allow other RRtypes in nsec3 table (for RRSIGs)

* The unique constraint on NSEC3 owner per zone is now removed
* NSEC3 RRSIGs are now returned next to their RR set
* Test was adjusted to check for RRSIGs too

[1899] Add another unittest to check that NSEC3 RRs are being returned

[1899] Break up UNION SQL query into two and merge manually

Also drop unnecessary columns and ORDER BYs.

[1899] Add a unique constraint to nsec3 table

This is to check that only one NSEC3 RR exists per owner name in a
zone.  The code to fix #1899 would depend on this assertion.

[master] add std:: for memset().  sunstudio requires this with <cstring>

[2275] fix strncpy in socketrequestortest

[2275] address review comments

[2275] valgrind fixes: uninitialized memory

[2275] valgrind fixes: uninitialized memory

[2275] use const char* for test globals

[2275] valgrind fix: initialize memory in labelsequence test

[2275] valgrind fix: store singleton instance in auto_ptr

[2275] valgrind fix: close db in test case

[master] Add libb10-cc.la to tests Makefile.am

[2278] Add temporary suppression for false positives

[2278] remove a bit of dead code

[2278] A number of cppcheck fixes

and whitespace things, while i was there

[2340] corrected the position of closing 'fi' for an 'if GXX = yes'.

[2340] added comment about why we need to explicitly define empty dtor.

[2340] added workaround for the clang++ "unused arguments" warn/error.

by specifying -Qunused-arguments.  Also, as a cleanup for better organization
I moved compiler specific settings to earlier part of configure.ac, where
this workaround was added.

[2340] Revert "[2340] specify thread flag as part CPPFLAGS, not LDFLAGS."

This reverts commit daf81c8846bf0b4e4308068a019c96beb09c5199.

I believe the sense of the change is correct, but it has effects on
other OSes, so it's probably better to use clang specific workaround
(will commit it later).

[2340] Revert "[2340] specify -pthread in CXXFLAGS and via system-indepedent variable."

This reverts commit d62a1e4250c88e765b3cd7f77fe9a948db0a565d.

I believe the sense of the change is correct, but it has effects on
other OSes, so it's probably better to use clang specific workaround
(will commit it later).

[2340] specify -pthread in CXXFLAGS and via system-indepedent variable.

specifying it in LDFLAGS would trigger build error with some versions of clang.
hardcoding "-pthread" is less portable.

[2340] removed an effectively unused variable.  clang would complain.

[2340] specify thread flag as part CPPFLAGS, not LDFLAGS.

newer version of clang treats the latter as a fatal error.

[2340] explictly define destructor of mostly empty template class.

without this new versions of clang++ seem to complain that instantiation of
the class results in an unused variable.

[2349] Ignore EXPECT_DEATH when run in valgrind

Valgrind complains (probably rightfully so) that memory is leaking in death-tests. This has been reported upstream, but is not addressed yet, and may not be in the near future, if at all (one response was don't use EXPECT_DEATH).

So I added a checker function, *if* valgrind headers are available, that returns true if the tests are run under valgrind. Death tests are now ignored if so.

Note that we should be very careful about using this; and only use it for things we are sure valgrind can't help, since it is possible to do things differently when using valgrind, which kind of defeats its purpose

[2268] Fix disabled tests without using InMemoryClient::add()

Also remove direct tests for add() as the method has been removed now.

[master] Compilation fix for OpenBSD.

For some reason comparison operator for std::vector<uint8_t> does
not work with EXPECT_EQ macro on OpenBSD.

[master] Warning shown on Debian6 with g++ 4.4 removed.

[master] Compilation fix for Solaris in LeaseMgr

Merge branch 'master' into trac2268

Conflicts:
src/lib/datasrc/memory/Makefile.am
src/lib/datasrc/memory/memory_client.cc
src/lib/datasrc/memory/memory_client.h
src/lib/datasrc/tests/memory/memory_client_unittest.cc

[master] Merge branch 'trac2237' (Subnet/pool storage for DHCPv4)

Conflicts:
ChangeLog
src/lib/dhcp/cfgmgr.h
src/lib/dhcp/tests/cfgmgr_unittest.cc

[2237] Changes after review.

[master] Merge branch 'trac2269' (configuration for b10-dhcp6)

[2269] Typo fixed.

[master] Merge branch 'trac2140' (Abstract API for DHCP database)

Conflicts:
ChangeLog
src/lib/dhcp/tests/Makefile.am

[2223] small change in comment

[2223] revert vector implementation, use a count function

[2304] Minor changes to header comments

[2140] LeaseMgr minor tweaks, ChangeLog updated.

[2266] Update ChangeLog

Merge branch 'master' into trac2266

[2332] (unrelated) style cleanups: func name convention, () after sizeof.

[2332] (unrelated) cleanup: removed unnecesary failure case in Locker.

We don't have to worry about the case lock() throws in the constructor.
In that case the destructor won't be called.
This simplification will make the code simpler, and we can also make
mutex_ member variable a const.

[2332] added class documentation

[2332] added more tests and some more comments

[2332] supported wait() and signal()

[2332] removed unused changed

[2332] makefile updates for the new files

[2332] added initial sets of condition variable encapsulation

currently it does almost nothing except initialize and cleanup

[2332] unrelated cleanup: noncopyable doesn't have to be public inheritance

(maybe a matter of taste, though)

[master] minor wording fix in a README file.

committing at my discretion.

[2340] Merge branch 'master' into trac2340

[master] changelog for #2339

[master] Merge branch 'trac2339'

Compilation fix: usual static_cast<X*>(NULL)

Sunstudio is unhappy with plain NULL inside EXPECT_EQ and thinks it is
int. This makes it think otherwise.

Remove extra comma at the end of enum

This should fix compilation issue on sunstudio.

[2207] Document the local zone loader

[2340] corrected the position of closing 'fi' for an 'if GXX = yes'.

[2140] DUID no longer returns unsafe references, tests updated.

[2140] Renew, rebind timers added to lease4, recycle-timer removed

[master] added changelog for #2244

[master] a minor grammar fix to changelog entry.

[master] Merge branch 'trac2244'

[master] Merge branch 'trac2204'

[2304] Code cleanup.