better argument checking for ->new
Jesse Luehrs [Mon, 2 May 2011 21:12:33 +0000 (16:12 -0500)]
lib/Package/Stash/PP.pm

index 9004c5d..fdb8da9 100644 (file)
@@ -26,6 +26,17 @@ This is a backend for L<Package::Stash> implemented in pure perl, for those with
 sub new {
     my $class = shift;
     my ($package) = @_;
+
+    if (!defined($package) || (ref($package) && ref($package) ne 'HASH')) {
+        confess "Package::Stash->new must be passed the name of the "
+              . "package to access";
+    }
+    elsif (ref($package) eq 'HASH') {
+        confess "The pure perl implementation of Package::Stash doesn't "
+              . "currently support anonymous stashes. You should install "
+              . "Package::Stash::XS";
+    }
+
     return bless {
         'package' => $package,
     }, $class;