Added TODOs to avoid creating unlocked store_table entries.

diff --git a/src/neighbors.cc b/src/neighbors.cc
index d145a393cca5c2934faa6e660a917505fbb09fb6..f9277309e7b74bf5b4f9199f6911f64ec63325a3 100644 (file)
--- a/src/neighbors.cc
+++ b/src/neighbors.cc
@@ -1072,6 +1072,7 @@ neighborsUdpAck(const cache_key * key, icp_common_t * header, const Ip::Address
     }
 
     if (entry->lock_count == 0) {
+        // TODO: many entries are unlocked; why is this reported at level 1?
         debugs(12, 1, "neighborsUdpAck: '" << storeKeyText(key) << "' has no locks");
         neighborCountIgnored(p);
         return;
@@ -1817,6 +1818,7 @@ neighborsHtcpReply(const cache_key * key, htcpReplyData * htcp, const Ip::Addres
     }
 
     if (e->lock_count == 0) {
+        // TODO: many entries are unlocked; why is this reported at level 1?
         debugs(12, 1, "neighborsUdpAck: '" << storeKeyText(key) << "' has no locks");
         neighborCountIgnored(p);
         return;

diff --git a/src/urn.cc b/src/urn.cc
index 8e1c4ad7145bf5009ec876ca79c37e6228a2feac..48d35506d8831bfd96585ef96388d8019bf4d69d 100644 (file)
--- a/src/urn.cc
+++ b/src/urn.cc
@@ -505,6 +505,8 @@ urnParseReply(const char *inbuf, const HttpRequestMethod& m)
 
         list[i].url = url;
         list[i].host = xstrdup(host);
+        // TODO: Use storeHas() or lock/unlock entry to avoid creating unlocked
+        // ones.
         list[i].flags.cached = storeGetPublic(url, m) ? 1 : 0;
         i++;
     }