--- rxvt-unicode/src/perl/selection-popup	2006/01/09 06:29:47	1.5
+++ rxvt-unicode/src/perl/selection-popup	2006/01/19 09:47:15	1.14
@@ -1,13 +1,11 @@
 #! perl
 
 sub msg {
-   my ($self, $msg1, $msg2) = @_;
+   my ($self, $msg) = @_;
 
-   my $overlay = $self->overlay (0, 0,
-      (List::Util::max map $self->strwidth ($_), $msg1, $msg2), 2);
-   $overlay->set (0, 0, $msg1);
-   $overlay->set (0, 1, $msg2);
-   my $iow; $iow = urxvt::timer->new->start (urxvt::NOW + 1)->cb (sub {
+   my $overlay = $self->overlay (0, 0, $self->strwidth ($msg), 1);
+   $overlay->set (0, 0, $msg);
+   my $iow; $iow = urxvt::timer->new->after (1)->cb (sub {
       undef $overlay;
       undef $iow;
    });
@@ -16,9 +14,15 @@
 sub on_start {
    my ($self) = @_;
 
+   $self->{browser} = $self->x_resource ("urlLauncher") || "x-www-browser";
+
    $self->grab_button (3, urxvt::ControlMask);
+
+   ()
 }
 
+our @hook;
+
 sub on_button_press {
    my ($self, $event) = @_;
 
@@ -47,7 +51,7 @@
                if ($orig ne $_) {
                   $self->selection ($_);
                   s/[\x00-\x1f\x80-\x9f]/·/g;
-                  $self->msg ("Selection replaced by", $_);
+                  $self->msg ($self->special_encode ($_));
                }
             }
          });
@@ -55,9 +59,7 @@
 
       for ($text) {
          $add_button->("rot13" => sub { y/A-Za-z/N-ZA-Mn-za-m/ });
-
-         urxvt::safe
-            and $add_button->("eval perl expression" => sub { $_ = eval urxvt::untaint $_ });
+         $add_button->("eval perl expression" => sub { no warnings; $_ = eval $_ });
 
          /^(\S+):(\d+):?$/
             and $add_button->("vi-commands to load '$1'" => sub { s/^(\S+):(\d+):?$/\x1b:e $1\x0d:$2\x0d/ });
@@ -69,7 +71,19 @@
             and $add_button->("shell quote" => sub { $_ = "\Q$_" });
 
          /^(http|ftp|telnet|irc|news):\//
-            and $add_button->("run x-www-browser" => sub { system "x-www-browser \Q$_\E &" });
+            and $add_button->("run $self->{browser}" => sub { $self->exec_async ($self->{browser}, $_) });
+
+         for my $hook (@hook) {
+            if (my ($title, $cb) = $hook->($popup)) {
+               $add_button->($title, $cb);
+            }
+         }
+
+         if (/^\s*((?:0x)?\d+)\s*$/) {
+            $popup->add_title (sprintf "%20s", eval $1);
+            $popup->add_title (sprintf "%20s", sprintf "0x%x", eval $1);
+            $popup->add_title (sprintf "%20s", sprintf "0%o", eval $1);
+         }
       }
 
       $popup->show;