[svn] commit: r3933 - in /branches/trac439/src/bin/auth: auth.spec.pre.in auth_srv.cc main.cc query.cc

[BIND 10 source code commits]

Author: chenzhengzhang
Date: Tue Dec 21 14:17:25 2010
New Revision: 3933

Log:
comments and docmentation

Modified:
    branches/trac439/src/bin/auth/auth.spec.pre.in
    branches/trac439/src/bin/auth/auth_srv.cc
    branches/trac439/src/bin/auth/main.cc
    branches/trac439/src/bin/auth/query.cc

Modified: branches/trac439/src/bin/auth/auth.spec.pre.in
==============================================================================
--- branches/trac439/src/bin/auth/auth.spec.pre.in (original)
+++ branches/trac439/src/bin/auth/auth.spec.pre.in Tue Dec 21 14:17:25 2010
@@ -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",

Modified: branches/trac439/src/bin/auth/auth_srv.cc
==============================================================================
--- branches/trac439/src/bin/auth/auth_srv.cc (original)
+++ branches/trac439/src/bin/auth/auth_srv.cc Tue Dec 21 14:17:25 2010
@@ -103,10 +103,11 @@
     /// 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 @@
     }
 
     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();

Modified: branches/trac439/src/bin/auth/main.cc
==============================================================================
--- branches/trac439/src/bin/auth/main.cc (original)
+++ branches/trac439/src/bin/auth/main.cc Tue Dec 21 14:17:25 2010
@@ -82,7 +82,7 @@
     } else if (command == "shutdown") {
         io_service->stop();
     }
-    
+
     return (answer);
 }
 

Modified: branches/trac439/src/bin/auth/query.cc
==============================================================================
--- branches/trac439/src/bin/auth/query.cc (original)
+++ branches/trac439/src/bin/auth/query.cc Tue Dec 21 14:17:25 2010
@@ -68,22 +68,22 @@
                 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.