Improve any/all support

Author: dlfivefifty

src/FillArrays.jl

@@ -426,8 +426,11 @@ function all(f::Function, IM::Eye{T}) where T
 end
 
 # In particular, these make iszero(Eye(n))  efficient.
-any(f::Function, x::AbstractFill) = f(getindex_value(x))
-all(f::Function, x::AbstractFill) = f(getindex_value(x))
+# use any/all on scalar to get Boolean error message
+any(f::Function, x::AbstractFill) = any(f, getindex_value(x))
+all(f::Function, x::AbstractFill) = all(f, getindex_value(x))
+any(x::AbstractFill) = any(getindex_value(x))
+all(x::AbstractFill) = all(getindex_value(x))
 
 include("fillalgebra.jl")
 include("fillbroadcast.jl")

test/runtests.jl

@@ -650,6 +650,22 @@ end
             @test ! iszero(m4)
         end
     end
+
+    @testset "all/any" begin
+        @test any(Ones{Bool}(10)) === all(Ones{Bool}(10)) === any(Fill(true,10)) === all(Fill(true,10)) === true
+        @test any(Zeros{Bool}(10)) === all(Zeros{Bool}(10)) === any(Fill(false,10)) === all(Fill(false,10)) === false
+    end
+
+    @testset "Error" begin
+        @test_throws TypeError any(exp, Fill(1,5))
+        @test_throws TypeError all(exp, Fill(1,5))
+        @test_throws TypeError any(Fill(1,5))
+        @test_throws TypeError all(Fill(1,5))
+        @test_throws TypeError any(Zeros(5))
+        @test_throws TypeError all(Zeros(5))
+        @test_throws TypeError any(Ones(5))
+        @test_throws TypeError all(Ones(5))
+    end
 end
 
 @testset "Eye identity ops" begin