Update ktypeof function to support BlockArrays and FillArrays

[amontoison]

close #573
It also concerns #605

[github-actions]

Package name	latest	stable
CaNNOLeS.jl
DCISolver.jl
JSOSolvers.jl
LLSModels.jl
Percival.jl
RipQP.jl

[github-actions]

Package name	latest	stable
CaNNOLeS.jl
DCISolver.jl
JSOSolvers.jl
LLSModels.jl
Percival.jl
RipQP.jl

[codecov]

Codecov Report

Merging #606 (b96ff11) into main (56196d7) will decrease coverage by 0.01%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##             main     #606      +/-   ##
==========================================
- Coverage   98.05%   98.03%   -0.02%     
==========================================
  Files          36       36              
  Lines        6062     6064       +2     
==========================================
+ Hits         5944     5945       +1     
- Misses        118      119       +1     

Impacted Files	Coverage Δ
src/krylov_utils.jl	94.32% <83.33%> (-0.64%)	⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[github-actions]

Package name	latest	stable
CaNNOLeS.jl
DCISolver.jl
JSOSolvers.jl
LLSModels.jl
Percival.jl
RipQP.jl

[gdalle]

Hi there, and thanks for fixing #573! I just took a look at the code, and I was wondering why the function ktypeof doesn't exploit multiple dispatch? My first thought would have been something like this:

function ktypeof(v::S) where {S<:SubArray}
    return S.types[1]
end

function ktypeof(v::S) where {T,S<:DenseVector{T}}
    return Vector{T}
end

function ktypeof(v::S) where {S<:SparseVector}
    return S.types[3]
end

function ktypeof(v::S) where {S<:AbstractSparseVector}
    return S.types[2]
end

function ktypeof(v::S) where {S<:AbstractVector}
    return S
end

Would that work too?