X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=src%2FCommandStubs.pl;h=7da444ec790176150a434ec4afa892d43285a9a2;hb=f7cae48a17d6decd0a9bd997188271daa0a885b1;hp=b94f4a40c68cf54e127f360feae6b8bfefb3742f;hpb=b2ea176f79c18483ad6195bd37f9835d6ae9f034;p=infobot.git

diff --git a/src/CommandStubs.pl b/src/CommandStubs.pl
index b94f4a4..7da444e 100644
--- a/src/CommandStubs.pl
+++ b/src/CommandStubs.pl
@@ -3,13 +3,22 @@
 # WARN: this file does not reload on HUP.
 #
 
-#use strict;
+# use strict;	# TODO
 
-$babel_lang_regex = "fr|sp|po|pt|it|ge|de|gr|en";
+use vars qw($who $msgType $conn $chan $message $ident $talkchannel
+	$bot_version $babel_lang_regex $bot_data_dir);
+use vars qw(@vernick @vernicktodo);
+use vars qw(%channels %cache %mask %userstats %myModules %cmdstats
+	%hooks_extra %lang %ver);
+# FIX THE FOLLOWING:
+use vars qw($total $x $type $i $good);
+
+$babel_lang_regex = "fr|sp|es|po|pt|it|ge|de|gr|en|zh|ja|jp|ko|kr|ru";
+$w3search_regex   = "google";
 
 ### COMMAND HOOK IMPLEMENTATION.
 # addCmdHook("SECTION", 'TEXT_HOOK',
-#	(CODEREF	=> 'Blah', 
+#	(CODEREF	=> 'Blah',
 #	Forker		=> 1,
 #	CheckModule	=> 1,			# ???
 #	Module		=> 'blah.pl'		# preload module.
@@ -35,7 +44,8 @@ sub addCmdHook {
 # RUN IF ADDRESSED.
 sub parseCmdHook {
     my ($hashname, $line) = @_;
-    $line =~ /^(\S+)( (.*))?$/;
+    $line =~ s/^\s+|\s+$//g;	# again.
+    $line =~ /^(\S+)(\s+(.*))?$/;
     my $cmd	= $1;	# command name is whitespaceless.
     my $flatarg	= $3;
     my @args	= split(/\s+/, $flatarg || '');
@@ -64,7 +74,7 @@ sub parseCmdHook {
 	    next;
 	}
 
-	&status("hooks($hashname): $cmd matched '$ident'");
+	&status("hooks($hashname): $cmd matched '$ident' '$flatarg'");
 	my %hash = %{ ${"hooks_$hashname"}{$ident} };
 
 	if (!scalar keys %hash) {
@@ -120,7 +130,7 @@ sub parseCmdHook {
 
 	    # check if CODEREF exists.
 	    if (!defined &{ $hash{'CODEREF'} }) {
-		&WARN("coderef $hash{'CODEREF'} don't exist.");
+		&WARN("coderef $hash{'CODEREF'} does not exist.");
 		if (defined $who) {
 		    &msg($who, "coderef does not exist for $ident.");
 		}
@@ -174,13 +184,13 @@ sub parseCmdHook {
 &addCmdHook("extra", 'd?find', ('CODEREF' => 'Debian::DebianFind',
 	'Forker' => 1, 'Identifier' => 'debian',
 	'Cmdstats' => 'Debian Search', 'Help' => "find" ) );
-&addCmdHook("extra", 'insult', ('CODEREF' => 'Insult::Insult',
-	'Forker' => 1, 'Identifier' => 'insult', 'Help' => "insult" ) );
+#&addCmdHook("extra", 'insult', ('CODEREF' => 'Insult::Insult',
+#	'Forker' => 1, 'Identifier' => 'insult', 'Help' => "insult" ) );
 &addCmdHook("extra", 'kernel', ('CODEREF' => 'Kernel::Kernel',
 	'Forker' => 1, 'Identifier' => 'kernel',
 	'Cmdstats' => 'Kernel', 'NoArgs' => 1) );
 &addCmdHook("extra", 'listauth', ('CODEREF' => 'CmdListAuth',
-	'Identifier' => 'search', Module => 'factoids', 
+	'Identifier' => 'search', Module => 'factoids',
 	'Help' => 'listauth') );
 &addCmdHook("extra", 'quote', ('CODEREF' => 'Quote::Quote',
 	'Forker' => 1, 'Identifier' => 'quote',
@@ -208,16 +218,39 @@ sub parseCmdHook {
 &addCmdHook("extra", 'slashdot', ('CODEREF' => 'Slashdot::Slashdot',
 	'Identifier' => 'slashdot', 'Forker' => 1,
 	'Cmdstats' => 'Slashdot') );
+&addCmdHook("extra", 'plug', ('CODEREF' => 'Plug::Plug',
+	'Identifier' => 'plug', 'Forker' => 1,
+	'Cmdstats' => 'Plug') );
 &addCmdHook("extra", 'uptime', ('CODEREF' => 'uptime', 'Identifier' => 'uptime',
 	'Cmdstats' => 'Uptime') );
 &addCmdHook("extra", 'nullski', ('CODEREF' => 'nullski', ) );
-&addCmdHook("extra", '(fm|freshmeat)', ('CODEREF' => 'Freshmeat::Freshmeat',
-	'Identifier' => 'freshmeat', 'Cmdstats' => 'Freshmeat',
-	'Forker' => 1, 'Help' => 'freshmeat') );
 &addCmdHook("extra", 'verstats', ('CODEREF' => 'do_verstats' ) );
 &addCmdHook("extra", 'weather', ('CODEREF' => 'Weather::Weather',
 	'Identifier' => 'weather', 'Help' => 'weather',
-	'Cmdstats' => 'Weather') );
+	'Cmdstats' => 'weather', 'Forker' => 1) );
+&addCmdHook("extra", 'bzflist', ('CODEREF' => 'BZFlag::list',
+	'Identifier' => 'bzflag', 'Cmdstats' => 'BZFlag',
+	'Forker' => 1) );
+&addCmdHook("extra", 'bzfquery', ('CODEREF' => 'BZFlag::query',
+	'Identifier' => 'bzflag', 'Cmdstats' => 'BZFlag',
+	'Forker' => 1, 'Help' => 'bzflag') );
+&addCmdHook("extra", 'zfi', ('CODEREF' => 'zfi::query',
+	'Identifier' => 'zfi', 'Cmdstats' => 'zfi',
+	'Forker' => 1) );
+&addCmdHook("extra", '(zippy|yow)', ('CODEREF' => 'zippy::get',
+	'Identifier' => 'zippy', 'Cmdstats' => 'zippy',
+	'Forker' => 1) );
+&addCmdHook("extra", 'zsi', ('CODEREF' => 'zsi::query',
+	'Identifier' => 'zsi', 'Cmdstats' => 'zsi',
+	'Forker' => 1) );
+&addCmdHook("extra", '(ex)?change', ('CODEREF' => 'Exchange::query',
+	'Identifier' => 'exchange', 'Cmdstats' => 'exchange',
+	'Forker' => 1) );
+&addCmdHook("extra", '(botmail|message)', ('CODEREF' => 'botmail::parse',
+	'Identifier' => 'botmail', 'Cmdstats' => 'botmail') );
+&addCmdHook("extra", 'httpdtype', ('CODEREF' => 'HTTPDtype::HTTPDtype',
+	'Identifier' => 'httpdtype', 'Cmdstats' => 'httpdtype',
+	'Forker' => 1) );
 
 ###
 ### END OF ADDING HOOKS.
@@ -235,9 +268,9 @@ sub Modules {
 		^\s*
 		(?:babel(?:fish)?|x|xlate|translate)
 		\s+
-		(to|from)		# direction of translation (through)
+		($babel_lang_regex)\w*	# from language?
 		\s+
-		($babel_lang_regex)\w*	# which language?
+		($babel_lang_regex)\w*	# to language?
 		\s*
 		(.+)			# The phrase to be translated
     }xoi) {
@@ -249,7 +282,7 @@ sub Modules {
 	return;
     }
 
-    my $debiancmd	 = 'conflicts?|depends?|desc|file|info|provides?';
+    my $debiancmd	 = 'conflicts?|depends?|desc|file|(?:d)?info|provides?';
     $debiancmd		.= '|recommends?|suggests?|maint|maintainer';
 
     if ($message =~ /^($debiancmd)(\s+(.*))?$/i) {
@@ -266,7 +299,7 @@ sub Modules {
     }
 
     # google searching. Simon++
-    if ($message =~ /^(?:search\s+)?(\S+)\s+for\s+['"]?(.*?)["']?\s*\?*$/i) {
+    if ($message =~ /^(?:search\s+)?($w3search_regex)\s+(?:for\s+)?['"]?(.*?)["']?\s*\?*$/i) {
 	return unless (&hasParam("wwwsearch"));
 
 	&Forker("wwwsearch", sub { &W3Search::W3Search($1,$2); } );
@@ -279,102 +312,8 @@ sub Modules {
     my $itc;
     $itc = &getChanConf("ircTextCounters");
     $itc = &findChanConf("ircTextCounters") unless ($itc);
-    if ($itc) {
-	$itc =~ s/([^\w\s])/\\$1/g;
-	my $z = join '|', split ' ', $itc;
-
-	if ($msgType eq "privmsg" and $message =~ / ($mask{chan})$/) {
-	    &DEBUG("ircTC: privmsg detected; chan = $1");
-	    $chan = $1;
-	}
-
-	if ($message =~ /^_stats(\s+(\S+))$/i) {
-	    &textstats_main($2);
-	    return;
-	}
-
-	if ($message =~ /^($z)stats(\s+(\S+))?$/i) {
-	    my $type	= $1;
-	    my $arg	= $3;
-
-	    # even more uglier with channel/time arguments.
-	    my $c	= $chan;
-#	    my $c	= $chan || "PRIVATE";
-	    my $where	= "type=".&dbQuote($type);
-	    $where	.= " AND channel=".&dbQuote($c) if (defined $c);
-	    &DEBUG("not using chan arg") if (!defined $c);
-	    my $sum = (&dbRawReturn("SELECT SUM(counter) FROM stats"
-			." WHERE ".$where ))[0];
-
-	    if (!defined $arg or $arg =~ /^\s*$/) {
-		# this is way fucking ugly.
-
-		my %hash = &dbGetCol("stats", "nick,counter",
-			$where." ORDER BY counter DESC LIMIT 3", 1);
-		my $i;
-		my @top;
-
-		# unfortunately we have to sort it again!
-		# todo: make dbGetCol return hash and array? too much effort.
-		my $tp = 0;
-		foreach $i (sort { $b <=> $a } keys %hash) {
-		    foreach (keys %{ $hash{$i} }) {
-			my $p	= sprintf("%.01f", 100*$i/$sum);
-			$tp	+= $p;
-			push(@top, "\002$_\002 -- $i ($p%)");
-		    }
-		}
-		my $topstr = "";
-		&DEBUG("*stats: tp => $tp");
-		if (scalar @top) {
-		    $topstr = ".  Top ".scalar(@top).": ".join(', ', @top);
-		}
-
-		if (defined $sum) {
-		    &pSReply("total count of \037$type\037 on \002$c\002: $sum$topstr");
-		} else {
-		    &pSReply("zero counter for \037$type\037.");
-		}
-	    } else {
-		my $x = (&dbRawReturn("SELECT SUM(counter) FROM stats".
-			" WHERE $where AND nick=".&dbQuote($arg) ))[0];
-
-		if (!defined $x) {	# !defined.
-		    &pSReply("$arg has not said $type yet.");
-		    return;
-		}
-
-		# defined.
-		my @array = &dbGet("stats", "nick",
-			$where." ORDER BY counter", 1);
-		my $good = 0;
-		my $i = 0;
-		for($i=0; $i<scalar @array; $i++) {
-		    next unless ($array[0] =~ /^\Q$who\E$/);
-		    $good++;
-		    last;
-		}
-		$i++;
-
-		my $total = scalar(@array);
-		my $xtra = "";
-		if ($total and $good) {
-		    my $pct = sprintf("%.01f", 100*(1+$total-$i)/$total);
-		    $xtra = ", ranked $i\002/\002$total (percentile: \002$pct\002 %)";
-		}
-
-		my $pct1 = sprintf("%.01f", 100*$x/$sum);
-		&pSReply("\002$arg\002 has said \037$type\037 \002$x\002 times (\002$pct1\002 %)$xtra");
-	    }
-
-	    return;
-	}
-
-	if ($@) {
-	    &DEBUG("regex failed: $@");
-	    return;
-	}
-    }
+    return if ($itc && &do_text_counters($itc) == 1);
+    # end of text counters.
 
     # list{keys|values}. xk++. Idea taken from #linuxwarez@EFNET
     if ($message =~ /^list(\S+)(\s+(.*))?$/i) {
@@ -384,7 +323,6 @@ sub Modules {
 	my $args	= $3 || "";
 
 	$thiscmd	=~ s/^vals$/values/;
-#	$args		=~ s/\s+$//g;
 	return if ($thiscmd ne "keys" && $thiscmd ne "values");
 
 	# Usage:
@@ -399,8 +337,8 @@ sub Modules {
 	    $args	=~ s/^["']|["']$//g;
 	}
 
-	if (length $args == 1) {
-	    &msg($who,"search string is too short.");
+	if (length $args < 2 && &IsFlag("o") ne "o") {
+	    &msg($who, "search string is too short.");
 	    return;
 	}
 
@@ -459,7 +397,7 @@ sub Modules {
 	    $percentage = "off the scale";
 	} else {
 	    $percentage = sprintf("%0.4f", $percentage);
-	    $percentage =~ s/\.?0+$//;
+	    $percentage =~ s/(\.\d+)0+$/$1/;
 	    $percentage .= '%';
 	}
 
@@ -541,19 +479,6 @@ sub Modules {
     return "CONTINUE";
 }
 
-# Freshmeat. xk++
-sub freshmeat {
-    my ($query) = @_;
-
-    if (!defined $query) {
-	&help("freshmeat");
-	&msg($who, "I have \002".&countKeys("freshmeat")."\002 entries.");
-	return;
-    }
-
-    &Freshmeat::Freshmeat($query);
-}
-
 # Uptime. xk++
 sub uptime {
     my $count = 1;
@@ -575,11 +500,6 @@ sub seen {
     my($person) = lc shift;
     $person =~ s/\?*$//;
 
-    if (&whatInterface() !~ /IRC/) {
-	&status("seen disabled in CLI.");
-	return;
-    }
-
     if (!defined $person or $person =~ /^$/) {
 	&help("seen");
 
@@ -594,12 +514,12 @@ sub seen {
 
     &seenFlush();	# very evil hack. oh well, better safe than sorry.
 
-    ### TODO: Support &dbGetColInfo(); like in &FactInfo();
+    # TODO: convert to &sqlSelectRowHash();
     my $select = "nick,time,channel,host,message";
     if ($person eq "random") {
 	@seen = &randKey("seen", $select);
     } else {
-	@seen = &dbGet("seen", $select, "nick=".&dbQuote($person) );
+	@seen = &sqlSelect("seen", $select, { nick => $person } );
     }
 
     if (scalar @seen < 2) {
@@ -614,6 +534,7 @@ sub seen {
     my $reply;
     ### TODO: multi channel support. may require &IsNick() to return
     ###	all channels or something.
+
     my @chans = &getNickInChans($seen[0]);
     if (scalar @chans) {
 	$reply = "$seen[0] is currently on";
@@ -624,7 +545,7 @@ sub seen {
 	    $reply .= " (".&Time2String(time() - $userstats{lc $seen[0]}{'Join'}).")";
 	}
 
-	if (&IsParam("seenStats")) {
+	if (&IsChanConf("seenStats") > 0) {
 	    my $i;
 	    $i = $userstats{lc $seen[0]}{'Count'};
 	    $reply .= ".  Has said a total of \002$i\002 messages" if (defined $i);
@@ -680,8 +601,8 @@ sub cookie {
     ### TODO: convert this to a Forker function!
     if ($arg) {
 	my @list = &searchTable("factoids", "factoid_key", "factoid_value", $arg);
-	$key  = &getRandom(@list);
-	$val  = &getFactInfo("factoids", $key, "factoid_value");
+	$key	= &getRandom(@list);
+	$value	= &getFactInfo($key, "factoid_value");
     } else {
 	($key,$value) = &randKey("factoids","factoid_key,factoid_value");
     }
@@ -810,7 +731,7 @@ sub do_verstats {
 	return;
     }
 
-    &msg($who, "Sending CTCP VERSION to #$chan...");
+    &msg($who, "Sending CTCP VERSION to $chan; results in 60s.");
     $conn->ctcp("VERSION", $chan);
     $cache{verstats}{chan}	= $chan;
     $cache{verstats}{who}	= $who;
@@ -864,6 +785,8 @@ sub do_verstats {
 	    }
 	}
 
+	# hack. this is one major downside to scheduling.
+	$chan = $c;
 	&pSReply( &formListReply(0, "IRC Client versions for $c ", @list) );
 
 	# clean up not-needed data structures.
@@ -887,28 +810,55 @@ sub verstats_flush {
     $conn->schedule(3, \&verstats_flush() );
 }
 
-sub textstats_main {
-    my($arg) = @_;
+sub do_text_counters {
+    my ($itc) = @_;
+    $itc =~ s/([^\w\s])/\\$1/g;
+    my $z = join '|', split ' ', $itc;
+
+    if ($msgType eq "privmsg" and $message =~ / ($mask{chan})$/) {
+	&DEBUG("ircTC: privmsg detected; chan = $1");
+	$chan = $1;
+    }
+
+    if ($message =~ /^_stats(\s+(\S+))$/i) {
+	&textstats_main($2);
+	return 1;
+    }
+
+    my ($type,$arg);
+    if ($message =~ /^($z)stats(\s+(\S+))?$/i) {
+	$type = $1;
+	$arg  = $3;
+    } else {
+	return 0;
+    }
 
     # even more uglier with channel/time arguments.
     my $c	= $chan;
-#    my $c	= $chan || "PRIVATE";
-    my $where	= "channel=".&dbQuote($c) if (defined $c);
-    &DEBUG("not using chan arg") if (!defined $c);
-    my $sum = (&dbRawReturn("SELECT SUM(counter) FROM stats"
-		." WHERE ".$where ))[0];
+#   my $c	= $chan || "PRIVATE";
+    my $where	= "type=".&sqlQuote($type);
+    if (defined $c) {
+	&DEBUG("c => $c");
+	$where	.= " AND channel=".&sqlQuote($c) if (defined $c);
+    } else {
+	&DEBUG("not using chan arg");
+    }
+
+    my $sum = (&sqlRawReturn("SELECT SUM(counter) FROM stats"
+			." WHERE ".$where ))[0];
 
     if (!defined $arg or $arg =~ /^\s*$/) {
 	# this is way fucking ugly.
-	&DEBUG("_stats: !arg");
 
-	my %hash = &dbGetCol("stats", "nick,counter",
-		$where." ORDER BY counter DESC LIMIT 3", 1);
+	# TODO convert $where to hash
+	my %hash = &sqlSelectColHash("stats", "nick,counter",
+			{ },
+			$where." ORDER BY counter DESC LIMIT 3", 1
+	);
 	my $i;
 	my @top;
 
 	# unfortunately we have to sort it again!
-	# todo: make dbGetCol return hash and array? too much effort.
 	my $tp = 0;
 	foreach $i (sort { $b <=> $a } keys %hash) {
 	    foreach (keys %{ $hash{$i} }) {
@@ -917,9 +867,7 @@ sub textstats_main {
 		push(@top, "\002$_\002 -- $i ($p%)");
 	    }
 	}
-
 	my $topstr = "";
-	&DEBUG("*stats: tp => $tp");
 	if (scalar @top) {
 	    $topstr = ".  Top ".scalar(@top).": ".join(', ', @top);
 	}
@@ -930,34 +878,30 @@ sub textstats_main {
 	    &pSReply("zero counter for \037$type\037.");
 	}
     } else {
-	my %hash = &dbGetCol("stats", "type,counter",
-		"$where AND nick=".&dbQuote($arg) );
-
-	foreach (keys %hash) {
-	    &DEBUG("_stats: hash{$_} => $hash{$_}");
-	    # ranking.
-	    my @array = &dbGet("stats", "nick",
-		$where." ORDER BY counter", 1);
-	    my $good = 0;
-	    my $i = 0;
-	    for($i=0; $i<scalar @array; $i++) {
-		next unless ($array[0] =~ /^\Q$who\E$/);
-		$good++;
-		last;
-	    }
-	    $i++;
-
-	    my $total = scalar(@array);
-	    &DEBUG("   i => $i, good => $good, total => $total");
-	}
-
-	return;
+	# TODO convert $where to hash and use a sqlSelect
+	my $x = (&sqlRawReturn("SELECT SUM(counter) FROM stats".
+			" WHERE $where AND nick=".&sqlQuote($arg) ))[0];
 
 	if (!defined $x) {	# !defined.
 	    &pSReply("$arg has not said $type yet.");
-	    return;
+	    return 1;
+	}
+
+	# defined.
+	# TODO convert $where to hash
+	my @array = &sqlSelect("stats", "nick", undef,
+			$where." ORDER BY counter", 1
+	);
+	my $good = 0;
+	my $i = 0;
+	for ($i=0; $i<scalar @array; $i++) {
+	    next unless ($array[0] =~ /^\Q$who\E$/);
+	    $good++;
+	    last;
 	}
+	$i++;
 
+	my $total = scalar(@array);
 	my $xtra = "";
 	if ($total and $good) {
 	    my $pct = sprintf("%.01f", 100*(1+$total-$i)/$total);
@@ -967,6 +911,102 @@ sub textstats_main {
 	my $pct1 = sprintf("%.01f", 100*$x/$sum);
 	&pSReply("\002$arg\002 has said \037$type\037 \002$x\002 times (\002$pct1\002 %)$xtra");
     }
+
+    return 1;
+}
+
+sub textstats_main {
+    my($arg) = @_;
+
+    # even more uglier with channel/time arguments.
+    my $c	= $chan;
+#    my $c	= $chan || "PRIVATE";
+    &DEBUG("not using chan arg") if (!defined $c);
+
+    # example of converting from RawReturn to sqlSelect.
+    my $where_href = (defined $c) ? { channel => $c } : "";
+    my $sum = &sqlSelect("stats", "SUM(counter)", $where_href);
+
+    if (!defined $arg or $arg =~ /^\s*$/) {
+	# this is way fucking ugly.
+	&DEBUG("_stats: !arg");
+
+	my %hash = &sqlSelectColHash("stats", "nick,counter",
+		$where_href,
+		" ORDER BY counter DESC LIMIT 3", 1
+	);
+	my $i;
+	my @top;
+
+	# unfortunately we have to sort it again!
+	my $tp = 0;
+	foreach $i (sort { $b <=> $a } keys %hash) {
+	    foreach (keys %{ $hash{$i} }) {
+		my $p	= sprintf("%.01f", 100*$i/$sum);
+		$tp	+= $p;
+		push(@top, "\002$_\002 -- $i ($p%)");
+	    }
+	}
+
+	my $topstr = "";
+	if (scalar @top) {
+	    $topstr = ".  Top ".scalar(@top).": ".join(', ', @top);
+	}
+
+	if (defined $sum) {
+	    &pSReply("total count of \037$type\037 on \002$c\002: $sum$topstr");
+	} else {
+	    &pSReply("zero counter for \037$type\037.");
+	}
+
+	return;
+    }
+
+    # TODO add nick to where_href
+    my %hash = &sqlSelectColHash("stats", "type,counter",
+		$where_href, " AND nick=".&sqlQuote($arg)
+    );
+    # this is totally fucked... needs to be fixed... and cleaned up.
+    my $total;
+    my $good;
+    my $ii;
+    my $x;
+
+    foreach (keys %hash) {
+	&DEBUG("_stats: hash{$_} => $hash{$_}");
+	# ranking.
+	# TODO convert $where to hash
+	my @array = &sqlSelect("stats", "nick", undef,
+		$where." ORDER BY counter", 1);
+	$good = 0;
+	$ii = 0;
+	for(my $i=0; $i<scalar @array; $i++) {
+	    next unless ($array[0] =~ /^\Q$who\E$/);
+	    $good++;
+	    last;
+	}
+	$ii++;
+
+	$total = scalar(@array);
+	&DEBUG("   i => $i, good => $good, total => $total");
+	$x .= " ".$total."blah blah";
+    }
+
+#    return;
+
+    if (!defined $x) {	# !defined.
+	&pSReply("$arg has not said $type yet.");
+	return;
+    }
+
+    my $xtra = "";
+    if ($total and $good) {
+	my $pct = sprintf("%.01f", 100*(1+$total-$ii)/$total);
+	$xtra = ", ranked $ii\002/\002$total (percentile: \002$pct\002 %)";
+    }
+
+    my $pct1 = sprintf("%.01f", 100*$x/$sum);
+    &pSReply("\002$arg\002 has said \037$type\037 \002$x\002 times (\002$pct1\002 %)$xtra");
 }
 
 sub nullski { my ($arg) = @_; return unless (defined $arg);