X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2Fif.pm;h=5f6bcc8ae16cfc8af3869a681613002b1b3c6af7;hb=dc8f638738195a8f1d6774aa5a9874607346c798;hp=32c4fad45eca3f456bbdc415afccbb71e9a68050;hpb=cd16c24cb16bb0bd0fa486b8e98c39fcf6ebcf5c;p=p5sagit%2Fp5-mst-13.2.git

diff --git a/lib/if.pm b/lib/if.pm
index 32c4fad..5f6bcc8 100644
--- a/lib/if.pm
+++ b/lib/if.pm
@@ -1,13 +1,18 @@
 package if;
 
-our $VERSION = '0.01';
+$VERSION = '0.05';
 
 sub work {
   my $method = shift() ? 'import' : 'unimport';
+  die "Too few arguments to `use if' (some code returning an empty list in list context?)"
+    unless @_ >= 2;
   return unless shift;		# CONDITION
-  my $p = shift;		# PACKAGE
-  eval "require $p" or die;	# Adds .pm etc if needed
-  $p->$method(@_) if $p->can($method);
+
+  my $p = $_[0];		# PACKAGE
+  (my $file = "$p.pm") =~ s!::!/!g;
+  require $file;		# Works even if $_[0] is a keyword (like open)
+  my $m = $p->can($method);
+  goto &$m if $m;
 }
 
 sub import   { shift; unshift @_, 1; goto &work }
@@ -35,6 +40,9 @@ the same as of
 
   use MODULE ARGUMENTS;
 
+Above C<< => >> provides necessary quoting of C<MODULE>.  If not used (e.g.,
+no ARGUMENTS to give), you'd better quote C<MODULE> yourselves.
+
 =head1 BUGS
 
 The current implementation does not allow specification of the