Add bounds check to internal PseudoBlockArray constructor (#163)

* Update test_blockarrays.jl

* Update pseudo_blockarray.jl

* Update test_blockarrays.jl

Author: putianyi889

src/pseudo_blockarray.jl

@@ -43,8 +43,10 @@ julia> A = PseudoBlockArray(sprand(6, 0.5), [3,2,1])
 struct PseudoBlockArray{T, N, R<:AbstractArray{T,N}, BS<:NTuple{N,AbstractUnitRange{Int}}} <: AbstractBlockArray{T, N}
     blocks::R
     axes::BS
-    PseudoBlockArray{T,N,R,BS}(blocks::R, axes::BS) where {T,N,R,BS<:NTuple{N,AbstractUnitRange{Int}}} =
-        new{T,N,R,BS}(blocks, axes)
+    function PseudoBlockArray{T,N,R,BS}(blocks::R, axes::BS) where {T,N,R,BS<:NTuple{N,AbstractUnitRange{Int}}}
+        checkbounds(blocks,axes...)
+        return new{T,N,R,BS}(blocks, axes)
+    end
 end
 
 const PseudoBlockMatrix{T} = PseudoBlockArray{T, 2}

test/test_blockarrays.jl

@@ -85,15 +85,12 @@ end
             fill!(ret, 0)
             @test Array(ret)  == zeros(6)
 
-            ret = PseudoBlockArray{Float64,1,Vector{Float64}}(undef, 1:3)
-            fill!(ret, 0)
-            @test Array(ret)  == zeros(6)
-
             ret = PseudoBlockArray{Float64}(undef, 1:3, 1:3)
             fill!(ret, 0)
             @test Matrix(ret) == zeros(6,6)
 
             A = [1,2,3,4,5,6]
+            @test_throws BoundsError PseudoBlockArray(A,10:20)
             @test A == PseudoBlockArray(A, 1:3) == PseudoBlockArray{Int}(A, 1:3) == 
                 PseudoBlockArray(A, (blockedrange(1:3),)) == PseudoBlockArray{Int}(A, (blockedrange(1:3),)) ==
                 PseudoBlockArray{Float64}(A, 1:3)