comments and docmentation

git-svn-id: svn://bind10.isc.org/svn/bind10/branches/trac439@3933 e5f2f494-b856-4b98-b285-d166d9295462

src/bin/auth/auth.spec.pre.in

@@ -6,7 +6,7 @@
       { "item_name": "use_memory_datasrc",
         "item_type": "boolean",
         "item_optional": true,
-        "item_default": true
+        "item_default": false
       },
       { "item_name": "database_file",
         "item_type": "string",

src/bin/auth/auth_srv.cc

@@ -103,10 +103,11 @@ public:
     /// Hot spot cache
     isc::datasrc::HotCache cache_;
 
+    // enable memory data source
+    bool use_memory_datasrc_;
     /// Currently, MemoryDataSrc isn't a derived class of AbstractDataSrc
     /// because the interface is so different, so we use a separate variable
     /// here.
-    bool use_memory_datasrc_;
     isc::datasrc::MemoryDataSrc memory_datasrc_;
 };
 
@@ -330,6 +331,8 @@ AuthSrvImpl::processNormalQuery(const IOMessage& io_message, Message& message,
     }
 
     try {
+        // If a memory data source is configured call the separate
+        // Query::process()
         if (use_memory_datasrc_) {
             ConstQuestionPtr question = *message.beginQuestion();
             const RRType& qtype = question->getType();

src/bin/auth/main.cc

@@ -82,7 +82,7 @@ my_command_handler(const string& command, ConstElementPtr args) {
     } else if (command == "shutdown") {
         io_service->stop();
     }
-    
+
     return (answer);
 }
 

src/bin/auth/query.cc

@@ -68,22 +68,22 @@ Query::process() const {
                 impl_->response_.setRcode(Rcode::NOERROR());
                 impl_->response_.addRRset(Message::SECTION_ANSWER,
                             boost::const_pointer_cast<RRset>(db_result.rrset));
-                // fill in authority and addtional sections.
+                // TODO : fill in authority and addtional sections.
                 break;
             case Zone::DELEGATION:
-                // add NS to authority section, fill in additional section.
+                // TODO : add NS to authority section, fill in additional section.
                 break;
             case Zone::NXDOMAIN:
                 impl_->response_.setRcode(Rcode::NXDOMAIN());
-                // add SOA to authority section
+                // TODO : add SOA to authority section
                 break;
             case Zone::NXRRSET:
                 impl_->response_.setRcode(Rcode::NXRRSET());
-                // add SOA to authority section
+                // TODO : add SOA to authority section
                 break;
             case Zone::CNAME:
             case Zone::DNAME:
-                // replace qname, continue lookup
+                // TODO : replace qname, continue lookup
                 keep_doing = true;
                 break;
             // should not happen, catch programming error here.