Better docs for confusing actualReplyHeader()/actualRequestHeader().

diff --git a/src/format/Format.cc b/src/format/Format.cc
index 8e1aec6d92795dcde13b0badf15dc7889b680525..d291f98b89d7af139a2b0f607f16ec436b6d45c1 100644 (file)
--- a/src/format/Format.cc
+++ b/src/format/Format.cc
@@ -330,21 +330,30 @@ sslErrorName(Security::ErrorCode err, char *buf, size_t size)
 }
 #endif
 
+/// XXX: Misnamed. TODO: Split <h (and this function) to distinguish received 
+/// headers from sent headers rather than failing to distinguish requests from responses.
+/// \retval HttpReply sent to the HTTP client (access.log and default context).
+/// \retval HttpReply received (encapsulated) from the ICAP server (icap.log context).
+/// \retval HttpRequest received (encapsulated) from the ICAP server (icap.log context).
 static const HttpMsg *
 actualReplyHeader(const AccessLogEntry::Pointer &al)
 {
     const HttpMsg *msg = al->reply;
 #if USE_ADAPTATION
+    // al->icap.reqMethod is methodNone in access.log context
     if (!msg && al->icap.reqMethod == Adaptation::methodReqmod)
         msg = al->adapted_request;
 #endif
     return msg;
 }
 
+/// XXX: Misnamed. See actualReplyHeader().
+/// \return HttpRequest or HttpReply for %http::>h.
 static const HttpMsg *
 actualRequestHeader(const AccessLogEntry::Pointer &al)
 {
 #if USE_ADAPTATION
+    // al->icap.reqMethod is methodNone in access.log context
     if (al->icap.reqMethod == Adaptation::methodRespmod) {
         // XXX: for now AccessLogEntry lacks virgin response headers
         return nullptr;