Changeset e0fba58 for perl/modules/IRC/lib

Timestamp:

Mar 11, 2009, 6:22:41 PM (16 years ago)

Author:

Nelson Elhage <nelhage@mit.edu>

Branches:

master, debian, release-1.10, release-1.4, release-1.5, release-1.6, release-1.7, release-1.8, release-1.9

Children:

82dd923

Parents:

19aeff4

Message:

IRC: Don't use a default target for irc-msg

Because /msg can take either a user or a channel, it's ambiguous whether
'irc-msg nelhage' with your cursor on a message to #barnowl is supposed
to tell #barnowl 'nelhage', or start a message to nelhage. Resolve this
by just never taking the channel from the current message.

File:

• perl/modules/IRC/lib/BarnOwl/Module/IRC.pm (modified) (2 diffs)

perl/modules/IRC/lib/BarnOwl/Module/IRC.pm

@@ -147 +147 @@
 
     BarnOwl::new_command(
-        'irc-msg' => mk_irc_command( \&cmd_msg, OPTIONAL_CHANNEL ),
+        'irc-msg' => mk_irc_command( \&cmd_msg ),
         {
             summary => 'Send an IRC message',
@@ -380 +380 @@
     my $cmd  = shift;
     my $conn = shift;
-    my $to = shift || shift or die("Usage: $cmd NICK\n");
+    my $to = shift or die("Usage: $cmd [NICK|CHANNEL]\n");
     # handle multiple recipients?
     if(@_) {