From: Doug Freed <redacted>
Date: Wed, 17 Jan 2018 05:32:02 +0000 (-0500)
Subject: timer.pl: minor improvements
X-Git-Url: http://git.99rst.org/?a=commitdiff_plain;h=16c99ae723f40428526c7f9c2383eccb82b6b532;p=irssi-scripts.irssi.org.git

timer.pl: minor improvements

* Remove finished timers during their last cycle, rather than one
  interval after
* Allow timer names to contain anything other than whitespace
---

diff --git a/scripts/timer.pl b/scripts/timer.pl
index cd8f860..1c5a8c7 100644
--- a/scripts/timer.pl
+++ b/scripts/timer.pl
@@ -19,7 +19,7 @@ use warnings;
 use vars  qw ($VERSION %IRSSI);
 use Irssi 20020325 qw (command_bind command_runsub command timeout_add timeout_remove signal_add_first);
 
-$VERSION = '0.6';
+$VERSION = '0.7';
 %IRSSI = (
     authors     => 'Kimmo Lehto, Marcus Rueckert',
     contact     => 'kimmo@a-men.org, darix@irssi.org' ,
@@ -35,13 +35,6 @@ our %timers;
 sub timer_command {
     my ( $name ) = @_;
     if ( exists ( $timers{$name} ) ) {
-        if ( $timers{$name}->{'repeat'} != -1 ) {
-            if ( $timers{$name}->{'repeat'}-- == 0) {
-                cmd_timerstop( $name );
-                return;
-            }
-        }
-
         my ($server, $item);
         if ($timers{$name}->{'server'}) {
             $server = Irssi::server_find_tag( $timers{$name}->{'server'} );
@@ -54,6 +47,12 @@ sub timer_command {
         } else {
             command( $timers{$name}->{'command'} );
         }
+
+        if ( $timers{$name}->{'repeat'} != -1 ) {
+            if ( --$timers{$name}->{'repeat'} == 0) {
+                cmd_timerstop( $name );
+            }
+        }
     }
 }
 
@@ -88,11 +87,11 @@ command_bind 'timer add' => sub {
     my ( $data, $server, $item ) = @_;
     my ( $name, $interval, $times, $command );
 
-    if ( $data =~ /^\s*(\w+)\s+(\d+(?:\.\d+)?)\s+(-?\d+)\s+(.*)$/ ) {
+    if ( $data =~ /^\s*(\S+)\s+(\d+(?:\.\d+)?)\s+(-?\d+)\s+(.*)$/ ) {
         ( $name, $interval, $times, $command ) = ( $1, $2, $3, $4 );
         $times = -1 if ( $times == 0 );
     }
-    elsif ( $data =~ /^\s*(\w+)\s+(\d+(?:\.\d+)?)\s+(.*)$/ )
+    elsif ( $data =~ /^\s*(\S+)\s+(\d+(?:\.\d+)?)\s+(.*)$/ )
     {
         ( $name, $interval, $times, $command ) = ( $1, $2, -1, $3 );
     }