[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.

[2198] Remove dependency on libb10-util in src/lib/util/unittests/

[2302] Add a set test for map Element

[2302] Check map data

[2302] Add ConstElementPtr version of tests for all types of Element

[2140] Changes after review: LeaseMgr::getLeaseX() methods expanded.

[2207] Definition of the local zone updater

[2304] Unit tests for Option6IntArray.

[2207] Documentation of the base ZoneUpdater

[2155] fixed wrong comment

[2207] Interface of the ZoneUpdater

[2155] inline methods in a canonical way

[2155] use auto-generated copy constructor

[2223] add full rrsets comparison in test

this implementation goes even further than the original one did; it converts both iterators (of rrsets) to string vectors, and sorts and compares the result, so it really does compare everything that is contained in all rrsets (as long as it is in toText() output).

It can compare vectors where the signatures are put in as separate rrsets and vectors where the signatures are contained in the rrset (by splitting them out when constructing the string vector), and as such is backwards comaptible with the existing tests that use rrsetsCheck.

Merge #2292

[2292] Add a TODO about re-adding tests

Planned for merge #2268

[2155] checkCountersAllZeroExcept() returns void

[2155] remove meaningless NULL check

[2155] added notes for including header files with EXTRA_DIST

[2157] fixed systest

[2157] update spec file

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

[2268] separated zone data Loader from ZoneDataUpdater.

it doesn't have to be part of the updater (with the ability of accessing
private members).

[2302] Untabify and delete trailing whitespace

[2302] Fix bugs in Element class

Two bugs in src/lib/cc/data.h:

* Element::find() does not return value because its argument is not a
  reference.

* getValue() member functions does not work for ConstElementPointer
  because getValue() member functions are not specified as const.

Aharen found these bugs.

[2268] cleanup: removed (now-)unnecessary include

[2268] constify

[2268] doxigen editorial fixes: use \code for code snippet; folded a long line

[2198] First use a mutex for mutual exclusion among threads, then a lock file

[2198] Add Mutex::tryLock() and make lock methods public

tryLock() is necessary inside InterprocessSyncFile. It also doesn't
make sense afterwards to make a dynamically constructed Mutex::Locker
that lives after the basic block is exited. So lock() and unlock()
have also been made public.

I've added comments to discourage the use of these methods directly
and use the Mutex::Locker where applicable.

[2204] removed an obsolete comment that only made sense for the previous code.

[2204] added notes about future extensions of incremental datasrc config update

[2339] changed the search order of python executable: seek python3.x first.

this works around a bit awkward installation setup where there's a link
from "python3" to "python3.x" but not from "python3-config" to the
corresponding "python3.x-config".  That happens for recent versions of
Homebrew.

[2198] Test that locks work from threads

This test fails now because there's no mutual exclusion among threads
in the implementation.

[2198] Build threads directory first because . depends on it

[2244] missing update: Component.running() was renamed is_running().

[2269] Changes in Dhcp6 config parser after review.

[2304] Added unit tests for Option6Int option.

[2237] Tests for addr utilities added, other minor fixes after review.

[2237] Changes after review.

[2304] Asserts on types of options created with factory functions.

[2292] Clarify test

And make it slightly stronger (it now checks something that should be
mostly obvious too, but who knows, with software).

[2292] Pass the old zone back

Instead of releasing it directly. While the internal release was more
convenient, it didn't allow for swapping things fast under a mutex and
then spending the time releasing it unlocked.

[2292] Don't pass SegmentObjectHolder

It is supposed to be mostly private. Use it internally only.

[master] Set lockfile path for DHCP tests

[master] Update .gitignore files