BIND 10 trac2109, updated. 054304b9814bcda55e3c4b096376f14fb3f8c0ef [2109] addressed review comments

[BIND 10 source code commits]

The branch, trac2109 has been updated
       via  054304b9814bcda55e3c4b096376f14fb3f8c0ef (commit)
      from  1982543450f925e2f6b3b0a029b6c9425788854e (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 054304b9814bcda55e3c4b096376f14fb3f8c0ef
Author: Jelte Jansen <jelte at isc.org>
Date:   Mon Sep 17 19:52:05 2012 +0200

    [2109] addressed review comments
    
    - fix doxygen and comments
    - inclusion order fix
    - remove dead line from makefile

-----------------------------------------------------------------------

Summary of changes:
 src/lib/datasrc/Makefile.am                        |    1 -
 .../datasrc/memory/tests/zone_finder_unittest.cc   |   14 ++++++++++----
 src/lib/datasrc/memory/zone_finder.cc              |    2 +-
 3 files changed, 11 insertions(+), 6 deletions(-)

-----------------------------------------------------------------------
diff --git a/src/lib/datasrc/Makefile.am b/src/lib/datasrc/Makefile.am
index aa93b77..bd96838 100644
--- a/src/lib/datasrc/Makefile.am
+++ b/src/lib/datasrc/Makefile.am
@@ -64,7 +64,6 @@ libb10_datasrc_la_LIBADD = $(top_builddir)/src/lib/exceptions/libb10-exceptions.
 libb10_datasrc_la_LIBADD += $(top_builddir)/src/lib/dns/libb10-dns++.la
 libb10_datasrc_la_LIBADD += $(top_builddir)/src/lib/log/libb10-log.la
 libb10_datasrc_la_LIBADD += $(top_builddir)/src/lib/cc/libb10-cc.la
-#libb10_datasrc_la_LIBADD += memory/libdatasrc_memory.la # convenience library
 libb10_datasrc_la_LIBADD += $(SQLITE_LIBS)
 
 BUILT_SOURCES = datasrc_config.h datasrc_messages.h datasrc_messages.cc
diff --git a/src/lib/datasrc/memory/tests/zone_finder_unittest.cc b/src/lib/datasrc/memory/tests/zone_finder_unittest.cc
index 15fec59..08265f8 100644
--- a/src/lib/datasrc/memory/tests/zone_finder_unittest.cc
+++ b/src/lib/datasrc/memory/tests/zone_finder_unittest.cc
@@ -13,12 +13,19 @@
 // PERFORMANCE OF THIS SOFTWARE.
 
 #include "memory_segment_test.h"
+
+// NOTE: this faked_nsec3 inclusion (and all related code below)
+// was ported during #2109 for the convenience of implementing #2218
+// In #2218 the NSEC3 test code in this file is expected to be finalized.
+// In #2219 the original is expected to be removed, and this file should
+// probably be moved here (and any leftover code not handled in #2218 should
+// be cleaned up)
 #include "../../tests/faked_nsec3.h"
-#include <datasrc/data_source.h>
-#include <testutils/dnsmessage_test.h>
 
 #include <datasrc/memory/zone_finder.h>
 #include <datasrc/memory/rdata_serialization.h>
+#include <datasrc/data_source.h>
+#include <testutils/dnsmessage_test.h>
 
 #include <boost/foreach.hpp>
 
@@ -57,6 +64,7 @@ const char* const zzz_hash = "R53BQ7CC2UVMUBFU5OCMM6PERS9TK9EN";
 // A simple faked NSEC3 hash calculator with a dedicated creator for it.
 //
 // This is used in some NSEC3-related tests below.
+// Also see NOTE at inclusion of "../../tests/faked_nsec3.h"
 class TestNSEC3HashCreator : public NSEC3HashCreator {
     class TestNSEC3Hash : public NSEC3Hash {
     private:
@@ -685,8 +693,6 @@ TEST_F(InMemoryZoneFinderTest, glue) {
  * \brief Test searching.
  *
  * Check it finds or does not find correctly and does not throw exceptions.
- * \todo This doesn't do any kind of CNAME and so on. If it isn't
- *     directly there, it just tells it doesn't exist.
  */
 void
 InMemoryZoneFinderTest::findCheck(ZoneFinder::FindResultFlags expected_flags,
diff --git a/src/lib/datasrc/memory/zone_finder.cc b/src/lib/datasrc/memory/zone_finder.cc
index 994b0f9..3768602 100644
--- a/src/lib/datasrc/memory/zone_finder.cc
+++ b/src/lib/datasrc/memory/zone_finder.cc
@@ -44,7 +44,7 @@ namespace {
 /// \param rdataset The RdataSet to create the RRsetPtr for
 /// \param rrclass The RRClass as passed by the client
 ///
-/// Returns an empty TreeNodeRRsetPtr is either node or rdataset is NULL.
+/// Returns an empty TreeNodeRRsetPtr if node is NULL or if rdataset is NULL.
 TreeNodeRRsetPtr
 createTreeNodeRRset(const ZoneNode* node,
                     const RdataSet* rdataset,