X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FMouse%2FUtil%2FTypeConstraints.pm;h=e05b870facd1ce7155b0c7d6b67419a7d5ba6ba6;hb=7e0e6837dcef39d945629cb04d5f4d7619b08034;hp=d1b68fa974460b7ae92fe4431235b83b74d9835e;hpb=4bc73e4760435ee34876be28bf4522e9e7eaf519;p=gitmo%2FMouse.git diff --git a/lib/Mouse/Util/TypeConstraints.pm b/lib/Mouse/Util/TypeConstraints.pm index d1b68fa..e05b870 100644 --- a/lib/Mouse/Util/TypeConstraints.pm +++ b/lib/Mouse/Util/TypeConstraints.pm @@ -1,7 +1,7 @@ package Mouse::Util::TypeConstraints; use Mouse::Util qw(does_role not_supported); # enables strict and warnings -use Carp qw(confess); +use Carp (); use Scalar::Util (); use Mouse::Meta::TypeConstraint; @@ -17,9 +17,12 @@ Mouse::Exporter->setup_import_methods( coerce find_type_constraint + register_type_constraint )], ); +our @CARP_NOT = qw(Mouse::Meta::Attribute); + my %TYPE; # The root type @@ -80,13 +83,13 @@ $TYPE{HashRef} {constraint_generator} = \&_parameterize_HashRef_for; # sugars -sub as ($) { (as => $_[0]) } -sub where (&) { (where => $_[0]) } -sub message (&) { (message => $_[0]) } -sub optimize_as (&) { (optimize_as => $_[0]) } +sub as ($) { (as => $_[0]) } ## no critic +sub where (&) { (where => $_[0]) } ## no critic +sub message (&) { (message => $_[0]) } ## no critic +sub optimize_as (&) { (optimize_as => $_[0]) } ## no critic sub from { @_ } -sub via (&) { $_[0] } +sub via (&) { $_[0] } ## no critic # type utilities @@ -101,7 +104,6 @@ sub list_all_builtin_type_constraints { @builtins } sub list_all_type_constraints { keys %TYPE } - sub _create_type{ my $mode = shift; @@ -148,7 +150,7 @@ sub _create_type{ if($TYPE{$name}){ my $that = $TYPE{$name}->{package_defined_in} || __PACKAGE__; - ($this eq $that) or confess( + ($this eq $that) or Carp::croak( "The type constraint '$name' has already been created in $that and cannot be created again in $this" ); } @@ -188,7 +190,7 @@ sub coerce { my $type_name = shift; my $type = find_type_constraint($type_name) - or confess("Cannot find type '$type_name', perhaps you forgot to load it."); + or Carp::croak("Cannot find type '$type_name', perhaps you forgot to load it."); $type->_add_type_coercions(@_); return; @@ -226,7 +228,8 @@ sub duck_type { @methods = (@_ == 1 && ref($_[0]) eq 'ARRAY') ? @{$_[0]} : @_; # DuckType - return _create_type 'type', $name => ( + return _create_type 'subtype', $name => ( + as => 'Object', optimized_as => Mouse::Util::generate_can_predicate_for(\@methods), ); } @@ -241,18 +244,22 @@ sub enum { %valid = map{ $_ => undef } (@_ == 1 && ref($_[0]) eq 'ARRAY' ? @{$_[0]} : @_); # EnumType - return _create_type 'type', $name => ( + return _create_type 'subtype', $name => ( + as => 'Str', optimized_as => sub{ defined($_[0]) && !ref($_[0]) && exists $valid{$_[0]} }, ); } sub _find_or_create_regular_type{ - my($spec) = @_; + my($spec, $create) = @_; return $TYPE{$spec} if exists $TYPE{$spec}; - my $meta = Mouse::Util::get_metaclass_by_name($spec) - or return undef; + my $meta = Mouse::Util::get_metaclass_by_name($spec); + + if(!defined $meta){ + return $create ? class_type($spec) : undef; + } if(Mouse::Util::is_a_metarole($meta)){ return role_type($spec); @@ -271,6 +278,7 @@ sub _find_or_create_parameterized_type{ } sub _find_or_create_union_type{ + return if grep{ not defined } @_; my @types = sort map{ $_->{type_constraints} ? @{$_->{type_constraints}} : $_ } @_; my $name = join '|', @types; @@ -283,72 +291,71 @@ sub _find_or_create_union_type{ } # The type parser -sub _parse_type{ - my($spec, $start) = @_; - - my @list; - my $subtype; - my $len = length $spec; - my $i; +# param : '[' type ']' | NOTHING +sub _parse_param { + my($c) = @_; - for($i = $start; $i < $len; $i++){ - my $char = substr($spec, $i, 1); + if($c->{spec} =~ s/^\[//){ + my $type = _parse_type($c, 1); - if($char eq '['){ - my $base = _find_or_create_regular_type( substr($spec, $start, $i - $start) ) - or return; - - ($i, $subtype) = _parse_type($spec, $i+1) - or return; - $start = $i+1; # reset - - push @list, _find_or_create_parameterized_type($base => $subtype); + if($c->{spec} =~ s/^\]//){ + return $type; } - elsif($char eq ']'){ - $len = $i+1; - last; - } - elsif($char eq '|'){ - my $type = _find_or_create_regular_type( substr($spec, $start, $i - $start) ); + Carp::croak("Syntax error in type: missing right square bracket in '$c->{orig}'"); + } - if(!defined $type){ - # XXX: Mouse creates a new class type, but Moose does not. - $type = class_type( substr($spec, $start, $i - $start) ); - } + return undef; +} - push @list, $type; +# name : [\w.:]+ +sub _parse_name { + my($c, $create) = @_; - ($i, $subtype) = _parse_type($spec, $i+1) - or return; + if($c->{spec} =~ s/\A ([\w.:]+) //xms){ + return _find_or_create_regular_type($1, $create); + } + Carp::croak("Syntax error in type: expect type name near '$c->{spec}' in '$c->{orig}'"); +} - $start = $i+1; # reset +# single_type : name param +sub _parse_single_type { + my($c, $create) = @_; - push @list, $subtype; - } - } - if($i - $start){ - my $type = _find_or_create_regular_type( substr $spec, $start, $i - $start ); + my $type = _parse_name($c, $create); + my $param = _parse_param($c); - if(defined $type){ - push @list, $type; + if(defined $type){ + if(defined $param){ + return _find_or_create_parameterized_type($type, $param); } - elsif($start != 0) { - # RT #50421 - # create a new class type - push @list, class_type( substr $spec, $start, $i - $start ); + else { + return $type; } } - - if(@list == 0){ - return; - } - elsif(@list == 1){ - return ($len, $list[0]); + elsif(defined $param){ + Carp::croak("Undefined type with parameter [$param] in '$c->{orig}'"); } else{ - return ($len, _find_or_create_union_type(@list)); + return undef; + } +} + +# type : single_type ('|' single_type)* +sub _parse_type { + my($c, $create) = @_; + + my $type = _parse_single_type($c, $create); + if($c->{spec}){ # can be an union type + my @types; + while($c->{spec} =~ s/^\|//){ + push @types, _parse_single_type($c, $create); + } + if(@types){ + return _find_or_create_union_type($type, @types); + } } + return $type; } @@ -361,6 +368,16 @@ sub find_type_constraint { return $TYPE{$spec}; } +sub register_type_constraint { + my($constraint) = @_; + Carp::croak("No type supplied / type is not a valid type constraint") + unless Mouse::Util::is_a_type_constraint($constraint); + my $name = $constraint->name; + Carp::croak("can't register an unnamed type constraint") + unless defined $name; + return $TYPE{$name} = $constraint; +} + sub find_or_parse_type_constraint { my($spec) = @_; return $spec if Mouse::Util::is_a_type_constraint($spec); @@ -368,7 +385,15 @@ sub find_or_parse_type_constraint { $spec =~ s/\s+//g; return $TYPE{$spec} || do{ - my($pos, $type) = _parse_type($spec, 0); + my $context = { + spec => $spec, + orig => $spec, + }; + my $type = _parse_type($context); + + if($context->{spec}){ + Carp::croak("Syntax error: extra elements '$context->{spec}' in '$context->{orig}'"); + } $type; }; } @@ -392,7 +417,7 @@ Mouse::Util::TypeConstraints - Type constraint system for Mouse =head1 VERSION -This document describes Mouse version 0.50_03 +This document describes Mouse version 0.63 =head2 SYNOPSIS @@ -493,10 +518,6 @@ If Mouse finds a name in brackets that it does not recognize as an existing type, it assumes that this is a class name, for example C. -B Unless you parameterize a type, then it is invalid to include -the square brackets. I.e. C will be treated as a new type -name, I as a parameterization of C. - B The C type constraint for the most part works correctly now, but edge cases may still exist, please use it sparingly.