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