use warnings;
use List::MoreUtils qw( any );
-
-our $VERSION = '1.19';
-$VERSION = eval $VERSION;
-our $AUTHORITY = 'cpan:STEVAN';
+use Moose::Util;
use Moose::Role;
return (
'sub {',
- $self->_inline_pre_body(@_),
'my ' . $inv . ' = shift;',
$self->_inline_curried_arguments,
$self->_inline_writer_core($inv, $slot_access),
- $self->_inline_post_body(@_),
'}',
);
}
$self->_inline_check_argument_count,
$self->_inline_process_arguments($inv, $slot_access),
$self->_inline_check_arguments('for writer'),
- $self->_inline_check_lazy($inv),
+ $self->_inline_check_lazy($inv, '$type_constraint', '$type_coercion', '$type_message'),
);
if ($self->_return_value($slot_access)) {
push @code, (
$self->_inline_coerce_new_values,
$self->_inline_copy_native_value(\$potential),
- $self->_inline_tc_code($potential),
+ $self->_inline_tc_code($potential, '$type_constraint', '$type_coercion', '$type_message'),
$self->_inline_get_old_value_for_trigger($inv, $old),
$self->_inline_capture_return_value($slot_access),
$self->_inline_set_new_value($inv, $potential, $slot_access),
sub _inline_coerce_new_values { return }
-sub _value_needs_copy {
+sub _writer_value_needs_copy {
my $self = shift;
return $self->_constraint_must_be_checked;
my $attr = $self->associated_attribute;
return $attr->has_type_constraint
- && (!$self->_is_root_type( $attr->type_constraint )
- || ( $attr->should_coerce && $attr->type_constraint->has_coercion)
- );
+ && ( !$self->_is_root_type( $attr->type_constraint )
+ || ( $attr->should_coerce && $attr->type_constraint->has_coercion ) );
}
sub _is_root_type {
my $self = shift;
- my ($type) = @_;
-
- my $name = $type->name;
-
- return any { $name eq $_ } @{ $self->root_types };
+ my $type = shift;
+
+ if (
+ Moose::Util::does_role( $type, 'Type::Constraint::Role::Interface' ) )
+ {
+ require Type::Library::Builtins;
+ return
+ any { $type->is_same_type_as( Type::Library::Builtins::t($_) ) }
+ @{ $self->root_types };
+ }
+ else {
+ my $name = $type->name;
+ return any { $name eq $_ } @{ $self->root_types };
+ }
}
sub _inline_copy_native_value {
my $self = shift;
my ($potential_ref) = @_;
- return unless $self->_value_needs_copy;
+ return unless $self->_writer_value_needs_copy;
my $code = 'my $potential = ' . ${$potential_ref} . ';';
around _inline_tc_code => sub {
my $orig = shift;
my $self = shift;
- my ($value, $for_lazy) = @_;
+ my ($value, $tc, $coercion, $message, $for_lazy) = @_;
return unless $for_lazy || $self->_constraint_must_be_checked;
return $self->$orig(@_);
};
-sub _inline_check_coercion {
- my $self = shift;
- my ($value) = @_;
-
- my $attr = $self->associated_attribute;
- return unless $attr->should_coerce && $attr->type_constraint->has_coercion;
-
- # We want to break the aliasing in @_ in case the coercion tries to make a
- # destructive change to an array member.
- return $value . ' = $type_constraint_obj->coerce(' . $value . ');';
-}
-
around _inline_check_constraint => sub {
my $orig = shift;
my $self = shift;
- my ($value, $for_lazy) = @_;
+ my ($value, $tc, $message, $for_lazy) = @_;
return unless $for_lazy || $self->_constraint_must_be_checked;
my $self = shift;
return $self->_inline_store_value(@_)
- if $self->_value_needs_copy
+ if $self->_writer_value_needs_copy
|| !$self->_slot_access_can_be_inlined
|| !$self->_get_is_lvalue;