X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FImport%2FInto.pm;h=59c861bdd848b9391fe0ccd466ce7fce455084b1;hb=ac6d2081e8a5f1e5dbe0f95685080edaf40ca48b;hp=09488bb025490c542495cdc89d33b39c3033816c;hpb=8c17b6f8371a0afcdf3b7e7eccebf2f62e983f79;p=p5sagit%2FImport-Into.git

diff --git a/lib/Import/Into.pm b/lib/Import/Into.pm
index 09488bb..59c861b 100644
--- a/lib/Import/Into.pm
+++ b/lib/Import/Into.pm
@@ -3,18 +3,31 @@ package Import::Into;
 use strict;
 use warnings FATAL => 'all';
 
-our $VERSION = '1.000001'; # 1.0.1
+our $VERSION = '1.001001'; # 1.1.1
 
 my %importers;
 
+sub _importer {
+  my $target = shift;
+  my ($package, $file, $line) = $target =~ /[^0-9]/ ? ($target) : caller($target + 1);
+  my $code = qq{package $package;\n}
+    . ($file ? "#line $line \"$file\"\n" : '')
+    . 'sub { my $m = splice @_, 1, 1; shift->$m(@_) };'."\n";
+  my $sub = \(eval $code
+    or die "Couldn't build importer for $package: $@");
+  $importers{$target} = $sub
+    unless $file;
+  $sub;
+}
+
 sub import::into {
   my ($class, $target, @args) = @_;
-  $class->${\(
-    $importers{$target} ||= eval qq{
-      package $target;
-      sub { shift->import(\@_) };
-    } or die "Couldn't build importer for $target: $@"
-  )}(@args);
+  $class->${_importer($target)}(import => @args);
+}
+
+sub unimport::out_of {
+  my ($class, $target, @args) = @_;
+  $class->${_importer($target)}(unimport => @args);
 }
 
 1;
@@ -56,6 +69,22 @@ Notably, this works:
     Thing1->import::into(scalar caller);
   }
 
+Note 2: You do B<not> need to do anything to Thing1 to be able to call
+C<import::into> on it. This is a global method, and is callable on any
+package (and in fact on any object as well, although it's rarer that you'd
+want to do that).
+
+Finally, we also provide an C<unimport::out_of> to allow the exporting of the
+effect of C<no>:
+
+  # unimport::out_of was added in 1.1.0 (1.001000)
+  sub unimport {
+    Moose->unimport::out_of(scalar caller); # no MyThing == no Moose
+  }
+
+If how and why this all works is of interest to you, please read on to the
+description immediately below.
+
 =head1 DESCRIPTION
 
 Writing exporters is a pain. Some use L<Exporter>, some use L<Sub::Exporter>,
@@ -111,16 +140,43 @@ effect, and from the right package for caller checking to work - and so
 behaves correctly for all types of exporter, for pragmas, and for hybrids.
 
 Remembering all this, however, is excessively irritating. So I wrote a module
-so I didn't have to anymore. Loading L<Import::Into> will create a method
-C<import::into> which you can call on a package to import it into another
+so I didn't have to anymore. Loading L<Import::Into> creates a global method
+C<import::into> which you can call on any package to import it into another
 package. So now you can simply write:
 
   use Import::Into;
 
   $thing->import::into($target, @import_args);
 
-Just make sure you already loaded C<$thing> - if you're receiving this from
-a parameter, I recommend using L<Module::Runtime>:
+This works because of how perl resolves method calls - a call to a simple
+method name is resolved against the package of the class or object, so
+
+  $thing->method_name(@args);
+
+is roughly equivalent to:
+
+  my $code_ref = $thing->can('method_name');
+  $code_ref->($thing, @args);
+
+while if a C<::> is found, the lookup is made relative to the package name
+(i.e. everything before the last C<::>) so
+
+  $thing->Package::Name::method_name(@args);
+
+is roughly equivalent to:
+
+  my $code_ref = Package::Name->can('method_name');
+  $code_ref->($thing, @args);
+
+So since L<Import::Into> defines a method C<into> in package C<import>
+the syntax reliably calls that.
+
+For more craziness of this order, have a look at the article I wrote at
+L<http://shadow.cat/blog/matt-s-trout/madness-with-methods> which covers
+coderef abuse and the C<${\...}> syntax.
+
+Final note: You do still need to ensure that you already loaded C<$thing> - if
+you're receiving this from a parameter, I recommend using L<Module::Runtime>:
 
   use Import::Into;
   use Module::Runtime qw(use_module);
@@ -129,6 +185,12 @@ a parameter, I recommend using L<Module::Runtime>:
 
 And that's it.
 
+=head1 ACKNOWLEDGEMENTS
+
+Thanks to Getty for asking "how can I get C<< use strict; use warnings; >>
+turned on for all consumers of my code?" and then "why is this not a
+module?!".
+
 =head1 AUTHOR
 
 mst - Matt S. Trout (cpan:MSTROUT) <mst@shadowcat.co.uk>
@@ -139,7 +201,7 @@ None yet - maybe this software is perfect! (ahahahahahahahahaha)
 
 =head1 COPYRIGHT
 
-Copyright (c) 2010-2011 the Import::Into L</AUTHOR> and L</CONTRIBUTORS>
+Copyright (c) 2012 the Import::Into L</AUTHOR> and L</CONTRIBUTORS>
 as listed above.
 
 =head1 LICENSE