=head1 VERSION
-Version 0.13
+Version 0.16
=cut
our $VERSION;
BEGIN {
- $VERSION = '0.13';
+ $VERSION = '0.16';
}
=head1 SYNOPSIS
use indirect;
my $y = new Pear; # ok
{
- no indirect hook => sub { die "You really wanted $_[0]\->$_[1]" };
- my $z = new Pineapple 'fresh'; # croaks 'You really wanted Pineapple->new'
+ no indirect hook => sub { die "You really wanted $_[0]\->$_[1] at $_[2]:$_[3]" };
+ my $z = new Pineapple 'fresh'; # croaks 'You really wanted Pineapple->new at blurp.pm:13'
}
}
+ try { ... }; # warns
+
no indirect ':fatal';
if (defied $foo) { ... } # croaks, note the typo
=head1 DESCRIPTION
When enabled (or disabled as some may prefer to say, since you actually turn it on by calling C<no indirect>), this pragma warns about indirect object syntax constructs that may have slipped into your code.
-This syntax is now considered harmful, since its parsing has many quirks and its use is error prone (when C<sub> isn't defined, C<sub $x> is actually interpreted as C<< $x->sub >>).
+This syntax is now considered harmful, since its parsing has many quirks and its use is error prone (when C<swoosh> isn't defined, C<swoosh $x> actually compiles to C<< $x->swoosh >>).
-It currently does not warn when the object is enclosed between braces (like C<meth { $obj } @args>) or for core functions (C<print> or C<say>).
+It currently does not warn for core functions (C<print>, C<say>, C<exec> or C<system>).
This may change in the future, or may be added as optional features that would be enabled by passing options to C<unimport>.
This module is B<not> a source filter.
=item *
-If the key/value pair C<< hook => $hook >> comes first, C<$hook> will be called for each error with the object name as C<$_[0]>, the method name as C<$_[1]>, the current file as C<$_[2]> and the line number as C<$_[3]>.
+If the key/value pair C<< hook => $hook >> comes first, C<$hook> will be called for each error with a string representation of the object as C<$_[0]>, the method name as C<$_[1]>, the current file as C<$_[2]> and the line number as C<$_[3]>.
+If and only if the object is actually a block, C<$_[0]> is assured to start by C<'{'>.
=item *
=cut
-my $msg = sub {
- "Indirect call of method \"$_[1]\" on object \"$_[0]\" at $_[2] line $_[3].\n"
-};
-
sub unimport {
shift;
if ($arg eq 'hook') {
$hook = shift;
} elsif ($arg eq ':fatal') {
- $hook = sub { die $msg->(@_) };
+ $hook = sub { die msg(@_) };
}
last if $hook;
}
- $hook = sub { warn $msg->(@_) } unless defined $hook;
+ $hook = sub { warn msg(@_) } unless defined $hook;
$^H |= 0x00020000;
$^H{+(__PACKAGE__)} = _tag($hook);
();
}
+=head1 FUNCTIONS
+
+=head2 C<msg $object, $method, $file, $line>
+
+Returns the default error message generated by C<indirect> when an invalid construct is reported.
+
+=cut
+
+sub msg {
+ my $obj = $_[0];
+
+ join ' ', "Indirect call of method \"$_[1]\" on",
+ ($obj =~ /^\s*\{/ ? "a block" : "object \"$obj\""),
+ "at $_[2] line $_[3].\n";
+};
+
=head1 CONSTANTS
=head2 C<I_THREADSAFE>
-True iff the module could have been built when thread-safety features.
+True iff the module could have been built with thread-safety features enabled.
+
+=head1 DIAGNOSTICS
+
+=head2 C<Indirect call of method "%s" on object "%s" at %s line %d.>
+
+The default warning/exception message thrown when an indirect call on an object is found.
+
+=head2 C<Indirect call of method "%s" on a block at %s line %d.>
+
+The default warning/exception message thrown when an indirect call on a block is found.
=head1 CAVEATS
+The implementation was tweaked to work around several limitations of vanilla C<perl> pragmas : it's thread safe, and doesn't suffer from a C<perl 5.8.x-5.10.0> bug that causes all pragmas to propagate into C<require>d scopes.
+
C<meth $obj> (no semicolon) at the end of a file won't be seen as an indirect object syntax, although it will as soon as there is another token before the end (as in C<meth $obj;> or C<meth $obj 1>).
With 5.8 perls, the pragma does not propagate into C<eval STRING>.
Bram, for motivation and advices.
+Andrew Main and Florian Ragwitz, for testing on real-life code and reporting issues.
+
=head1 COPYRIGHT & LICENSE
Copyright 2008-2009 Vincent Pit, all rights reserved.