Fix issue: copy should not copy the fill's element (#64)

Author: cstjean

src/FillArrays.jl

@@ -75,7 +75,7 @@ convert(::Type{AbstractFill}, F::AbstractFill) = F
 convert(::Type{AbstractFill{T}}, F::AbstractFill) where T = convert(AbstractArray{T}, F)
 convert(::Type{AbstractFill{T,N}}, F::AbstractFill) where {T,N} = convert(AbstractArray{T,N}, F)
 
-copy(F::Fill) = Fill(copy(F.value), F.axes)
+copy(F::Fill) = Fill(F.value, F.axes)
 
 """ Throws an error if `arr` does not contain one and only one unique value. """
 function unique_value(arr::AbstractArray)

test/runtests.jl

@@ -137,8 +137,9 @@ import FillArrays: AbstractFill, RectDiagonal
         @test copy(x) ≡ x
         x = Fill([1.,2.],10)
         @test copy(x) == x
-        @test copy(x) !== x
+        @test copy(x) === x   # because isbits(x)
         @test copy(x) isa Fill
+        @test copy(Fill(:a, 4)) == fill(:a, 4)    # FillArrays#63
     end
 
     @testset "vec" begin
@@ -694,4 +695,4 @@ end
     @test Ones(5)' isa Adjoint # Vectors still need special dot product
     @test Fill([1+im 2; 3 4; 5 6], 2,3)' == Fill([1+im 2; 3 4; 5 6]', 3,2)
     @test transpose(Fill([1+im 2; 3 4; 5 6], 2,3)) == Fill(transpose([1+im 2; 3 4; 5 6]), 3,2)
-end
+end