[2353] Indent code

[2372] further simplification: don't need to save/restore options.

paren_count has all information, and we can handle it completely locally.

[2372] simplification: we actually don't need END_OF_LINE option.

it can be completely local in Start::handle().

[2372] handle crlf class

[2372] make sure last_was_eol becomes false in EOF even with an error

[2372] handle the case of comment after '('

[2372] cleanup: we now don't need to pass options to state::handle()

[2372] localize disabling options in (); we can do this without mutable state.

[master] changelog for #2356

[2372] handle comments in the start state

[master] Merge remote-tracking branch 'origin/trac2356'

[trac2305] change enable to disable for !verbose off

[trac2305] remove repeated word

[2342] Have unit tests create database schema on the fly

The unit tests now create the schema before each test and destroy
it afterwards.  This will be more flexible than needing to update
the schema on test machines each time.  The drawback is the need
to keep the database creation script in step with the hardcoded
set of SQL statements in the test program.

[2414] Minor corrections to comments and spacing between operators

[2371] added notes about the two mode of error handling: retval and exception.

[2371] cleanup: removed now-unnecessary include

[2384] Provide a note about used data type

[2384] Adjust other tests to match

Obviously, the master loader now can accept the TTL units. This is good
news, but we need to adjust the test that checked it throws.

[2384] Implement the units in TTLs

[2384] Tests for units in TTL text

They fail, since this is not implemented yet.

[2417] Initialize standard option definitions and keep them in storage.

[2417] Added missing header and corrected typo.

This fixes build issues discovered during code review.

[2305] initial addition of bindctl section

[2417] Append configured DHCPv6 options to server's response.

[2353] Test BoB.reap_children

[2342] Move all SQL statement definitions to head of file

It is easier to add a new statement to the head of the file rather
than searching through the file for the method in which the prepared
statements are created.

[2342] Add getLease6 selecting by DUID, IAID and Subnet ID

[2369] Add InputSource::mark() as a combination of saveLine() and compact()

[2369] Constify variables in various places

[2369] Read data from the test file to check the InputSource

[2369] Use the more specialized std::ifstream instead of std::fstream

[2369] Rename non-existent filename in test

[2342] Add selection of Lease6 by DUID and IAID

[2372] handle non-closed parenthesis error.

[2372] handle unbalanced parentheses.

[2372] covered a minor case of initial_ws after skipping a newline

[2372] internal refactoring: pass original options at the time of start

this will make the overall interface simpler.

[2372] more parentheses support

[2372] use the END_OF_STREAM constant instead of hardocded condition of '< 0'

[2372] more conflict resolution that git didn't detect.

[2372] more conflict resolution

[2371] more comment wording adjustments

[2372] initial parentheses support

[2372] handled spaces.

handle() signature was extended so it can refer to and update lexer
options.

[2372] lexer state classes, some initial defs and tests

Fixing merge conflicts:
src/lib/dns/master_lexer.cc

[2372] Merge branch 'trac2371' into trac2372

[2371] catch an error in opening a source file and gives feedback to caller.

[2371] adjusted test name/comment to match the method name change of open/close

[2371] renamed open/close() {push,pop}Source() as suggested in review.

[2371] define END_OF_STREAM within InputSource to avoid duplicate def.

unfortunately this requires another usual workaround in the test, so
it was also made.

[2371] use the real InputSource class instead of the temporary fake version.

note: right now build fails due to the duplicate definition of
InputSource::END_OF_STREAM.  It will be fixed in the next commit.

[2371] Merge branch 'trac2369' into trac2371

[2417] Create basic set of stdandard DHCPv6 option definition instances.

[2414] All outstanding changes after review.

[2356] try -Wl,-R first to make it safer for build-time failure

Merge #2383

Add a new constructor to name, one that takes an optional origin and completes
relative names with it.

Also clarify the '@' handling as a side effect.

[2356] fixed typo: use ISC_RPATH_FLAG instead of hardcoded "-R"

[2414] 2 new tests implemented (ManySolicits, ManyRequests), comments added

[2414] Changes after review.

[2414] Memory leaks in AllocEngine tests fixed.

[2369] Move exceptions to the top of class

[2369] Throw OpenError when opening the input file fails

[2369] Add a file test as well

[2371] fixed a trivial description error in doc

[2371] doc wording update as suggested in review

[master] update Changelog for merge of #2213

[2414] {first,last}AddrInPrefix not handles invalid prefix lengths

[2414] DHCP6_SUBNET_SELECTED log message fixed

[2414] Compilation fix for Solaris

[2414] Fix in checkLease method

[2414] Allocation process is now logged properly.

[2383] Clarify @ in names

If it is alone, it means the origin (or root, if the absolute-name
constructor is used). If it is not alone, it is taken verbatim, without
any special meaning.

Add some tests to verify this.

Remove unused enum value. It was a relict from non-exact port of bind9
code.

[2414] Tests for selectSubnet() implemented.

[master] Merge branch 'trac2213'

Conflicts:
src/bin/auth/command.cc

[master] Fix leaks in AllocEngine tests reported by Valgrind

[2298_1] use sorted() instead of using sort() redundantly

[2371] Make minor comment updates

[2369] Explain how compact() affects ungetAll()

[2369] Distinguish between EOF and failures in getChar()

[2369] Use a const variable's value for END_OF_STREAM state

[2369] Initialize buffer_pos_ with 0

This is equivalent code to the former. The former was written such to
drive the point that buffer_pos_ is by default pointing at the end of
the buffer_ vector.  At construction, the buffer_ vector is empty, so
both work.

[2369] Constify name_

[2369] Rename exception to UngetBeforeBeginning

[2369] Rename files to master_lexer_inputsource.{cc,h}

[2416] Wait for AUTH_LOAD_ZONE in lettuce test

For the SOA=1239 case, lettuce (instead of waiting for a new
AUTH_LOAD_ZONE message) waits for the last output AUTH_LOAD_ZONE
message because the last SOA=1238 case generated a AUTH_LOAD_ZONE that
was not skipped past in RunningProcess._wait_for_output_str().  So it
immediately matches the last AUTH_LOAD_ZONE and continues right away
which causes it to query Auth prematurely.

[2298_1] style: folded long lines

[2414] Test for REQUEST and LeaseMgr storage implemented.

[2414] New tests for Dhcpv6Srv implemented, old ones refactored.

[2414] Dhcp6SrvTest.SolicitBasic updated to test real address allocation

[2356] share the rpath m4 function with examples in the top level configure.ac

[2356] more note about rpath, in README.

[2356] added some note about rpath

[2356] introduce the helper m4 macro to identify -R variant linker options

extracted from the top-level configure.ac.

[2356] set the saved LIBS correctly

[2356] corrected ressoring LDFLAGS.

we should have used _SAVED; also we need to make a copy of LDFLAGS (in
LDFLAGS_SAVED) before extending it.  it now does both of these correctly.

[2356] wording fix in an error message

[2356] add BOOST_CPPFLAGS for checking BIND 10 common .h and lib

util/buffer.h needs a boosst header file, so we make sure it's in the path

[2356] renamed CPPFLAGS_SAVES to _SAVED for consistency.

this is editorial; no behavior change.

[master] Changelog entry for #2318.