Restore original .defined semantic in .minmax

Except that a Failure will always throw. Only noise-levels slower.
rakudo · May 1, 2016 · 6df0539 · 6df0539
1 parent 4379cf0
commit 6df0539
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/src/core/Any-iterable-methods.pm b/src/core/Any-iterable-methods.pm
@@ -781,7 +781,10 @@ Did you mean to add a stub (\{...\}) or did you mean to .classify?"
         my int $excludes-max;
 
         for self.list {
-            if nqp::defined($_) {
+            if nqp::istype($_,Failure) {
+                .throw;  # XXX or just ignore ???
+            }
+            elsif .defined {
                 if nqp::istype($_,Range) {
                     if !$min.defined || $cmp($_.min, $min) < 0 {
                         $min = .min;

diff --git a/src/core/Supply.pm b/src/core/Supply.pm
@@ -1061,7 +1061,10 @@ my class Supply {
             my $min;
             my $max;
             whenever self -> \val {
-                if nqp::defined(val) {
+                if nqp::istype(val,Failure) {
+                    val.throw;  # XXX or just ignore ???
+                }
+                elsif val.defined {
                     if !$min.defined {
                         emit( Range.new($min = val, $max = val) );
                     }