[1958] Code cleanup: naming, getters layout and return statements.

[1958] Replaced unsigned long long with uin64_t

[1958] Replaced printf with cout.

[1958] Make packet types const and passing them by reference.

[master] reverted a small comment merge mistake

[master] Merge branch 'trac2148'

Conflicts:
src/lib/dns/labelsequence.h

[2148] fix typo in comment

Merge branch 'master' into trac2124

[2124] Don't catch InvalidBufferPosition when parsing wiredata

[2124] Rewrite code to avoid duplication

Merge branch 'master' into trac2106

[2106] Rewrite code to avoid using a variable

Merge branch 'master' into trac2092_2

Conflicts:
src/lib/datasrc/rbtree.h

Also fixes some other API and data differences between master and
trac2092_2 branch which caused several compile and test failures.

[2092] Add comment about pointer dereference asked by reviewer

[master] qualify memcpy with std::, it won't compile with sunstudio otherwise.

in response to buildbot report.  confirmed the fix on the box, and should be
trivial.  so directly committing.

[2095] include <cstring> for memcpy and qualify it with std::

will be needed for sunstudio.

[2095] editorial cleanups in doc.

[master] Merge branch 'trac2091b'

[2091b] removed the assert from getLabels(), which will soon be moot anyway.

[2113] Check the MasterFiles params

They are skipped in the ClientList, so not checked at all.

[2113] Check the configuration by creating the list

A small bugfix in the validation is included. There's no test for it
directly - it is tested implicitly in the plugin test, but I wasn't able
to reproduce it on small test spec files. If we weren't planning for
replacement of the whole thing, it would need some investigation.

[2113] Construct the classes

[2113] Validate against the spec

[master] fixed a typo in doxygen doc.
trivial, direct commit.

[master] corrected trivial style matters I happened to notice.

folded long lines, and adjusted the position of '*'s.
should be trivial enough, directly committing.

[master]Merge branch 'trac1999'

[2148] Address review comments

- typos
- few code fixes
- test updates

[2124] some editorial fixes to test data comments

[2124] prefer empty() over size() > 0 (see the previous commit)

[2124] some suggested changes:
- use !empty() instead of size() > 0.  for vectors there should be
  essentially no difference, but for other type of containers the former
  is generally better, and so I believe it's better to always follow that
  convention.
- constify

[2137] Removed 'setstats' command and revied the argument of 'getstats' from
 the bind10 daemon

'setstats' is a command duplicate for sending statistics and it was useless
since boss already has same command 'getstats'. Besides, 'getstats' command get
changed. It chacks validity of statistics data before returing. The second
parameter of the 'getstats' is more simpler. The keys of 'owner' and 'data'
needs to be specified.  Statistics data are directly specifed. If such
obsoleted name is specified, the stats module might discard sent data.

[2106] updated documentation

[2106] Change variable naming slightly

[2106] Remove Name variant of a RBTree::find() method

It was only used in the memory datasrc, which has been modified to use
the LabelSequence variant.

Also unify some tests.

[2095] a few more editorial fixes to comments.

[2124] Rename SSHFP getter methods

[2124] Add another toWire() test

[2124] Don't add trailing space in toText() if fingerprint is empty

Also add some tests.

[2124] Don't access fingerprint_ data when size is 0

[master] Merge branch 'trac2152'

[master] report actual error in masterload

if file cannot be opened.

acked on jabber by jinmei

[2152] don't play the game with union.  the init hack didn't work.

it was wrong anyway because it tried to write to one field and read
from the other.  and, it actually caused a failure on Solaris/Sparc.
I suspect it's due to endian and the data size differences, but since
the code was wrong anyway, I now decided to make it (in some sense) less
safer but more straightforward.

[2124] don't keep a reference to a temporary object.

'rdf' in the original code immediately became dangling reference.

[2106] Allow RBTree::find() to take LabelSequence

The Name find() variants already test the implementation well as they
wrap around it, but one test was added for checking the LabelSequence
variant API.

[2095] cleanup: removed unnecessary header file.

[2095] more documentation/comment update

[2095] doxygen update

[2095] two more minor test cases

[2095] reject too big RRSIG

[2095] supported RRSIG encoding with basic tests

[2095] rejected RRSIG in start()

[2095] cleanup: removed now-unused stuff.

[2095] handled various error cases

[master] remove the file generated by unittest

remove file generated by src/bin/auth/tests/command_unittest.cc

to fix distcheck build

[2095] added some basic parameter validations.

[master] the datasrc.spec file is generated so use builddir

[master] make sure src/bin/auth/tests/testdata is in dist tarball

[2091b] removed an essentially-redundant assertion check.

[2091b] cleanup: removed a leftover comment line

[2091b] simplified bitfield swap

[2091b] more comment clarifications.

[2148] final touches

[2148] Create test fixture for common names and buf

(not for actual labelsequences, IMO it looks better if they are defined locally)

[2091b] doxygen comment wording fix.

[2148] cleanup and documentation

[2148] extend() improvements and more tests

[master] regen some docs ; catch up on some doc changes

(hopefully soon we won't keep these in the git tree.)

[2148] minor cleanup

[2148] add LabelSequence::extend() method

[master] fix errors with ambiguous types and other boost uses

Stop doing "using namespace boost" and be specific with the using boost::.

Problem noticed on Solaris 11 with g++ 4.5.2 and boost 1.50.0.
Tested with older g++ 4.1.3 with boost 1.42.0 (on NetBSD).

Discussed little on jabber.

[master] fix "error: reference to `map' is ambiguous"

Explicitly use std:: for map

It seems some compilers can't find it otherwise. This should make them
happy.

Don't require dynamic libs on static link

There were tests that should have been disabled too, as they use the
static data source, which is loaded dynamically from a .so. Some systems
break on them.

[1958] Added perfdhcp statistics printing functions.

[2148] update doxygen for constructor

[2148] 'extendable' labelsequence constructor

[2124] Test toWire() with empty fingerprint data

[2124] Handle empty SSHFP fingerprints

[2124] Add wiredata tests where the record is shorter than rdata len indicates

[2095] tested multi-RDATA cases.

the implementation already supported this case, so only need to adjust tests.
encoding parts of the temporary hack is now unnecessary and removed.
tests were refactored so we can share the same test logic in multiple
scenarios.

[2095] cleanup: removed the old checkData tests.  It's now tested via encoder.

[2095] added initial version of rdata encoder.

generally working, but missing some validity, corner case tests, etc.

[2092] Remove some initializers

[2092] Stop using NULLNODE inside RBNode and RBTree

Also change the dot dumper to use the offset_ptr getters.

[2095] adjusted utility functions for tests with new LabelSeq interface.

This is a kind of conflict resolution after the initial merge.

[2091a] more clarification on the use of separate memsgmt for each zone.

also fixed some text that didn't really make sense.

Remove unused function

Changelog for #1976

Merge #1976

Finally! This one was big.

With some minor updates to catch up to the changes after merge.

Conflicts:
src/bin/auth/auth_srv.cc
src/bin/auth/auth_srv.h
src/bin/auth/tests/auth_srv_unittest.cc
src/lib/datasrc/client_list.cc
src/lib/datasrc/tests/client_list_unittest.cc

[2136] preserve all statistics data including the inactive instances'
one while stats module is working
also system test of statistics is disabled until codes of both
Boss(#2137) and Auth(#2138) due to their dependencies

[2136] implemented polling to each modules for collecting statistics data

[2136] revised test_util.py miscellaneously

* change of b10-config dbfile
* update specfile of MockBoss
* invoking three extra Auth modules at a time in the BaseModules class
* implement 'getstats' command in each mock module

[2136] revised the debug message of request

since stats doesn't request only boss but request every module

Merge remote-tracking branch 'origin/trac1976-cont-3' into work/merge

Merge remote-tracking branch 'origin/trac1976-cont-2' into work/merge

Merge remote-tracking branch 'origin/trac1976-cont' into work/merge

Conflicts:
src/bin/auth/tests/datasrc_configurator_unittest.cc

[1976] Make the reload test more realistic

We change the file, not the memory content.

[2091b] more documentation

[2092] Test the case when the upper node is indeed the null node

[2092] Update doc comment