summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorom <om@e03df62e-2008-0410-955e-edbf42e46eb7>2006-07-22 14:56:35 +0000
committerom <om@e03df62e-2008-0410-955e-edbf42e46eb7>2006-07-22 14:56:35 +0000
commit628f8a1840eb8b4300c9f988a492051afb56fb2f (patch)
treea7d2d333764df918bb9b0305bc5122a27e7e3612
parenta1ff11998b541e47c24097cc816496513d56d002 (diff)
Change comment to actually be rightish
git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@4513 e03df62e-2008-0410-955e-edbf42e46eb7
-rw-r--r--src/modules/extra/m_sqlauth.cpp8
1 files changed, 3 insertions, 5 deletions
diff --git a/src/modules/extra/m_sqlauth.cpp b/src/modules/extra/m_sqlauth.cpp
index 8e2ea34f5..683d70c58 100644
--- a/src/modules/extra/m_sqlauth.cpp
+++ b/src/modules/extra/m_sqlauth.cpp
@@ -115,11 +115,9 @@ public:
/* When we get the query response from the service provider we will be given an ID to play with,
* just an ID number which is unique to this query. We need a way of associating that ID with a userrec
* so we insert it into a map mapping the IDs to users.
- * This isn't quite enough though, as if the user quit while the query was in progress then when the result
- * came to be processed we'd get an invalid userrec* out of the map. Now we *could* solve this by watching
- * OnUserDisconnect() and iterating the map every time someone quits to make sure they didn't have any queries
- * in progress, but that would be relatively slow and inefficient. Instead (thanks to w00t ;p) we attach a list
- * of query IDs associated with it to the userrec, so in OnUserDisconnect() we can remove it immediately.
+ * Thankfully m_sqlutils provides this, it will associate a ID with a user or channel, and if the user quits it removes the
+ * association. This means that if the user quits during a query we will just get a failed lookup from m_sqlutils - telling
+ * us to discard the query.
*/
log(DEBUG, "Sent query, got given ID %lu", req.id);