[svn] commit: r919 - /branches/jinmei-dnsrrset/src/lib/dns/cpp/rrset.h

[BIND 10 source code commits]

Author: jinmei
Date: Tue Feb 23 08:26:59 2010
New Revision: 919

Log:
more documentation

Modified:
    branches/jinmei-dnsrrset/src/lib/dns/cpp/rrset.h

Modified: branches/jinmei-dnsrrset/src/lib/dns/cpp/rrset.h
==============================================================================
--- branches/jinmei-dnsrrset/src/lib/dns/cpp/rrset.h (original)
+++ branches/jinmei-dnsrrset/src/lib/dns/cpp/rrset.h Tue Feb 23 08:26:59 2010
@@ -149,8 +149,8 @@
 ///      "sort" and "search(find)" method?
 ///   - what about comparing two RRsets of the same type?  If we need this,
 ///     should it compare rdata's as a set or as a list (i.e. compare
-///     each %rdata one by one or as a whole)?  c.f. NLnet Labs'
-///     <a href="http://www.nlnetlabs.nl/projects/ldns/doc/index.html">ldns</a>
+///     each %rdata one by one or as a whole)?  c.f. NLnet Labs' ldns
+///     (http://www.nlnetlabs.nl/projects/ldns/doc/index.html)
 ///     has \c ldns_rr_list_compare(), which takes the latter approach
 ///     (seemingly assuming the caller sorts the lists beforehand).
 ///   - BIND9 libdns has some special DNSSEC-related methods
@@ -498,13 +498,16 @@
 /// \brief The \c BasicRRset class is a concrete derived class of
 /// \c AbstractRRset that defines a straightforward RRset implementation.
 ///
-/// designed to be as portable as possible.  performance is a secondary
-/// concern for this class.
-///
-/// We'd use the default implementations for the toWire()
-/// variants as defined in the base class.  These are not fully optimized
-/// for performance, but, again, it's a secondary goal for this generic
-/// class.
+/// This class is designed to be as portable as possible, and so it adopts
+/// the Pimpl idiom to hide as many details as possible.
+/// Performance is a secondary concern for this class.
+///
+/// This class is intended to be used by applications that only need
+/// moderate level of performance with full functionality provided by
+/// the \c AbstractRRset interfaces.
+/// Highly performance-sensitive applications, such as a large scale
+/// authoritative or caching name servers will implement and use a customized
+/// version of derived \c AbstractRRset class.
 class BasicRRset : public AbstractRRset {
     ///
     /// \name Constructors and Destructor