[bind10-dev] DNS message API: name class is ready for review
JINMEI Tatuya / 神明達哉
jinmei at isc.org
Sat Dec 19 05:16:28 UTC 2009
I now believe the DNS name class and supplemental classes are
sufficiently complete and ready for review. I've created a Trac
ticket for the review cycle as suggested by Shane:
http://bind10.isc.org/ticket/16
The code is available at subversion branch and should be retrievable
(by anyone) by:
svn co svn://bind10.isc.org/svn/bind10/branches/jinmei-dnsmessageapi
Note that since Makefile.in, ./configure, etc are not included in the
branch, anyone who wants to play with the code should create them by
hand, e.g., by:
% aclocal; autoheader; libtoolize; automake --foreign --add-missing --copy; autoconf
before doing the usual steps of "./configure; make".
For unit tests, google tests need to be installed (the library itself
can be built without the tests) using the --with-gtest=PATH switch of
the ./configure script.
For those who are willing to review the documentation but don't have a
doxygen environment, I've copied the generated html files at:
http://www.jinmei.org/bind10/cpp/
According to the suggested review process, I expect Shane to designate
someone as the (main?) reviewer, but any comments from the dev list
subscribers are also welcome.
Finally, some hints for the review process: this is a fairly large
chunk of code and may be too big for a single reviewer. If so, I
guess we can divide it into the following two sets:
- buffer + exception
- name + messagerenderer
The former is independent from the latter, and (I believe) can be
reviewed separately. The latter depends on the former, but hopefully
each class is sufficiently modular and the latter set can be reviewed
without knowing much about the former, too.
Thanks,
---
JINMEI, Tatuya
More information about the bind10-dev
mailing list