[2573] (minor) Spacing and capitalization fixes

Just comments and style. No functionality or code change.

[2637] Removed dead initialization from dhcp4_srv_unittest.cc.

[2637] Fixed doxygen warnings in the DHCP code.

[2637] Cast uint8_t to int when logging them.

When directly logging uint8_t values they are rather printed as char,
not the integer value within the range of 0..255. Converting them to
integer guarantees that they are printed as numbers.

[2438] added python wrapper for ZoneUpdater::getRRsetCollection().

[2438] introduce Python RRsetCollectionError and convert it find().

[2438] make sure cppobj is NULL in the base RRsetCollection wrapper classs.

[2438] more conflict resolution

[2438] Merge branch 'trac2438-0' into trac2438

resolved Conflicts:
src/lib/datasrc/Makefile.am
src/lib/datasrc/database.cc
src/lib/datasrc/tests/database_unittest.cc
src/lib/datasrc/tests/master_loader_callbacks_test.cc
src/lib/datasrc/tests/zone_loader_unittest.cc
src/lib/datasrc/zone.h
src/lib/dns/rrset_collection_base.h

[2573] count added RR within datasrc loader, and deprecate MasterLoader method

in fact, the master loader's method is redundant because its user can count
them the same way.  its own method might be of some use in the future,
but until then I'd simpy remove it based on YAGNI.

Also made some unrelated cleanup based on recent clarification of
null tests on pointer/shared pointers.

[2637] Use full configuration everytime the new configuration is commited.

[2573] updated python wrapper using loader's getProgress.

get_size/get_position were deprecated and removed.

[2637] Validate option space name in the configuration parsers.

[2573] introduce getProgress instead size/position.

[master] Merge branch 'trac2572-fix'

[2595] Better error reporting on failed login in bindctl

If the connection is reset, it is likely there is an SSL problem, so report that.
Also a tiny refactor for the common login code in login_to_cmdctl()

[2595] Better handling of bad files in b10-cmdctl

Still a fatal error if it gets to that point, but
- exact error message
- basic checks (exists, isfile, isreadable) are done when updating configuration as well

Changelog for #2436

Merge #2436

Let isc::datasrc::ZoneLoader validate the data before committing by
isc::dns::checkZone. Also, remove the in-house validation from b10-loadzone,
since it's only a subset of the checkZone, which is now performed.

Import a merge fixup from mukund.

[2591] Code cleanup in Dhcp4SrvTest.processRequest and .processDiscover.

Merge branch 'master' into trac2435

Conflicts:
src/lib/datasrc/tests/database_unittest.cc

[master] Initialize the list of strings before passing it to boost foreach.

This overcomes the compilation failure on Sunstudio that is caused by the
boost::foreach_detail_::is_rvalue_() overloading ambiguity when
passing the rvalue to boost foreach.

[2157] corrected order of EXPECT_EQ params: 1st param should be the 'expected'.

[2157] constify

[2591] Append requested options to the OFFER message.

[2591] Added est that checks if requested options are appended to offer.

[2559] Remove the temporary configuration of the lease database

The DHCP6 server had the lease database configuration temporarily
hard-coded.  With the introduction of database configuration
parameters, that is no longer needed.

[2314] Added the test to check that standard V4 option can be encapsulated.

[2559] Make opening of the lease database an informational message

[2596] "Interface" parameter in Subnet6 configuration added.

[2314] Append sub-options to DHCPv4 options.

[2314] Encapsulate standard DHCPv6 options.

[2314] Added encapsulated option space name to std option definitions.

[2314] Removed debug message.

[2314] Append sub-options to options being configured.

[2314] Simplified getting configured options when creating answer packet.

[2314] Configure encapsulated option space for DHCPv6 options.

[2314] Create option definition with encapsulated option space name.

[2314] Added encapsulated option space name to option definition.

[2314] Validate option space name received in a configuration.

[2559] Changes to comments

[2597] Some changes to messages and documentation

[master] Added ChangeLog entry for #2317.

[master] Merge branch 'trac2317'

Conflicts:
src/bin/dhcp6/dhcp6_srv.cc
src/lib/dhcpsrv/Makefile.am
src/lib/dhcpsrv/cfgmgr.h

[master] Add std:: to memcmp

[2582] Lettuce tests for msgq logging

This way we check the logging works to the correct place and that the
configuration is handled. It is better checked by lettuce than unit
tests, since there's a lot of inter-module communication in it.

[2582] Background the handling of commands

[2582] Logging of msgq configuration

[2582] Unrelated cleanup: don't use global variable

The original code had a TODO, fixing it.

[2582] Perform some locking of msgq

Not that it would be obvious what should be locked, mostly to be on the
safe side.

[2582] Let the msgq connect to itself

Once the config manager is ready, connect to itself to get the config.
Not covered by unit tests, since it is part of startup routine and
requires interaction with other modules, but lettuce succeeds, which
means msgq itself works.

[2582] Synchronisation when the config manager connects

[master] Merge branch 'trac2498'

Conflicts:
src/lib/dns/gen-rdatacode.py.in

[2317] Minor: removed a spurious whitespace.

[2587] use sockaddr_storage

[2582] Signal connection of config manager

Let the config manager call a function when the config manager
subscribes. The function is empty in this commit, but it'll be used
later on.

[2317] Pass a string object to is_any_of() instead of char* value.

This is a workaround for the warnings on Debian6: "array subscript is
above array bounds".

[2572] Make minor comment updates

[2225_statistics] do imp.reload(counters) in setUp() method instead of in setUpClass() method

Because it was found that reloading in setUp() method was not enough
in another environment.

[2225_statistics] Revert "[2225_statistics] use timedelta.total_seconds() instead of redundant computation"

This reverts commit 31b16f3568b95073fc15b68e435215572f25c5b8.

This is because timedelta.total_seconds() is not supported yet on Python3.1.

[2225_statistics] rename counter.py to counters.py

[2435] Elaborate note about unsupported RRTypes

[2435] Add a note that some records may not be found by the datasrc RRsetCollection

[2435] Throw an exception if the database RRsetCollection implementation is used after commit()

[2435] Updating after getting RRsetCollection on DatabaseUpdater must fail

[2435] Return a isc::datasrc::RRsetCollectionBase in ZoneUpdater::getRRsetCollection()

[2435] Introduce an abstract datasrc::RRsetCollectionBase class

[2435] Revise RRsetCollection constructor doc

[2435] Make ZoneUpdater::getRRsetCollection() return a RRsetCollectionBase&

... because in case of some updaters, the returned RRsetCollectionBase
is very well a singleton for that updater instance, and it does not make
sense to return multiple RRsetCollections for it.

[2435] Add documentation for RRsetCollectionPtr

[master] Merge branch 'trac2576'

[2572] convert OpenError to Unexpected in the stream ver. of pushSource.

with the change for #2572 this version can now have this exception, and
since it's intended to be hidden within the lexer and input source classes,
we need to convert it.  doc/tests are also added/updated.

[2572] avoid direct comparison between size_t and streampos

[2572] handled solaris oddity: catch uncommon/buggy results of seekg/tellg.

Merge branch 'master' of ssh://git.bind10.isc.org/var/bind10/git/bind10

[master] place internal exception class in a separate namespace.

this is the same workaround with particular versions of clang++ (on a
particular instance of OS X box) as that we did for isc.log wrapper.
committing it at my discretion.

[master] removed a garbage from a comment.

It was introduced at commit 71eee6e2 for no apparent reason.  I believe
it's a merge error or simple typo.  Directly committing.

[2587] move deletes

So members aren't deleted if the code doesn't even reach the call to set them again

[2587] Fix a memleak in a unit test

[2587] Fix for unit test; set sa_len if available

On NetBSD4, getnameinfo() used in a test does a strict check on the sa_len field, which isn't set by asio. So this fix sets it if the field is there.

[2582] Run the MSGQ poller in separate thread

This will be needed in the following work. If we run in the same thread
and tried to connect to self by the cc library, it would deadlock, since
the library blocks waiting for answer.

Also, small cleanup of shutdown is included.

No functional change should be observable.

[2597] Documentation updated.

[master] editorial: removed an unnecessary semicolon

commiting at my discretion.

[2597] Server-id in DHCPv4 is now stored to a file.

[2597] Server-ID is now stored and read from a file.

[master] Merge branch 'trac680'

[680] Update text and added code comments

[2498] Add another HINFO test

and remove some code that should not have been committed

[2571] Explain unit test for character 255.

[2571] a small optimization: avoid calling strlen() every time in loop

[2571] editorial cleanups: &/* position, use camel name for func; const

[2571] Change from char to int when we need special character values.

In cases where we use values outside of normal character range (-128
to 127 or 0 to 255) to indicate special things, like EOF, we need to
insure that we can hold these values. To do that, we use an int.

It turns out that the default type conversions also meant that our
JSON conversion from strings would break in some cases. A test was
added to show that, and to confirm the fix.

[master] Added missing virtual destructor to Lease class

[2582] Missing log message added

A leftover from logging switch, there was a call to write to stdout,
instead of logging.

[680] Update 'no default' bindctl message

[2317] Eliminated cppcheck errors.

[2317] Added changes as a result of the review.

[2498] Add a few quoting tests for toText() in HINFO and NAPTR

[2498] Add operator= and use scoped_ptr in HINFO and NAPTR

[2573] added python wrapper for ZoneLoader extension