diff --git a/src/readonly.jl b/src/readonly.jl
index 95734131..49becda4 100644
--- a/src/readonly.jl
+++ b/src/readonly.jl
@@ -10,7 +10,7 @@ struct ReadOnly{T,N,V<:AbstractArray{T,N}} <: AbstractArray{T,N}
 end
 # ReadOnly of ReadOnly is meaningless
 ReadOnly(x::ReadOnly) = x
-Base.getproperty(x::ReadOnly, s) = Base.getproperty(parent(x), s)
+Base.getproperty(x::ReadOnly, s::Symbol) = Base.getproperty(parent(x), s)
 @inline Base.parent(x::ReadOnly) = getfield(x, :parent)
 
 for i in [:length, :first, :last, :eachindex, :firstindex, :lastindex, :eltype]
diff --git a/src/sparsevector.jl b/src/sparsevector.jl
index e3a555d2..e2a24a9e 100644
--- a/src/sparsevector.jl
+++ b/src/sparsevector.jl
@@ -825,7 +825,7 @@ function findall(p::F, x::SparseVectorUnion{<:Any,Ti}) where {Ti,F<:Function}
 
     return I
 end
-findall(p::Base.Fix2{typeof(in)}, x::AbstractCompressedVector{<:Any,Ti}) where {Ti} =
+findall(p::Base.Fix2{typeof(in)}, x::SparseVectorUnion{<:Any,Ti}) where {Ti} =
     invoke(findall, Tuple{Base.Fix2{typeof(in)}, AbstractArray}, p, x)
 
 """