doc formatting and spelling fixes
[gitmo/MooseX-Types-Structured.git] / lib / MooseX / Types / Structured.pm
index ecea251..a239fc8 100644 (file)
@@ -1,23 +1,18 @@
 package MooseX::Types::Structured;
+# ABSTRACT: Structured Type Constraints for Moose
 
 use 5.008;
 
-use Moose::Util::TypeConstraints;
+use Moose::Util::TypeConstraints 1.06;
 use MooseX::Meta::TypeConstraint::Structured;
 use MooseX::Meta::TypeConstraint::Structured::Optional;
 use MooseX::Types::Structured::OverflowHandler;
-use MooseX::Types -declare => [qw(Dict Map Tuple Optional)];
-use Sub::Exporter -setup => [ qw(Dict Map Tuple Optional slurpy) ];
-use Devel::PartialDump;
+use MooseX::Types::Structured::MessageStack;
+use MooseX::Types 0.22 -declare => [qw(Dict Map Tuple Optional)];
+use Sub::Exporter 0.982 -setup => [ qw(Dict Map Tuple Optional slurpy) ];
+use Devel::PartialDump 0.13;
 use Scalar::Util qw(blessed);
 
-our $VERSION = '0.19';
-our $AUTHORITY = 'cpan:JJNAPIORK';
-
-=head1 NAME
-
-MooseX::Types::Structured - Structured Type Constraints for Moose
-
 =head1 SYNOPSIS
 
 The following is example usage for this module.
@@ -45,7 +40,7 @@ The following is example usage for this module.
      ],
     );
 
-       ## Remainder of your class attributes and methods
+    ## Remainder of your class attributes and methods
 
 Then you can instantiate this class with something like:
 
@@ -73,7 +68,7 @@ Or with:
         description => ['A great student!'],
     );
 
-But all of these would cause a constraint error for the 'name' attribute:
+But all of these would cause a constraint error for the C<name> attribute:
 
     ## Value for 'name' not a HashRef
     Person->new( name => 'John' );
@@ -96,7 +91,7 @@ But all of these would cause a constraint error for the 'name' attribute:
         last => 'Li',
     });
 
-And these would cause a constraint error for the 'description' attribute:
+And these would cause a constraint error for the C<description> attribute:
 
     ## Should be an ArrayRef
     Person->new( description => 'Hello I am a String' );
@@ -112,13 +107,13 @@ Please see the test cases for more examples.
 =head1 DESCRIPTION
 
 A structured type constraint is a standard container L<Moose> type constraint,
-such as an ArrayRef or HashRef, which has been enhanced to allow you to
+such as an C<ArrayRef> or C<HashRef>, which has been enhanced to allow you to
 explicitly name all the allowed type constraints inside the structure.  The
 generalized form is:
 
     TypeConstraint[@TypeParameters or %TypeParameters]
 
-Where 'TypeParameters' is an array reference or hash references of
+Where C<TypeParameters> is an array reference or hash references of
 L<Moose::Meta::TypeConstraint> objects.
 
 This type library enables structured type constraints. It is built on top of the
@@ -128,7 +123,7 @@ if you are not familiar with it.
 =head2 Comparing Parameterized types to Structured types
 
 Parameterized constraints are built into core Moose and you are probably already
-familar with the type constraints 'HashRef' and 'ArrayRef'.  Structured types
+familiar with the type constraints C<HashRef> and C<ArrayRef>.  Structured types
 have similar functionality, so their syntax is likewise similar. For example,
 you could define a parameterized constraint like:
 
@@ -142,15 +137,15 @@ other hand, a structured type constraint explicitly names all it's allowed
     subtype StringFollowedByInt,
      as Tuple[Str,Int];
 
-would constrain it's value to things like ['hello', 111] but ['hello', 'world']
-would fail, as well as ['hello', 111, 'world'] and so on.  Here's another
+would constrain its value to things like C<< ['hello', 111] >>  but C<< ['hello', 'world'] >>
+would fail, as well as C<< ['hello', 111, 'world'] >> and so on.  Here's another
 example:
 
        package MyApp::Types;
 
     use MooseX::Types -declare [qw(StringIntOptionalHashRef)];
     use MooseX::Types::Moose qw(Str Int);
-       use MooseX::Types::Structured qw(Tuple Optional);
+    use MooseX::Types::Structured qw(Tuple Optional);
 
     subtype StringIntOptionalHashRef,
      as Tuple[
@@ -164,18 +159,18 @@ This defines a type constraint that validates values like:
     ['World', 200];
 
 Notice that the last type constraint in the structure is optional.  This is
-enabled via the helper Optional type constraint, which is a variation of the
-core Moose type constraint 'Maybe'.  The main difference is that Optional type
-constraints are required to validate if they exist, while 'Maybe' permits
+enabled via the helper C<Optional> type constraint, which is a variation of the
+core Moose type constraint C<Maybe>.  The main difference is that C<Optional> type
+constraints are required to validate if they exist, while C<Maybe> permits
 undefined values.  So the following example would not validate:
 
     StringIntOptionalHashRef->validate(['Hello Undefined', 1000, undef]);
 
 Please note the subtle difference between undefined and null.  If you wish to
-allow both null and undefined, you should use the core Moose 'Maybe' type
+allow both null and undefined, you should use the core Moose C<Maybe> type
 constraint instead:
 
-       package MyApp::Types;
+    package MyApp::Types;
 
     use MooseX::Types -declare [qw(StringIntMaybeHashRef)];
     use MooseX::Types::Moose qw(Str Int Maybe);
@@ -193,7 +188,7 @@ This would validate the following:
     ['World', 200];
 
 Structured constraints are not limited to arrays.  You can define a structure
-against a HashRef with the 'Dict' type constaint as in this example:
+against a C<HashRef> with the C<Dict> type constraint as in this example:
 
     subtype FirstNameLastName,
      as Dict[
@@ -201,7 +196,7 @@ against a HashRef with the 'Dict' type constaint as in this example:
         lastname => Str,
      ];
 
-This would constrain a HashRef that validates something like:
+This would constrain a C<HashRef> that validates something like:
 
     {firstname => 'Christopher', lastname => 'Parsons'};
 
@@ -228,26 +223,26 @@ combine various structured, parameterized and simple constraints all together:
 
 Which would match:
 
-       [1, {name=>'John', age=>25},[10,11,12]];
+    [1, {name=>'John', age=>25},[10,11,12]];
 
 Please notice how the type parameters can be visually arranged to your liking
 and to improve the clarity of your meaning.  You don't need to run then
-altogether onto a single line.  Additionally, since the 'Dict' type constraint
+altogether onto a single line.  Additionally, since the C<Dict> type constraint
 defines a hash constraint, the key order is not meaningful.  For example:
 
-       subtype AnyKeyOrder,
-        as Dict[
-               key1=>Int,
-               key2=>Str,
-               key3=>Int,
-        ];
+    subtype AnyKeyOrder,
+      as Dict[
+        key1=>Int,
+        key2=>Str,
+        key3=>Int,
+     ];
 
 Would validate both:
 
-       {key1 => 1, key2 => "Hi!", key3 => 2};
-       {key2 => "Hi!", key1 => 100, key3 => 300};
+    {key1 => 1, key2 => "Hi!", key3 => 2};
+    {key2 => "Hi!", key1 => 100, key3 => 300};
 
-As you would expect, since underneath its just a plain old Perl hash at work.
+As you would expect, since underneath it's just a plain old Perl hash at work.
 
 =head2 Alternatives
 
@@ -273,7 +268,7 @@ example:
         ),
     );
 
-This method may take some additional time to setup but will give you more
+This method may take some additional time to set up but will give you more
 flexibility.  However, structured constraints are highly compatible with this
 method, granting some interesting possibilities for coercion.  Try:
 
@@ -341,6 +336,8 @@ If you are not familiar with how coercions work, check out the L<Moose> cookbook
 entry L<Moose::Cookbook::Recipe5> for an explanation.  The section L</Coercions>
 has additional examples and discussion.
 
+=for stopwords Subtyping
+
 =head2 Subtyping a Structured type constraint
 
 You need to exercise some care when you try to subtype a structured type as in
@@ -407,10 +404,10 @@ Coercions currently work for 'one level' deep.  That is you can do:
      };
 
 And that should just work as expected.  However, if there are any 'inner'
-coercions, such as a coercion on 'Fullname' or on 'DateTime', that coercion
+coercions, such as a coercion on C<Fullname> or on C<DateTime>, that coercion
 won't currently get activated.
 
-Please see the test '07-coerce.t' for a more detailed example.  Discussion on
+Please see the test F<07-coerce.t> for a more detailed example.  Discussion on
 extending coercions to support this welcome on the Moose development channel or
 mailing list.
 
@@ -420,55 +417,55 @@ Newer versions of L<MooseX::Types> support recursive type constraints.  That is
 you can include a type constraint as a contained type constraint of itself.  For
 example:
 
-       subtype Person,
-        as Dict[
-               name=>Str,
-               friends=>Optional[
-                       ArrayRef[Person]
-               ],
-        ];
-
-This would declare a Person subtype that contains a name and an optional
-ArrayRef of Persons who are friends as in:
-
-       {
-               name => 'Mike',
-               friends => [
-                       { name => 'John' },
-                       { name => 'Vincent' },
-                       {
-                               name => 'Tracey',
-                               friends => [
-                                       { name => 'Stephenie' },
-                                       { name => 'Ilya' },
-                               ],
-                       },
-               ],
-       };
-
-Please take care to make sure the recursion node is either Optional, or declare
-a Union with an non recursive option such as:
-
-       subtype Value
-        as Tuple[
-               Str,
-               Str|Tuple,
-        ];
+    subtype Person,
+     as Dict[
+         name=>Str,
+         friends=>Optional[
+             ArrayRef[Person]
+         ],
+     ];
+
+This would declare a C<Person> subtype that contains a name and an optional
+C<ArrayRef> of C<Person>s who are friends as in:
+
+    {
+        name => 'Mike',
+        friends => [
+            { name => 'John' },
+            { name => 'Vincent' },
+            {
+                name => 'Tracey',
+                friends => [
+                    { name => 'Stephenie' },
+                    { name => 'Ilya' },
+                ],
+            },
+        ],
+    };
+
+Please take care to make sure the recursion node is either C<Optional>, or declare
+a union with an non-recursive option such as:
+
+    subtype Value
+     as Tuple[
+         Str,
+         Str|Tuple,
+     ];
 
 Which validates:
 
-       [
-               'Hello', [
-                       'World', [
-                               'Is', [
-                                       'Getting',
-                                       'Old',
-                               ],
-                       ],
-               ],
-       ];
-
-Otherwise you will define a subtype thatis impossible to validate since it is
+    [
+        'Hello', [
+            'World', [
+                'Is', [
+                    'Getting',
+                    'Old',
+                ],
+            ],
+        ],
+    ];
+
+Otherwise you will define a subtype that is impossible to validate since it is
 infinitely recursive.  For more information about defining recursive types,
 please see the documentation in L<MooseX::Types> and the test cases.
 
@@ -498,23 +495,23 @@ hashref.  For example:
 
     Dict[name=>Str, age=>Int]; ## Validates {name=>'John', age=>39}
 
-The keys in %constraints follow the same rules as @constraints in the above
+The keys in C<%constraints> follow the same rules as C<@constraints> in the above
 section.
 
 =head2 Map[ $key_constraint, $value_constraint ]
 
-This defines a HashRef based constraint in which both the keys and values are
+This defines a C<HashRef>-based constraint in which both the keys and values are
 required to meet certain constraints.  For example, to map hostnames to IP
 addresses, you might say:
 
   Map[ HostName, IPAddress ]
 
-The type constraint would only be met if every key was a valid HostName and
-every value was a valid IPAddress.
+The type constraint would only be met if every key was a valid C<HostName> and
+every value was a valid C<IPAddress>.
 
 =head2 Optional[$constraint]
 
-This is primarily a helper constraint for Dict and Tuple type constraints.  What
+This is primarily a helper constraint for C<Dict> and C<Tuple> type constraints.  What
 this allows is for you to assert that a given type constraint is allowed to be
 null (but NOT undefined).  If the value is null, then the type constraint passes
 but if the value is defined it must validate against the type constraint.  This
@@ -527,7 +524,7 @@ or a tuple where some of the values are not required.  For example:
         middle=>Optional[Str],
     ];
 
-Creates a constraint that validates against a hashref with the keys 'first' and
+...creates a constraint that validates against a hashref with the keys 'first' and
 'last' being strings and required while an optional key 'middle' is must be a
 string if it appears but doesn't have to appear.  So in this case both the
 following are valid:
@@ -535,13 +532,15 @@ following are valid:
     {first=>'John', middle=>'James', last=>'Napiorkowski'}
     {first=>'Vanessa', last=>'Li'}
 
-If you use the 'Maybe' type constraint instead, your values will also validate
-against 'undef', which may be incorrect for you.
+If you use the C<Maybe> type constraint instead, your values will also validate
+against C<undef>, which may be incorrect for you.
 
 =head1 EXPORTABLE SUBROUTINES
 
 This type library makes available for export the following subroutines
 
+=for stopwords slurpy
+
 =head2 slurpy
 
 Structured type constraints by their nature are closed; that is validation will
@@ -584,11 +583,11 @@ This will now work as expected, validating ArrayRef structures such as:
 A few caveats apply.  First, the slurpy type constraint must be the last one in
 the list of type constraint parameters.  Second, the parent type of the slurpy
 type constraint must match that of the containing type constraint.  That means
-that a Tuple can allow a slurpy ArrayRef (or children of ArrayRefs, including
-another Tuple) and a Dict can allow a slurpy HashRef (or children/subtypes of
-HashRef, also including other Dict constraints).
+that a C<Tuple> can allow a slurpy C<ArrayRef> (or children of C<ArrayRef>s, including
+another C<Tuple>) and a C<Dict> can allow a slurpy C<HashRef> (or children/subtypes of
+HashRef, also including other C<Dict> constraints).
 
-Please note the the technical way this works 'under the hood' is that the
+Please note the technical way this works 'under the hood' is that the
 slurpy keyword transforms the target type constraint into a coderef.  Please do
 not try to create your own custom coderefs; always use the slurpy method.  The
 underlying technology may change in the future but the slurpy keyword will be
@@ -619,9 +618,9 @@ be found also in the 't/examples.t' test.  Your contributions are also welcomed.
 =head2 Normalize a HashRef
 
 You need a hashref to conform to a canonical structure but are required accept a
-bunch of different incoming structures.  You can normalize using the Dict type
+bunch of different incoming structures.  You can normalize using the C<Dict> type
 constraint and coercions.  This example also shows structured types mixed which
-other MooseX::Types libraries.
+other L<MooseX::Types> libraries.
 
     package Test::MooseX::Meta::TypeConstraint::Structured::Examples::Normalize;
 
@@ -635,25 +634,25 @@ other MooseX::Types libraries.
 
     subtype Person,
      as Dict[
-       name=>Str,
-       age=>Int,
+         name=>Str,
+         age=>Int,
      ];
 
     coerce Person,
      from Dict[
-       first=>Str,
-       last=>Str,
-       years=>Int,
+         first=>Str,
+         last=>Str,
+         years=>Int,
      ], via { +{
         name => "$_->{first} $_->{last}",
         age => $_->{years},
      }},
      from Dict[
-       fullname=>Dict[
-               last=>Str,
-               first=>Str,
-       ],
-       dob=>DateTime,
+         fullname=>Dict[
+             last=>Str,
+             first=>Str,
+         ],
+         dob=>DateTime,
      ],
      ## DateTime needs to be inside of single quotes here to disambiguate the
      ## class package from the DataTime type constraint imported via the
@@ -724,17 +723,37 @@ my $Optional = MooseX::Meta::TypeConstraint::Structured::Optional->new(
     }
 );
 
+my $IsType = sub {
+    my ($obj, $type) = @_;
+
+    return $obj->can('equals')
+        ? $obj->equals($type)
+        : undef;
+};
+
+my $CompiledTC = sub {
+    my ($obj) = @_;
+
+    my $method = '_compiled_type_constraint';
+    return(
+          $obj->$IsType('Any')  ? undef
+        : $obj->can($method)    ? $obj->$method
+        :                         sub { $obj->check(shift) },
+    );
+};
+
 Moose::Util::TypeConstraints::register_type_constraint($Optional);
 Moose::Util::TypeConstraints::add_parameterizable_type($Optional);
 
 Moose::Util::TypeConstraints::get_type_constraint_registry->add_type_constraint(
-       MooseX::Meta::TypeConstraint::Structured->new(
-               name => "MooseX::Types::Structured::Tuple" ,
-               parent => find_type_constraint('ArrayRef'),
-               constraint_generator=> sub {
-                       ## Get the constraints and values to check
-            my ($type_constraints, $values) = @_;
-                       my @type_constraints = defined $type_constraints ?
+    MooseX::Meta::TypeConstraint::Structured->new(
+        name => "MooseX::Types::Structured::Tuple" ,
+        parent => find_type_constraint('ArrayRef'),
+        constraint_generator=> sub {
+            ## Get the constraints and values to check
+            my ($self, $type_constraints) = @_;
+            $type_constraints ||= $self->type_constraints;
+            my @type_constraints = defined $type_constraints ?
              @$type_constraints : ();
 
             my $overflow_handler;
@@ -743,55 +762,87 @@ Moose::Util::TypeConstraints::get_type_constraint_registry->add_type_constraint(
                 $overflow_handler = pop @type_constraints;
             }
 
-                       my @values = defined $values ? @$values: ();
-                       ## Perform the checking
-                       while(@type_constraints) {
-                               my $type_constraint = shift @type_constraints;
-                               if(@values) {
-                                       my $value = shift @values;
-                                       unless($type_constraint->check($value)) {
-                        $_[2]->{message} = $type_constraint->get_message($value)
-                         if ref $_[2];
-                                               return;
-                                       }
-                               } else {
-                    ## Test if the TC supports null values
-                    unless ($type_constraint->is_subtype_of($Optional)) {
-                        $_[2]->{message} = $type_constraint->get_message('NULL')
-                         if ref $_[2];
-                                               return;
-                                       }
-                               }
-                       }
-                       ## Make sure there are no leftovers.
-                       if(@values) {
-                if($overflow_handler) {
-                    return $overflow_handler->check([@values], $_[2]);
+            my $length = $#type_constraints;
+            foreach my $idx (0..$length) {
+                unless(blessed $type_constraints[$idx]) {
+                    ($type_constraints[$idx] = find_type_constraint($type_constraints[$idx]))
+                      || die "$type_constraints[$idx] is not a registered type";
+                }
+            }
+
+            my (@checks, @optional, $o_check, $is_compiled);
+            return sub {
+                my ($values, $err) = @_;
+                my @values = defined $values ? @$values : ();
+
+                ## initialise on first time run
+                unless ($is_compiled) {
+                    @checks   = map { $_->$CompiledTC } @type_constraints;
+                    @optional = map { $_->is_subtype_of($Optional) } @type_constraints;
+                    $o_check  = $overflow_handler->$CompiledTC
+                        if $overflow_handler;
+                    $is_compiled++;
+                }
+
+                ## Perform the checking
+              VALUE:
+                for my $type_index (0 .. $#checks) {
+
+                    my $type_constraint = $checks[ $type_index ];
+
+                    if(@values) {
+                        my $value = shift @values;
+
+                        next VALUE
+                            unless $type_constraint;
+
+                        unless($type_constraint->($value)) {
+                            if($err) {
+                               my $message = $type_constraints[ $type_index ]->validate($value,$err);
+                               $err->add_message({message=>$message,level=>$err->level});
+                            }
+                            return;
+                        }
+                    } else {
+                        ## Test if the TC supports null values
+                        unless ($optional[ $type_index ]) {
+                            if($err) {
+                               my $message = $type_constraints[ $type_index ]->get_message('NULL',$err);
+                               $err->add_message({message=>$message,level=>$err->level});
+                            }
+                            return;
+                        }
+                    }
+                }
+
+                ## Make sure there are no leftovers.
+                if(@values) {
+                    if($overflow_handler) {
+                        return $o_check->([@values], $err);
+                    } else {
+                        if($err) {
+                            my $message = "More values than Type Constraints!";
+                            $err->add_message({message=>$message,level=>$err->level});
+                        }
+                        return;
+                    }
                 } else {
-                    $_[2]->{message} = "More values than Type Constraints!"
-                     if ref $_[2];
-                    return;
+                    return 1;
                 }
-                       } elsif(@type_constraints) {
-                $_[2]->{message} =
-                 "Not enough values for all defined type constraints.  Remaining: ". join(', ',@type_constraints)
-                 if ref $_[2];
-                               return;
-                       } else {
-                               return 1;
-                       }
-               }
-       )
+            };
+        }
+    )
 );
 
 Moose::Util::TypeConstraints::get_type_constraint_registry->add_type_constraint(
-       MooseX::Meta::TypeConstraint::Structured->new(
-               name => "MooseX::Types::Structured::Dict",
-               parent => find_type_constraint('HashRef'),
-               constraint_generator=> sub {
-                       ## Get the constraints and values to check
-            my ($type_constraints, $values) = @_;
-                       my @type_constraints = defined $type_constraints ?
+    MooseX::Meta::TypeConstraint::Structured->new(
+        name => "MooseX::Types::Structured::Dict",
+        parent => find_type_constraint('HashRef'),
+        constraint_generator => sub {
+            ## Get the constraints and values to check
+            my ($self, $type_constraints) = @_;
+            $type_constraints = $self->type_constraints;
+            my @type_constraints = defined $type_constraints ?
              @$type_constraints : ();
 
             my $overflow_handler;
@@ -799,48 +850,75 @@ Moose::Util::TypeConstraints::get_type_constraint_registry->add_type_constraint(
               && $type_constraints[-1]->isa('MooseX::Types::Structured::OverflowHandler')) {
                 $overflow_handler = pop @type_constraints;
             }
-            my (%type_constraints) = @type_constraints;
-                       my %values = defined $values ? %$values: ();
-                       ## Perform the checking
-                       while(%type_constraints) {
-                               my($key, $type_constraint) = each %type_constraints;
-                               delete $type_constraints{$key};
-                               if(exists $values{$key}) {
-                                       my $value = $values{$key};
-                                       delete $values{$key};
-                                       unless($type_constraint->check($value)) {
-                        $_[2]->{message} = $type_constraint->get_message($value)
-                         if ref $_[2];
-                                               return;
-                                       }
-                               } else {
-                    ## Test to see if the TC supports null values
-                    unless ($type_constraint->is_subtype_of($Optional)) {
-                        $_[2]->{message} = $type_constraint->get_message('NULL')
-                         if ref $_[2];
-                                               return;
-                                       }
-                               }
-                       }
-                       ## Make sure there are no leftovers.
-                       if(%values) {
-                if($overflow_handler) {
-                    return $overflow_handler->check(+{%values});
+            my %type_constraints = @type_constraints;
+            foreach my $key (keys %type_constraints) {
+                unless(blessed $type_constraints{$key}) {
+                    ($type_constraints{$key} = find_type_constraint($type_constraints{$key}))
+                      || die "$type_constraints{$key} is not a registered type";
+                }
+            }
+
+            my (%check, %optional, $o_check, $is_compiled);
+            return sub {
+                my ($values, $err) = @_;
+                my %values = defined $values ? %$values: ();
+
+                unless ($is_compiled) {
+                    %check    = map { ($_ => $type_constraints{ $_ }->$CompiledTC) } keys %type_constraints;
+                    %optional = map { ($_ => $type_constraints{ $_ }->is_subtype_of($Optional)) } keys %type_constraints;
+                    $o_check  = $overflow_handler->$CompiledTC
+                        if $overflow_handler;
+                    $is_compiled++;
+                }
+
+                ## Perform the checking
+              KEY:
+                for my $key (keys %check) {
+                    my $type_constraint = $check{ $key };
+
+                    if(exists $values{$key}) {
+                        my $value = $values{$key};
+                        delete $values{$key};
+
+                        next KEY
+                            unless $type_constraint;
+
+                        unless($type_constraint->($value)) {
+                            if($err) {
+                                my $message = $type_constraints{ $key }->validate($value,$err);
+                                $err->add_message({message=>$message,level=>$err->level});
+                            }
+                            return;
+                        }
+                    } else {
+                        ## Test to see if the TC supports null values
+                        unless ($optional{ $key }) {
+                            if($err) {
+                               my $message = $type_constraints{ $key }->get_message('NULL',$err);
+                               $err->add_message({message=>$message,level=>$err->level});
+                            }
+                            return;
+                        }
+                    }
+                }
+
+                ## Make sure there are no leftovers.
+                if(%values) {
+                    if($overflow_handler) {
+                        return $o_check->(+{%values});
+                    } else {
+                        if($err) {
+                            my $message = "More values than Type Constraints!";
+                            $err->add_message({message=>$message,level=>$err->level});
+                        }
+                        return;
+                    }
                 } else {
-                    $_[2]->{message} = "More values than Type Constraints!"
-                     if ref $_[2];
-                    return;
+                    return 1;
                 }
-                       } elsif(%type_constraints) {
-                $_[2]->{message} =
-                 "Not enough values for all defined type constraints.  Remaining: ". join(', ',values %values)
-                 if ref $_[2];
-                               return;
-                       } else {
-                               return 1;
-                       }
-               },
-       )
+            }
+        },
+    )
 );
 
 Moose::Util::TypeConstraints::get_type_constraint_registry->add_type_constraint(
@@ -849,7 +927,8 @@ Moose::Util::TypeConstraints::get_type_constraint_registry->add_type_constraint(
     parent => find_type_constraint('HashRef'),
     constraint_generator=> sub {
       ## Get the constraints and values to check
-      my ($type_constraints, $values) = @_;
+      my ($self, $type_constraints) = @_;
+      $type_constraints = $self->type_constraints;
       my @constraints = defined $type_constraints ? @$type_constraints : ();
 
       Carp::confess( "too many args for Map type" ) if @constraints > 2;
@@ -858,34 +937,51 @@ Moose::Util::TypeConstraints::get_type_constraint_registry->add_type_constraint(
                                   : @constraints == 1 ? (undef, @constraints)
                                   :                     ();
 
-      my %values = defined $values ? %$values: ();
-      ## Perform the checking
-      if ($value_type) {
-        for my $value (values %$values) {
-          unless ($value_type->check($value)) {
-            $_[2]->{message} = $value_type->get_message($value) if ref $_[2];
-            return;
+      my ($key_check, $value_check, $is_compiled);
+      return sub {
+          my ($values, $err) = @_;
+          my %values = defined $values ? %$values: ();
+
+          unless ($is_compiled) {
+              ($key_check, $value_check)
+                = map { $_ ? $_->$CompiledTC : undef }
+                      $key_type, $value_type;
+              $is_compiled++;
           }
-        }
-      }
 
-      if ($key_type) {
-        for my $key (keys %$values) {
-          unless ($key_type->check($key)) {
-            $_[2]->{message} = $key_type->get_message($key) if ref $_[2];
-            return;
+          ## Perform the checking
+          if ($value_check) {
+            for my $value (values %$values) {
+              unless ($value_check->($value)) {
+                if($err) {
+                  my $message = $value_type->validate($value,$err);
+                  $err->add_message({message=>$message,level=>$err->level});
+                }
+                return;
+              }
+            }
+          }
+          if ($key_check) {
+            for my $key (keys %$values) {
+              unless ($key_check->($key)) {
+                if($err) {
+                  my $message = $key_type->validate($key,$err);
+                  $err->add_message({message=>$message,level=>$err->level});
+                }
+                return;
+              }
+            }
           }
-        }
-      }
 
-      return 1;
+          return 1;
+      };
     },
   )
 );
 
 sub slurpy ($) {
-       my ($tc) = @_;
-       return MooseX::Types::Structured::OverflowHandler->new(
+    my ($tc) = @_;
+    return MooseX::Types::Structured::OverflowHandler->new(
         type_constraint => $tc,
     );
 }
@@ -897,35 +993,6 @@ The following modules or resources may be of interest.
 L<Moose>, L<MooseX::Types>, L<Moose::Meta::TypeConstraint>,
 L<MooseX::Meta::TypeConstraint::Structured>
 
-=head1 TODO
-
-Here's a list of stuff I would be happy to get volunteers helping with:
-
-       * All POD examples need test cases in t/documentation/*.t
-       * Want to break out the examples section to a separate cookbook style POD.
-       * Want more examples and best practice / usage guidance for authors
-       * Need to clarify deep coercions,
-
-=head1 AUTHOR
-
-John Napiorkowski <jjnapiork@cpan.org>
-
-=head1 CONTRIBUTORS
-
-The following people have contributed to this module and agree with the listed
-Copyright & license information included below:
-
-    Florian Ragwitz, <rafl@debian.org>
-    Yuval Kogman, <nothingmuch@woobling.org>
-    Tomas Doran, <bobtfish@bobtfish.net>
-
-=head1 COPYRIGHT & LICENSE
-
-Copyright 2008-2009, John Napiorkowski <jjnapiork@cpan.org>
-
-This program is free software; you can redistribute it and/or modify it under
-the same terms as Perl itself.
-
 =cut
 
 1;