From bc6c7c26b682d2652139a5a4280f8db4451c15d9 Mon Sep 17 00:00:00 2001
From: defacer <defacer>
Date: Mon, 17 Jan 2005 22:26:20 +0000
Subject: [PATCH] Better comments and some more diagnostic messages to help
 with bug hunting.

Also changed a > to >= to make the polling interval precice to the second.
---
 mod/chat/chatd.php | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/mod/chat/chatd.php b/mod/chat/chatd.php
index 8123e6891c..0bf0a812d7 100755
--- a/mod/chat/chatd.php
+++ b/mod/chat/chatd.php
@@ -84,6 +84,9 @@ class ChatDaemon {
     var $sets_info = array();    // Keyed by sessionid exactly like conn_sets, one of these for each of those
     var $chatrooms = array();    // Keyed by chatid, holding arrays of data
 
+    // IMPORTANT: $conn_sets, $sets_info and $chatrooms must remain synchronized!
+    //            Pay extra attention when you write code that affects any of them!
+
     function ChatDaemon() {
         $this->_trace_level         = E_ALL ^ E_USER_NOTICE;
         $this->_pcntl_exists        = function_exists('pcntl_fork');
@@ -113,7 +116,9 @@ class ChatDaemon {
                 if(!empty($chatroom['users'])) {
                     foreach($chatroom['users'] as $sessionid => $userid) {
                         // We will be polling each user as required
+                        $this->trace('...shall we poll '.$sessionid.'?');
                         if($this->sets_info[$sessionid]['chatuser']->lastmessageping < $this->_last_idle_poll) {
+                            $this->trace('YES!');
                             // This user hasn't been polled since his last message
                             if($this->write_data($this->conn_sets[$sessionid][CHAT_CONNECTION_CHANNEL], '<!-- poll -->') === false) {
                                 // User appears to have disconnected
@@ -627,6 +632,7 @@ class ChatDaemon {
         unset($this->conn_sets[$sessionid]);
         unset($this->sets_info[$sessionid]);
         unset($this->chatrooms[$chatroom]['users'][$sessionid]);
+        $this->trace('Removed all traces of user with session '.$sessionid, E_USER_NOTICE);
         return true;
     }
 
@@ -1036,7 +1042,7 @@ while(true) {
     $now = time();
 
     // Clean up chatrooms with no activity as required
-    if($now - $DAEMON->_last_idle_poll > $DAEMON->_freq_poll_idle_chat) {
+    if($now - $DAEMON->_last_idle_poll >= $DAEMON->_freq_poll_idle_chat) {
         $DAEMON->poll_idle_chats($now);
     }
 
-- 
2.39.5