Comments on polling code

Hello,

    do {
      sleep(3);
      $buddylist_online = _drupalchat_buddylist_online($buddylist);
      //$message_count = db_query('SELECT COUNT(*) FROM {drupalchat_msg} m INNER JOIN {users} u ON m.uid1 = u.uid WHERE m.uid2 = :m.uid2 AND m.timestamp > :m.timestamp', array(':m.uid2' => $user->uid, ':m.timestamp' => $last_timestamp))->fetchField();
      $message_count = db_query('   SELECT COUNT(*)
                                    FROM {drupalchat_msg} m
                                    INNER JOIN {users} u ON m.uid1 = u.uid
                                    WHERE m.uid2 = :uid2
                                    AND m.timestamp > :timestamp', array(':uid2' => $user->uid, ':timestamp' => $last_timestamp))->fetchField();
      _drupalchat_touch_user($user->uid);
      module_invoke_all('drupalchat_longpoll'); // Long poll hook
    } while (((REQUEST_TIME - $initial_time) < (ini_get('max_execution_time') - 5)) && ($message_count == 0) && (_drupalchat_buddylist_diff($buddylist_online_old, $buddylist_online)));

1. Wanted to bring to your attention that sleep(3) in the polling code does not count as execution_time on linux.

2. The code above, per my tests, will never really break out from exhausting the max_execution_time because of point #1 and because $initial_time never changes. I've modified it as follows:

      _drupalchat_touch_user($user->uid);
      module_invoke_all('drupalchat_longpoll'); // Long poll hook
      $initial_time -= 3;

Thoughts?