take a few more things literally

[infobot.git] / blootbot / src / Modules / Uptime.pl

diff --git a/blootbot/src/Modules/Uptime.pl b/blootbot/src/Modules/Uptime.pl
index 5f8b55f6c98ba1d3eebc6cdf90eeda13db0d450d..83b2e1b23b956f6d136d7f90f3c4e9e443e91508 100644 (file)
--- a/blootbot/src/Modules/Uptime.pl
+++ b/blootbot/src/Modules/Uptime.pl
@@ -5,7 +5,7 @@
 #   Created: 19990925.
 #
 
-use strict;
+# use strict;  # TODO
 
 my $uptimerecords      = 3;
 
@@ -17,14 +17,14 @@ sub uptimeStr {
   my $uptimenow = &uptimeNow();
 
   if (defined $_[0]) {
-    return "$uptimenow.$$ running $bot_version, ended ". localtime(time());
+    return "$uptimenow.$$ running $bot_version, ended ". gmtime(time());
   } else {
     return "$uptimenow running $bot_version";
   }
 }
 
 sub uptimeGetInfo {
-  my (%uptime,%done,%pid);
+  my (%uptime,%done);
   my ($uptime,$pid);
   my @results;
   my $file = $file{utm};
@@ -51,7 +51,7 @@ sub uptimeGetInfo {
   # should be no problems, even if uptime or pid is duplicated.
   ## WARN: run away forks may get through here, have to fix.
   foreach $uptime (sort {$b <=> $a} keys %uptime) {
-    foreach $pid (keys %{$uptime{$uptime}}) {
+    foreach $pid (keys %{ $uptime{$uptime} }) {
        next if (exists $done{$pid});
 
        push(@results,"$uptime.$pid $uptime{$uptime}{$pid}");
@@ -69,7 +69,7 @@ sub uptimeWriteFile {
   my $file = $file{utm};
 
   if ($$ != $bot_pid) {
-    &FIXME("uptime: forked process doing weird things!");
+    &FIXME('uptime: forked process doing weird things!');
     exit 0;
   }
 
@@ -83,9 +83,11 @@ sub uptimeWriteFile {
   }
 
   close OUT;
+  &status('--- Saved uptime records.');
 
-  $conn->schedule(&getRandomInt("1800-3600"), \&uptimeWriteFile, "");
-  return;
+  return unless defined $conn;
+
+  $conn->schedule(&getRandomInt('1800-3600'), \&uptimeWriteFile, '');
 }
 
 1;