BIND 10 #2208: Revise InMemoryClient and ConfigurableClientList::configure() using ZoneTableSegment

[BIND 10 Development]

#2208: Revise InMemoryClient and ConfigurableClientList::configure() using
ZoneTableSegment
-------------------------------------+-------------------------------------
                   Reporter:         |                 Owner:  muks
  jinmei                             |                Status:  reviewing
                       Type:  task   |             Milestone:
                   Priority:         |  Sprint-20121023
  medium                             |            Resolution:
                  Component:  data   |             Sensitive:  0
  source                             |           Sub-Project:  DNS
                   Keywords:         |  Estimated Difficulty:  5
            Defect Severity:  N/A    |           Total Hours:  0
Feature Depending on Ticket:         |
  background zone loading            |
        Add Hours to Ticket:  0      |
                  Internal?:  0      |
-------------------------------------+-------------------------------------
Changes (by jelte):

 * owner:  UnAssigned => muks


Comment:

 Could you please provide a few hints for the reviewer next time? For
 instance, why is RRClass now a member of ZoneTable instead of passing it
 from the client like before?

 Isn't that hardcoded RRClass::IN() going to present a problem (given that
 the 'static' datasource also uses in-memory)?

 I think it would be a bit more clean to consistently call arguments and
 members ztable_segment (a few of them are now named the more general
 'segment' in client_list)

 on a smaller note, you have a number of cases where you use
 *somesharedptr.get() which can be written with just *somesharedptr (at
 least in client_list.cc:96 and client_list_unittest.cc:260), but maybe
 more

-- 
Ticket URL: <http://bind10.isc.org/ticket/2208#comment:6>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development