show eltype for Zeros/Ones (#283)

* show eltype for Zeros/Ones

* Bump version to v1.5.0

Author: jishnub

src/FillArrays.jl

@@ -781,11 +781,19 @@ function Base.show(io::IO, ::MIME"text/plain", x::Union{Eye, AbstractFill})
 end
 
 function Base.show(io::IO, x::AbstractFill)  # for example (Fill(π,3),)
-    print(io, nameof(typeof(x)), "(")
-    if x isa Union{Zeros, Ones}
+    print(io, nameof(typeof(x)))
+    sz = size(x)
+    args = if x isa Union{Zeros, Ones}
+        T = eltype(x)
+        if T != Float64
+            print(io,"{", T, "}")
+        end
+        print(io, "(")
     else
-        show(io, getindex_value(x))  # show not print to handle (Fill(1f0,2),)
-        ndims(x) > 0 && print(io, ", ")
+        # show, not print, to handle (Fill(1f0,2),)
+        print(io, "(")
+        show(io, getindex_value(x))
+        ndims(x) == 0 || print(io, ", ")
     end
     join(io, size(x), ", ")
     print(io, ")")

test/runtests.jl

@@ -1725,7 +1725,11 @@ end
     @test Base.replace_in_print_matrix(Zeros(5,3), 1, 2, "0.0") == " ⋅ "
 
     # 2-arg show, compact printing
+    @test repr(Zeros{Int}()) == "Zeros{$Int}()"
+    @test repr(Zeros{Int}(3)) == "Zeros{$Int}(3)"
     @test repr(Zeros(3)) == "Zeros(3)"
+    @test repr(Ones{Int}(3)) == "Ones{$Int}(3)"
+    @test repr(Ones{Int}(3,2)) == "Ones{$Int}(3, 2)"
     @test repr(Ones(3,2)) == "Ones(3, 2)"
     @test repr(Fill(7,3,2)) == "Fill(7, 3, 2)"
     @test repr(Fill(1f0,10)) == "Fill(1.0f0, 10)"  # Float32!