use base qw(Moo::Object);
use Sub::Quote;
use B 'perlstring';
+use Scalar::Util 'blessed';
+use overload ();
BEGIN {
our $CAN_HAZ_XS =
!$ENV{MOO_XS_DISABLE}
if (($spec->{trigger}||0) eq 1) {
$spec->{trigger} = quote_sub('shift->_trigger_'.$name.'(@_)');
}
- if (exists $spec->{default}) {
- my $default = $spec->{default};
- # default can be either a coderef or an overloaded object
- die "Invalid default $default" unless ref $default
- and ( ref $default eq 'CODE' or eval { \&$default } );
+
+ for my $setting (qw( default coerce )) {
+ next if !exists $spec->{$setting};
+ my $value = $spec->{$setting};
+ my $invalid = "Invalid $setting '" . overload::StrVal($value)
+ . "' for $into->$name - not a coderef";
+ die "$invalid or code-convertible object"
+ unless ref $value and (ref $value eq 'CODE' or blessed($value));
+ die "$invalid and could not be converted to a coderef: $@"
+ if !eval { \&$value };
}
my %methods;
sub _generate_coerce {
my ($self, $name, $value, $coerce) = @_;
- $self->_generate_call_code($name, 'coerce', "${value}", $coerce);
+ $self->_generate_die_prefix(
+ "coercion for ${\perlstring($name)} failed: ",
+ $self->_generate_call_code($name, 'coerce', "${value}", $coerce)
+ );
}
sub generate_trigger {
($code, delete $self->{captures});
}
+sub _generate_die_prefix {
+ my ($self, $prefix, $inside) = @_;
+ "do {\n"
+ .' my $sig_die = $SIG{__DIE__} || sub { die $_[0] };'."\n"
+ .' local $SIG{__DIE__} = sub {'."\n"
+ .' $sig_die->(ref($_[0]) ? $_[0] : '.perlstring($prefix).'.$_[0]);'."\n"
+ .' };'."\n"
+ .$inside
+ ."}\n"
+}
+
sub _generate_isa_check {
my ($self, $name, $value, $check) = @_;
- $self->_generate_call_code($name, 'isa_check', $value, $check);
+ $self->_generate_die_prefix(
+ "isa check for ${\perlstring($name)} failed: ",
+ $self->_generate_call_code($name, 'isa_check', $value, $check)
+ );
}
sub _generate_call_code {
my ($self, $name, $type, $values, $sub) = @_;
if (my $quoted = quoted_from_sub($sub)) {
my $code = $quoted->[1];
- my $at_ = '@_ = ('.$values.');';
if (my $captures = $quoted->[2]) {
my $cap_name = qq{\$${type}_captures_for_${name}};
$self->{captures}->{$cap_name} = \$captures;
Sub::Quote::inlinify(
- $code, $values, Sub::Quote::capture_unroll($cap_name, $captures, 6)
+ $code, $values, Sub::Quote::capture_unroll($cap_name, $captures, 6), 1
);
} else {
- Sub::Quote::inlinify($code, $values);
+ Sub::Quote::inlinify($code, $values, undef, 1);
}
} else {
my $cap_name = qq{\$${type}_for_${name}};