i.e. jinmei's patch, see http://bind10.isc.org/ticket/1207?replyto=15#comment:15

note that this is mostly a way to at least get it working, we need to figure out a more permanent solution related to static linking (or stop supporting it in the first place)

The removal of memory_datasrc.cc in b10_auth_SOURCES was premature; zone loading code is still in auth (but memory_datasrc-specific)

Which is now no longer necessary

instead of fighting the API to get empty zones in, actually change the zone in the database and reload

- RAII InMemoryClient's createInstance
- remove dead code
- added more comments

and store it as a scoped ptr

- fixed a few include statements
- amended a few comments

it is no longer necessary to derive it from InMemoryClient, but it can be directly derived from DataSourceClient

The special case is still there, but it's been moved into the general DataSourceConfig class; this makes later generalization easier.

It also allowed the removal of the specialized createAuthConfigParser (the overloaded version with the 'internal' argument), and the special case therein.

See http://bind10.isc.org/ticket/1207#comment:11

The temporary name with the added P is no longer necessary now the original method has been removed.

Made the getZoneCount() method from inmemory client a general datasource client
call (which throws NotImplemented by default), as this does seem useful in other
 cases as well.

Now the work starts to pay off; removed the temporary pointer variable (getInMemoryClientP() still returns a pointer but now directly retrieved from the container), and any direct reference to isc::datasrc::InMemoryClient is removed from au
th_srv (but not from tests as of yet)

Instead of the direct shared_ptr.

Changes needed:
Moved loading of zones in config into the memory_datasrc implementation; at this moment the general API has no method for this (and it may not be necessary; datasources should handle their own config anyway).
This does have one drawback; through the dynamic loading mechanism we currently lose exception type information. So for now specific exceptions such as MasterLoadError end up as 'general' DataSourceErrors. Tests have been updated to reflect this.

Also, I made config handler slightly more lenient; it has hardcoded defaults and will now accept config parts without "class" or "zones"; using .spec -derived values is impractical here with the current config API, but in this case it shouldn't *really* matter; as IN is hardcoded in several places anyway atm.

(temporarily)
In order to add datasource factory, but keep current code working, store both the shared ptr (for reference counting) and the raw pointer it encapsulates. The latter is then used to call query::process().

We use anonymous namespace instead of static before a function.

other binaries in log/unittests also need to link to our local libs first

The test code now reflects the code change.

so that sys-installed bind10 libraries do not interfere with those in the build tree

This should fix the unit tests complaining about wrong error messages.

While debugging, I also found an uninitialized value error that has been present probably since the logger code exists. This has also been fixed.