Browse Source

[2218] Simplify code, also shortcutting RRset construction

Mukund Sivaraman 12 years ago
parent
commit
5ba139c688
1 changed files with 6 additions and 9 deletions
  1. 6 9
      src/lib/datasrc/memory/zone_finder.cc

+ 6 - 9
src/lib/datasrc/memory/zone_finder.cc

@@ -689,10 +689,9 @@ InMemoryZoneFinder::findNSEC3(const isc::dns::Name& name, bool recursive) {
             RdataSet* rdataset = node->getData();
             ConstRRsetPtr closest = createTreeNodeRRset(node, rdataset,
                                                         getClass());
-            ConstRRsetPtr next =
-                createTreeNodeRRset(covering_node,
-                                    (covering_node != NULL ?
-                                     covering_node->getData() : NULL),
+            ConstRRsetPtr next = (covering_node == NULL) ?
+                ConstRRsetPtr() :
+                createTreeNodeRRset(covering_node, covering_node->getData(),
                                     getClass());
 
             LOG_DEBUG(logger, DBG_TRACE_BASIC,
@@ -725,11 +724,9 @@ InMemoryZoneFinder::findNSEC3(const isc::dns::Name& name, bool recursive) {
             }
 
             if (!recursive) {   // in non recursive mode, we are done.
-                ConstRRsetPtr closest =
-                    createTreeNodeRRset(covering_node,
-                                        (covering_node != NULL ?
-                                         covering_node->getData() :
-                                         NULL),
+                ConstRRsetPtr closest = (covering_node == NULL) ?
+                    ConstRRsetPtr() :
+                    createTreeNodeRRset(covering_node, covering_node->getData(),
                                         getClass());
 
                 if (closest) {