[2526] Added universe parameter to OptionIntArray ctor.

[2526] Renamed Option6IntArray to OptionIntArray.

... as this class is going to be used either for V6 or V4 now.

[2157] test only for concerned counters

[2526] OptionInt can be now used for V4 and V6.

[2526] Renamed Option6Int to OptionInt as this will be used for V4 too.

[2526] Added definitions for most of the DHCPv4 std options.

[2157] fix checks for response is TSIG signed or not

[master] Added ChangeLog entry for #2491.

[2157] unify similar test codes

[2157] add a testcase of truncated message

[master] Merge branch 'trac2491'

[2157] unify counter initialization check into a method

[master] Changelog entry for merge of #2445

[2157] Counters::get() now returns ConstElementPtr.

[2377] Check we warn about EOF without EOLN

[2157] update doxygen comment

[2377] Make error message similar to bind9

[2377] Note about SOA

[2503] Replace private zone name with 'example.com'

[master] Merge branch 'trac2445'

[2445] Revert "Remove setLogLevel(TRACE) in createBufferAppender"

This reverts commit 3f9c7bb8a987dc1b4e2a9266ac95cf231d8fe621.

[2157] pass IP versions and transport protocols with enum

[2157] declare MessageAttributes as temporary variable

[2470] use MasterLoader instead of masterLoad for in-memory loading from text.

there's one compatibility issus in unit tests in auth InMemoryQueryTest;
some owner names in the test zone file must be escaped.  It's fixed in this
commit, too.

[2470] added documentation for RRCollator

[2503] Update doxygen comment for renamed arg

[master] Update ChangeLog for #2504

[2470] tested RRCollator with MasterLoader.

[2470] tested the case where rrset callback throws.

[2470] handled some advanced/unusual cases of RRCollator.

[2470] added RRCollator class with some simpler cases of def and tests

[2470] added wrapper interface to create null loader callbacks

this is especially useful for tests.

[2377] Warn on unexpected EOF after error

When there's an error, we expect there'll be EOLN somewhere, so we can
recover. Warn in case there's EOF.

[2377] Minor test code cleanups

[2377] Allow checking if there was an error

[2377] Make the construction exception safe

[2377] Reject load incremental of 0 RRs

As it makes little sense.

[2377] constify

[2503] Rename arguments in protos to match what's used in definitions

[2503] Add doxygen comment for zone_name args

[master] Clear components dict as documented in BoB.kill_started_components()

This issue was detected post-merge, and seems to have been introduced
by commit 86ed7ae9cfb4184f5637a2e478242f0a646ba2e1 in master.

Merge branch 'master' into trac2353

Conflicts:
src/bin/bind10/bind10_src.py.in

[2445] Remove setLogLevel(TRACE) in createBufferAppender

This has a chance that some log messages get lost (DEBUG messages during startup in case -v is not given but the config has DEBUG set)

[1351] Use TSIGKeyRing class in xfrin

Also updated said class (added a find(name) without algorithm)

[1351] Update docstring of set_tsig_key_name()

[1351] Fix xml error in b10-xfrin.xml source

[2491] Fixed a typo pointed out in the code review.

[2541] Documentation and cleanup

don't duplicate the duplicate zone check in sqlite3 accessor's addZone()
added documentation

[master] set DISTCHECK_GTEST_CONFIGURE_FLAG for --with-gtest too

This is for trac #2481. Review via jabber.

[2491] Use .empty() function to check if container is empty.

[2541] add createZone() to DataSourceClient interface

Note: currently called createZone so as not to interfere with an existing addZone() call in MemoryClient.
Instead of pure virtual I made it have a default 'NotImplemented' method.
Also, the DatabaseClient holds the transaction (not the SQLite3Accessor); we could do it on that level but we already have a transaction API on a higher level and it makes a little bit more sense to me to do it there

[2541] Add addZone() to the abstract database accessor base class

[2541] addZone in SQLite3Accessor

[master] pass method param by const reference instead of by object

this will suppress a cppcheck error:
http://git.bind10.isc.org/~tester/builder//BIND10-cppcheck/20121209143001-FreeBSD8-amd64-GCC/logs/cppcheck.out

okayed on jabber.

[2428] Implement popping of sources

At least in the normal case. Error cases must still be handled.

[2428] Test popping the source

[2428] Implement basic inclusion

[2491] Use insert instead of memcpy to append to a vector.

[2445] make initializaation for BufferAppenderTest more exception safe.

by encapsulating the appender in SharedAppenderPtr immediately after the
creation, we can make sure buffer_appender1 won't leak even if the
creation of the second appender fails.

[2445] constify

[2428] Tests for basic $INCLUDE handling

No special cases, no error handling, no domain name after that.

[2428] Generic $DIRECTIVE handler

Decide which directive it is and call corresponding call based on that.
This currently throws NotImplemented for everything except for the
INCLUDE, which will get implemented in the next few commits in this
branch.

[2428] Check the special handling of $

[2546] Changed a missed "deleteLease6" to "deleteLease"

[2546] Fixed some Doxygen warnings

[2377] Warn on unexpected EOF after error

When there's an error, we expect there'll be EOLN somewhere, so we can
recover. Warn in case there's EOF.

[2377] Minor test code cleanups

[2377] Allow checking if there was an error

[2491] Minor: fixed typo and updated comments.

[2377] Make the construction exception safe

[2377] Reject load incremental of 0 RRs

As it makes little sense.

[2546] Correct issues raised by cppcheck

Mostly adding "const" to methods that don't change the class, but
also correcting a few other points that were raised.

[2157] apply review comments

[2546] Unify deleteLease4() and deleteLease6() into a single deleteLease()

[2491] Moved option std option definitions to a separate header.

[2445] Some more cleanups

- renamed test class
- added a subclass of BufferAppender to directly test append()
(and re-enabled the relevant test case)
- minor doxy updates

[2546] Define ClientIdPtr type

This is akin to the DuidPtr and is a shared point to a Client ID.

[2445] Add assert() after dynamic_cast

[2445] rename unit test file as well

[2445] Store string version of log level in buffer

And use those in the case of flushStdout() upon destruction

[2546] Remove LeaseMgr::getLease4(address, subnetid)

As the address is th eprimary key of the table, there can only ever
be at most one lease in the database with a given address.  Additional
selection by subnet ID is not needed.

[2445] rename log_buffer_impl to buffer_appender_impl

Since LogBuffer no longer exists, the file name did not match its content anymore

[2445] Remove LogBuffer class

and pull its functionality into BufferAppender
(one unit test temporarily disabled, might be removed in next commit or updated to get around the protected method it uses)

[2503] Check result of findNSEC3()

[2353] Update comments (recommended by review)

[2353] Rename and document BoB._run_under_unittests

[2353] Unify repeated test code

[2353] Use a test environment instead of an empty one

[2353] Use assertRaises() instead of the excessive code

[2504] Add ChangeLog entry

Merge branch 'master' into trac2504

[2504] Check result of find()

[2377] constify

[2470] Merge branch 'trac2377' into trac2470

[master] perfdhcp test creates template files in a local folder.

The test used to create template files in a testdata folder without write
permissions. That caused 'make distcheck' errors. This regression had
been introduced with the ticket #2479. Okayed on jabber.

[master] Remove indirection in defining size of arrays

cppcheck was complaining about what was effectively:
const int A = 6; const int B = A; std::string array[b];

Changing this to
const int B = 6; std::string array[b];

... works

[2377] Check for EOF on recover from problem

[2377] More broken cases

Some quoted strings where they can't happen and some broken tokens.

[2377] Check the implementation accepts quoted names

This is questionable if it should be allowed (at least vim marks it in
red as error), but bind9 accepts it, so shall we too.

[2377] Document fields of ErrorCase in tests