BIND 10 #423: simple master file loader

BIND 10 Development do-not-reply at isc.org
Thu Dec 9 06:39:31 UTC 2010


#423: simple master file loader
--------------------------------+-------------------------------------------
      Reporter:  jinmei         |        Owner:  UnAssigned           
          Type:  task           |       Status:  reviewing            
      Priority:  major          |    Milestone:  y2 12 month milestone
     Component:  DNSPacket API  |   Resolution:                       
      Keywords:                 |    Sensitive:  0                    
Estimatedhours:  0.0            |        Hours:  0                    
      Billable:  1              |   Totalhours:  0                    
      Internal:  0              |  
--------------------------------+-------------------------------------------
Changes (by jinmei):

  * owner:  jinmei => UnAssigned
  * status:  new => reviewing


Comment:

 branches/trac423 is ready for review.

 This is a relatively big among the tasks of this sprint, but I believe the
 main part of code is easy to understand.

 The most tricky (and perhaps controversial) part would be the supplemental
 !MasterLoadCallback class.  I introduced this so that we can accept
 variuos kinds of callbacks (functors/functions) transparently, while still
 avoiding relying on  boost.function.  If this part of the code is
 difficult to understand or the reviewer doesn't agree with the idea of
 "in-house" version of boost.function, we could simply replace it with the
 following typedef:
 {{{
 typedef boost::function<void(RRsetPtr rrset)> MasterLoadCallback;
 }}}
 and move forward, while discussing whether to rely on boost.function in
 our public header files project-wide.

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


More information about the bind10-tickets mailing list