$x++;  # keep track of how many users have logged in
[download]

Actually, I think that's a poor comment, unless the text around it makes it make more sense. I'd prefer to see something like:

$x++; # $x is number of users currently logged in, and we have a new o
+ne
[download]

Hmm. On re-reading your comment, I can't tell whether you meant total number of users that have ever logged in, or whether it's a current count. That's probably why I already flagged it as bad. {grin}

Even better would be to never use $x for something like that:

$current_users++; # we have a new one
[download]

That's a lot better.

-- Randal L. Schwartz, Perl hacker