Audit documentation fixes

diff --git a/src/cache_manager.cc b/src/cache_manager.cc
index 5f9664432ae8da0916639668b6f1bd8122e0c1e2..fed773e6c5ed20b13cb180b2e508c0687e85f2ab 100644 (file)
--- a/src/cache_manager.cc
+++ b/src/cache_manager.cc
@@ -91,6 +91,12 @@ CacheManager::registerProfile(const Mgr::ActionProfile::Pointer &profile)
     }
 }
 
+/**
+ \ingroup CacheManagerAPI
+ * Registers a C-style action, which is implemented as a pointer to a function
+ * taking as argument a pointer to a StoreEntry and returning void.
+ * Implemented via CacheManagerActionLegacy.
+ */
 void
 CacheManager::registerProfile(char const * action, char const * desc, OBJH * handler, int pw_req_flag, int atomic)
 {

diff --git a/src/comm/Connection.h b/src/comm/Connection.h
index 42519ad0bbdaafc2ef2e8626233b3dd99f25d9b0..efdac6a1cdd8e9c131030841c2ca93692b0ac429 100644 (file)
--- a/src/comm/Connection.h
+++ b/src/comm/Connection.h
@@ -79,7 +79,7 @@ namespace Comm {
  * object for state data. But a semantic equivalent for FD with easily
  * accessible cached properties not requiring repeated complex lookups.
  *
- * Connection properties may be changed until tehe connection is opened.
+ * Connection properties may be changed until the connection is opened.
  * Properties should be considered read-only outside of the Comm layer
  * code once the connection is open.
  *
@@ -176,9 +176,12 @@ inline std::ostream &
 operator << (std::ostream &os, const Comm::Connection &conn)
 {
     os << "FD " << conn.fd << " local=" << conn.local <<
-        " remote=" << conn.remote << " flags=" << conn.flags;
+        " remote=" << conn.remote;
+    if (flags != COMM_UNSET)
+        os << " flags=" << conn.flags;
 #if USE_IDENT
-    os << " IDENT::" << conn.rfc931;
+    if (*conn.rfc931)
+        os << " IDENT::" << conn.rfc931;
 #endif
     return os;
 }

diff --git a/src/pconn.cc b/src/pconn.cc
index 234eea0a2bcbf0b22aab15ff28e53547ba1752c5..7058da6b847a795c9d543ae024bcd4aeea2f71da 100644 (file)
--- a/src/pconn.cc
+++ b/src/pconn.cc
@@ -135,11 +135,6 @@ IdleConnList::push(const Comm::ConnectionPointer &conn)
         for (int index = 0; index < size_; index++)
             theList_[index] = oldList[index];
 
-/* TODO: re-attach to MemPools.
-        if (size_ == PCONN_FDS_SZ)
-            pconn_fds_pool->freeOne(oldList);
-        else
-*/
         delete[] oldList;
     }
 

diff --git a/src/pconn.h b/src/pconn.h
index fc39a773d0c5ff9e052623b8367ca22637de86d7..94cd4a4e96bc527c9bcdc5e16944a03eb9a9c11c 100644 (file)
--- a/src/pconn.h
+++ b/src/pconn.h
@@ -61,7 +61,7 @@ public:
 
 private:
     /** List of connections we are holding.
-     * Sorted oldest to newest for most efficient speeds on pop() and findUsable()
+     * Sorted as FIFO list for most efficient speeds on pop() and findUsable()
      * The worst-case pop() and scans occur on timeout and link closure events
      * where timing is less critical. Occasional slow additions are okay.
      */