[2676] some more unrelated editorial cleanups.

just happen to notice them in review, and since these files will be updated
this is probably a good opportunity to fix them with the branch.

[2676] editorial cleanup: removed unnecessary backslash

[master] remove generated file from dist tarball

Don't include example.zone test data file in dist tarball.
It is generated from an .in file via ./configure.

This was noticed by emhn during Debian packaging.
(distclean created a source different than extract tarball.)

I didn't get okay, but I mentioned my patch on jabber.
I tested on one system. This seems to be trivial.

[2676] remove the comment about using "protected" member in __notify_update.

the revised code eliminated its use as a side effect.

[2676] indentation fix

[master] fix case in acronym

(noticed when another fix was suggested for same line)

Some starting questions for recursive resolver research.

[2641] Fix __init__() args

Typo fixes

[2669] Update check_valgrind.h documentation

[2676] (minor) Explain a FIXME comment

[2676] Translate to the correct exception

If the remote is not there, it's not "Data error"

[2676] Remove unused variable

[2676] (minor) Comment clarification

[2676] Error when missing ConfigManager

In the code before refactoring, when the CfgMgr wasn't there, it did a
SessionTimeout. Now the RPCRecipientMissing is raised instead, but as it
is subclass of RPCError, we need to exclude it from the error ignoring.

Actually, the error ignoring itself is very questionable thing, but it
is as it was before.

[2676] Make the try-except block smaller

As the rpc_call is the only place the exception can be raised from, this
is equivalent, just cleaner.

[2676] Fix FIXME

According to the notes in the review, we need to wait for the answer.

[2676] Reuse constants for remote loadzone command

[2676] Remove the notion of timeouts from the message docs

Remove the part of the DDNS_UPDATE_NOTIFY_FAIL message description,
complaining about how bad the timeouts are. The timeouts should be
solved by now and should happen only in really rare circumstances. If
the other module is not there, it should return error right away.

[2676] Reorder and tweak exception to be closer to original

In case the auth wasn't there before the refactoring, it did a session
timeout, which resulted in DDNS_(START|STOP)_FORWARDER_ERROR log
message. So we handle the RPCRecipientMissing in the same branch (and
list above the RPCError, since it is a subclass of that).

[2676] (minor) Comment tweaks

[2676] Use constant for success code

Use CC_REPLY_SUCCESS instead of hardcoding 0.

[2641] Test /login API call

[2641] Rename test

[2641] Test /users-exist API call

[2641] Add test for _is_session_valid()

This commit was actually to update the statement inside
_is_session_valid() to be a None check, but I've written a unittest for
it anyway.

[2641] Test that _try_login() makes the right API call to cmdctl

[2641] Test that login_to_cmdctl() calls _have_users()

[2641] Test that _have_users() makes the right API call to cmdctl

[2641] Add tests for _have_users()

[2689] eliminated the need for threads in handler test.

needed to make some non trivial changes.  hopefully it preserves the
intent of the test.

[2689] eliminate threads in test_start.

some refactoring to Stats class (behavior shouldn't change) and mock classes
were introduced.  shutdown case was extracted to a spearate test function.

[2689] simplify: remove unused param for send_command(), remove send_shutdow.

this is just a refactoring for further changes.  shouldn't change the
behavior.

[2676] editorial: folded a long line

[2669] piggyback style/editorial cleanups: const, spacing

[2669] Use valgrind-check in fork/thread tests

Similar to how it is checked for death tests; if valgrind is detected (when possible), parts of tests are skipped

[2641] Update doc for _try_login()

[2641] Add doc for _have_users()

[2641] Add doc for get_num_users()

[2641] Update if statement

This doesn't change any behavior, but changes the syntax used.

[2641] Move get_user_info() call to its own unittest

[2641] a minor piggy-back style cleanup: removed space in default param spec.

[2641] From bindctl, call /users-exist in cmdctl to check if users exist

Note: There are no tests for any HTTP API calls made to cmdctl. Adding
this framework as part of this bug would go way beyond the estimate.  I
suggest creating another bug to go through bindctl and add missing
unittests for cmdctl RPCs.

[2641] Handle /users-exist API call in cmdctl

Note: There are no tests for any HTTP API calls in cmdctl. Adding this
framework as part of this bug would go way beyond the estimate.
I suggest creating another bug to go through cmdctl and add missing
unittests for the HTTP API.

[2641] Add method to get the number of users

[2641] Move populated cmdctl-accounts.csv to testdata directory

[2768] Interface of the ModuleCCSession::rpcCall

[2676] Note why Cmdctl is not converted.

Similar to previous commit, there's a hack in cmdctl that doesn't allow
it now.

[2676] Note why XfrIn is not converted

There's a hack in XfrIn that prevents straight-forward conversion.
Adding the want_answer parameter for now as a temporary measure, until
the thing is fixed.

[2676] Convert stats to use rpc_call

Most parts are converted, with one exception.

There's a place where many messages are sent at once and then it waits
for all the answers. This allows the other modules to process the
commands in parallel. The rpc_call is not flexible enough to allow this
(we'll need the rpc_call_async for that), so we stay with the original,
just adding the want_answer parameter.

[2676] Convert ZoneMgr to use the rpc_call

Just a plain conversion.

[2676] Convert DDNS to use rpc_call

Just a straight-forward conversion.

Unify common code

[master] Merge branch 'trac2225_statistics_4'

[2225_xfrout] add directory names of "zone", "socket" and "unixdomain" into the man page

[2225_xfrout] update "item_description" of "zone", "socket" and "unixdomain" in spec

[master] corrected typo: s/NSEC/NSEC3/

should be trivial, directly committing.

[2440] noted some differences from BIND 9

[2440] update documentation of RdataSet::create().

[2440] handle varying TTL cases.

the behavior was changed slightly: on second thought it seemed to make
more sense to adopt the smallest TTL.  An existing test case was removed
because it's now covered in more comprehensive tests for this change.

[2440] a bit of cleanup: unify 2 versons of RdataSet::create into single one.

[2440] check RR type consistency in merge mode RdataSet::create.

[2440] handle duplicate RDATA in RdataSet create appropriately.

[2440] refactoring: unify two versions of create().

[2440] mostly completed merge version of create().

with lot of duplicate code right now, and some corner cases are not yet
handled.

[2440] merge version of RdataSet::create.

all basic cases are covered and tested.  still some code duplicate.
ignoring corner cases.

[2440] documentation update for extensions to RdataEncoder with merge support.

[2440] update add methods of RdataEncoder to return bool depending on duplicate

This is necessary to extend RdataSet class so it calculates the correct
number of RDTA/RRSIGs excluding duplicates.

[2676] Use dict, not keyword arguments

As it turns out, it is not as flexible as needed (it wouldn't be
possible to pass no arguments, it would not be possible to specify
parameters of name `class` easily, and so on).

[2225_statistics_4] calculate an entire server's counts even though they are zero

As well as ffd4a283b18f8eaa453712e1e615e2440b12aa0d, in the platform (FreeBSD8,
amd64), the timer counters can be zero. However get_statistics() in counters.py
didn't calculate an entire server's counts if they are zero. This change is
added so that it calculates counters even though they are zero.  An new
unittest is added to check that. This change fixes the issue reported in #2741.
For details about errors related to this change, see the following build farm
reports.

 http://git.bind10.isc.org/~tester/builder/BIND10-cppcheck/20130214075001-FreeBSD8-amd64-GCC/logs/unittests.out
 http://git.bind10.isc.org/~tester/builder/BIND10-cppcheck/20130206173746-FreeBSD8-amd64-GCC/logs/unittests.out
 http://git.bind10.isc.org/~tester/builder/BIND10-cppcheck/20130211135501-FreeBSD8-amd64-GCC/logs/unittests.out

[master] Merge branch 'trac2679'

use explicit operator precedence; reviewed via jabber

[master] update copyright year in guide and messages manual

Update copyright year for guide.

Update copyright year for messages manual.
Also remove the paragraph introducing bind10 from the messages manual
(it was incomplete, so misleading).
These were in the script)

This is trivial, so no review.

[master] remove TODO

This TODO was handled a few minutes ago.

[master] editorial: removed spaces after EOL

Update the sponsor acknowledgment section in the BIND 10 Guide to include all of the sponsors.

[2701] Added the key_from_key.h header file.

[2701] Added missing const to the function returning DUID.

[2676] Document the rpc_call

[2701] Added warning about undefined behavior when using invalid option definition.

[2701] Moved the KeyFromKey class to the common header.

[master] Add init.pyc to CLEANFILES to make distcheck pass

[2669] Use vector instead of raw array in unit test

[2701] Added composite index to the container holding leases.

This change applies to the Memfile backend and is intended to significantly
improve its performance.

[2676] Error handling in the rpc_call

Merge branch 'trac2281'

Conflicts:
src/lib/datasrc/memory/memory_messages.mes
src/lib/datasrc/memory_datasrc.cc

[2676] Further tests for rpc_call

An unusual case and a malformed answer.

[1676] Basics of the rpc_call in python

Successful case. Small fixes of the tests.

[2676] Test for rpc_call

Just the simplest one for now, with success

[2676] Store the want_answer in fake session

So we are both compatible and we can check it in the tests.

Merge remote-tracking branch 'refs/remotes/origin/master'

[2440] suppress duplicate Rdata in merge. extend test cases more.

[2440] generalize the merge-start initialization using RdataReader.

[2440] update encode() considering old RRSIGs. start() still uses hardcoding.

[2440] extended RdataEncoder so it can begin with existing ecoded data.

some test cases are hardcoded, not all cases are covered yet.

[2669] Initialize memory in tcp unit test

[2669] Make valgrind silent for forked() tests

To supress leaks for data on parent's stack

[master] bump version

(note this is not the rc version, just the version for master)

[master] add date stamp for the 1.0.0-rc

[master] fix misspellings