[2377] Use rdata::compare to check equality of Rdata

[2377] Pass string by reference

It was obviously forgotten (and cppcheck complained).

[2377] cleanup: Simplify the test callback

[2377] docs: Add forgotten update to the addRRCallback

[2377] Keep a boolean variable for option

The option would be used often. While the produced code would probably
be the same, because the compiler can see it's equivalent, this is more
convenient to use.

[2377] Minor code simplification

[2377] Don't accept quoted TTLs and such

[2377] Small optimizations

[2377] Move a method outside of the class definition

Just to make it easier for some editors to parse.

[2377] Prepare MasterLoader::pushSource for future

When we support $INCLUDE, it should report reasonable place of error,
not :0.

[2377] #include just to make sure

[2377] Report error on !MANY_ERRORS as exception

In this case, the user should be aware of the error as much as possible.
In case of MANY_ERRORS, we can't do this, exception is able to handle
just one error.

[2504] use example.com zone for test

[2377] docs: Reference note about Lenient mode

[2445] Merge branch 'trac2445' of ssh://git.bind10.isc.org/var/bind10/git/bind10 into trac2445

[2377] Provide MasterLoader constructor from stream

Instead of file name. And use it for some of the tests, to avoid
juggling with files.

[2377] Use rrsetCheck() for comparing RRsets

Comparing their toText() might be unreliable. In theory, the toText()
result could be slightly different even for equal RRsets (if their
background implementation is different) and the test fail.

Include a hack for the rrsetCheck interface being incompatible.

[1351] Use TSIG key ring in b10-xfrin

instead of separately configured keys;

with this change, the Xfrin/zones[X]/tsig_key value is not interpreted as a full key string, but as the name to look up in the /tsig_keys/keys list (like acls do)

[2377] Fix BasicRRset::getRRsig() documentation

It claimed it throws every time and returns NULL at the same time. It is
impossible and the first one is not true.

[master] ChangeLog for trac 2404

[2404] Merge branch 'master' into trac2404

[2404] Alter memfile "getName()" to return "memory"

This reflects the fact that at the moment, the data is stored in
memory and not in a backing file.

[2404] Minor changes as a result of review

[2445] Remove LogBuffer singleton

LogBuffers are now members of BufferAppender instances, and the singleton has been removed. LoggerManagerImpl keeps them (if any) around while processing a new specification, and flushes them when done.

[master] ChangeLog for trac 2396

[2396] Merge branch 'master' into trac2396

[2445] Address smaller points in logbuffer code

- update README
- copy vector of event on flush
- only use very basic functions in case of flush-on-destruct

[2445] Rename log_buffer and its namespace

To prevent accidental outside use, all LogBugger (and BufferAppender) code is in the namespace isc::log::internal now, and the files have been renamed to _impl

[2157] fix indent

[2377] style fixes: folded long lines, constify

[2377] hide test definitions in an unnamed namespace.

[2377] constify

[2377] style fix: don't use result of x&y as a boolean directly

[master] fix typo and standardize one spelling

in docs. trivial. no review.

[master] include notify's source IP in ZONEMGR_UNKNOWN_ZONE_NOTIFIED log msg.

okayed on jabber.

[master] Merge branch 'trac2442'
with fixing Conflicts for:
src/lib/dns/gen-rdatacode.py.in
src/lib/dns/tests/rdata_txt_like_unittest.cc

[2442] in CharStringTooLong what() msg, clarify it includes an extra octet.

[2377] style fix: added a blank line between method defs for readability.

[2445] fixed a typo

[master] Added ChangeLog entry for #2479.

[2491] Changes as a result of the review.

[2225_statistics] change the variable names

`number` to `concurrency` and `cycle` to `number`

[2225_statistics] correct the variable names from the counter object

[2225_statistics] remove the unnecessary parameters

[2225_statistics] manipulate the counter via incrementer and getter methods

[2225_statistics] add checking validation of statistics data in case of no TEST_SPECFILE_LOCATION

[2225_statistics] add a new test case TestCounter0() in case of no TEST_SPECFILE_LOCATION

[2225_statistics] add the new test methods according to the new interface:

test_perzone_counters()
test_xfrrunning_counters()
test_socket_counters()
test_unknown_item()

[2225_statistics] remove the obsolete test methods:

test_perzone_counters()
test_xfrrunning_counters()
test_unixsocket_counters()
test_perzone_timers()
test_ipsocket_counters()

[2225_statistics] remove the unnecessary parameters

[2225_statistics] add Counter() constructors according to the update

[2225_statistics] remove the unnecessary setUpClass() methods

[2225_statistics] update the docstring

[2225_statistics] correct the variable names from the statistics class

[2225_statistics] change the default statistics spec in case of a spec file specified in the constructor

[2225_statistics] add _Statistics() class having default statistics data set

[2503] Insert the origin name into the NSEC3 tree by default

[2513] Merge branch 'master' into trac2513

[2396] Modified IOAddress tests in light of review

[2503] Add testcase that reproduces the problem

[2504] Fix getClosestNSEC() when tree.find() results in SUBDOMAIN

When tree.find() results in SUBDOMAIN, the node_path's top contains the
previous node.  Calling previousNode() yet again will return its
further previous node.

[2270] ChangeLog updated.

[2225_statistics] rename _Counter() to Counter() and change its interface

[2225_statistics] rename _stoptimer() to stop(), change its interface, and update the description

It doesn't raise DataNotFound if the timer isn't started but it raises if the
timer is stopped, which isn't defined in the spec file.

[2225_statistics] rename _starttimer() to start() and change its interface

[2225_statistics] rename _getter() to get() and change its interface

[2225_statistics] rename _decrementer() to dec() and change its interface

[2225_statistics] rename _incrementer() to inc() and change its interface

[master] Merge branch 'trac2479'

[2504] Add testcase that reproduces the problem

This test currently fails with an assertion.

[2353] Check that BoB.start_msgq() calls isc.cc.Session() and group_subscribe()

[2353] Test BoB.startup() when socket already exists

[2353] Remove unused assignments

[2353] Rename variable

[2353] Remove run_under_unittests hack from BoB.start_cfgmgr()

[2353] Make DummySession() return a real message in group_recvmsg()

[2353] Restore isc.config.ModuleCCSession during tearDown()

[2225_statistics] remove the unnecessary parameters

[2225_statistics] remove the unnecessary Counter() class

[2225_statistics] remove _parse_stats_spec()

[2353] Restore time.time() and time.sleep() during tearDown()

[2353] Unify verbose and non-verbose cases

[2225_statistics] remove the unnecessary functor creators

[2353] Don't call real sleep() in mock function

[2445] put LogBuffer::flush() call into processEnd()

And clean up the process() calls and processSpecification() method (which should mostly look like they were now)

[2445] Address the smaller points in the review

[master] Merge branch 'trac2382'
with fixing Conflicts for
src/lib/dns/gen-rdatacode.py.in
src/lib/dns/rdata.cc
src/lib/dns/rrparamregistry-placeholder.cc
src/lib/dns/tests/rdata_hinfo_unittest.cc
src/lib/dns/tests/rdata_txt_like_unittest.cc

[2442] updated comment for string::operator[] when it's empty

this is actually ensured by the C++ standard lib spec.

[2442] use std::string instead of POD array for constructing obj of DDD.

[2445] Don't replace appenders in processSpecification()

slightly changed outer layer process() call so it isn't necessary anymore

[2270] Dhcp4ConfigParser members are now private.

[2270] Changes after review

- added boundary checks for Uint32Parser
- fixed #include order
- Class renamed to Dhcp4ConfigParser
- added extra test for Uint32Parser
- Many Doxygen fixes and clean-ups

[2270] Fix for repeated reconf in dhcp6 (old defaults are now removed)

[2270] Fixes in Doxygen comments in src/bin/dhcp6

[2445] minor cleanups: brace position, indent, use 'protected' in test class

[2445] do explicit check for NULL against a pointer

[2353] Check if msgq_socket_file is carried over in BoB.startup()

[2445] constify, unify two short lines, spacing

[2353] Change assertion type for dict

[2353] Explain why we throw after 15 recv()s, in a comment