[svn] commit: r3734 - in /branches/trac408/src/lib/nsas: nameserver_entry.cc resolver_interface.h tests/nameserver_address_store_unittest.cc tests/nsas_test.h zone_entry.cc

[BIND 10 source code commits]

Author: vorner
Date: Mon Dec  6 15:38:05 2010
New Revision: 3734

Log:
Minor fixes

Modified:
    branches/trac408/src/lib/nsas/nameserver_entry.cc
    branches/trac408/src/lib/nsas/resolver_interface.h
    branches/trac408/src/lib/nsas/tests/nameserver_address_store_unittest.cc
    branches/trac408/src/lib/nsas/tests/nsas_test.h
    branches/trac408/src/lib/nsas/zone_entry.cc

Modified: branches/trac408/src/lib/nsas/nameserver_entry.cc
==============================================================================
--- branches/trac408/src/lib/nsas/nameserver_entry.cc (original)
+++ branches/trac408/src/lib/nsas/nameserver_entry.cc Mon Dec  6 15:38:05 2010
@@ -177,7 +177,7 @@
             family_(family),
             type_(type)
         { }
-        virtual void success(shared_ptr<AbstractRRset> response) {
+        virtual void success(const shared_ptr<AbstractRRset>& response) {
             time_t now = time(NULL);
 
             Lock lock(entry_->mutex_);

Modified: branches/trac408/src/lib/nsas/resolver_interface.h
==============================================================================
--- branches/trac408/src/lib/nsas/resolver_interface.h (original)
+++ branches/trac408/src/lib/nsas/resolver_interface.h Mon Dec  6 15:38:05 2010
@@ -46,7 +46,8 @@
         class Callback {
             public:
                 /// \short Some data arrived.
-                virtual void success(boost::shared_ptr<isc::dns::AbstractRRset>
+                virtual void success(
+                    const boost::shared_ptr<isc::dns::AbstractRRset>&
                     response) = 0;
                 /**
                  * \short No data available.
@@ -67,8 +68,8 @@
          * \param question What to ask. The resolver will decide who.
          * \param callback What should happen when the answer is ready.
          */
-        virtual void resolve(isc::dns::QuestionPtr question,
-            CallbackPtr callback) = 0;
+        virtual void resolve(const isc::dns::QuestionPtr& question,
+            const CallbackPtr& callback) = 0;
         /// \short Virtual destructor, so descendants are properly cleaned up
         virtual ~ ResolverInterface() {}
 };

Modified: branches/trac408/src/lib/nsas/tests/nameserver_address_store_unittest.cc
==============================================================================
--- branches/trac408/src/lib/nsas/tests/nameserver_address_store_unittest.cc (original)
+++ branches/trac408/src/lib/nsas/tests/nameserver_address_store_unittest.cc Mon Dec  6 15:38:05 2010
@@ -282,7 +282,7 @@
         getCallback());
     // There should be no questions, because there's nothing to ask
     EXPECT_EQ(0, resolver_->requests.size());
-    // And there should be one „unreachable“ answer for the query
+    // And there should be one "unreachable" answer for the query
     ASSERT_EQ(1, NSASCallback::results.size());
     EXPECT_FALSE(NSASCallback::results[0].first);
 }

Modified: branches/trac408/src/lib/nsas/tests/nsas_test.h
==============================================================================
--- branches/trac408/src/lib/nsas/tests/nsas_test.h (original)
+++ branches/trac408/src/lib/nsas/tests/nsas_test.h Mon Dec  6 15:38:05 2010
@@ -231,7 +231,7 @@
     public:
         typedef pair<QuestionPtr, CallbackPtr> Request;
         vector<Request> requests;
-        virtual void resolve(QuestionPtr q, CallbackPtr c) {
+        virtual void resolve(const QuestionPtr& q, const CallbackPtr& c) {
             PresetAnswers::iterator it(answers_.find(*q));
             if (it == answers_.end()) {
                 requests.push_back(Request(q, c));

Modified: branches/trac408/src/lib/nsas/zone_entry.cc
==============================================================================
--- branches/trac408/src/lib/nsas/zone_entry.cc (original)
+++ branches/trac408/src/lib/nsas/zone_entry.cc Mon Dec  6 15:38:05 2010
@@ -68,7 +68,7 @@
         ResolverCallback(shared_ptr<ZoneEntry> entry) :
             entry_(entry)
         { }
-        virtual void success(shared_ptr<AbstractRRset> answer) {
+        virtual void success(const shared_ptr<AbstractRRset>& answer) {
             shared_ptr<Lock> lock(new Lock(entry_->mutex_));
             RdataIteratorPtr iterator(answer->getRdataIterator());
             iterator->first();