Minor improvements of _tablesz implementation (#39126)

Probably this does not affect any Julia Base code, but in general the original code was not type stable and safe: Before the PR: ``` julia> Base._tablesz(true) # now it will be an error 16 julia> Base._tablesz(Int32(20)) # now it will be Int32(32) 0 ``` Co-authored-by: Kristoffer Carlsson <[email protected]>
JuliaLang · May 17, 2022 · 7bff5cd · 7bff5cd
1 parent 4605704
commit 7bff5cd
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/base/abstractdict.jl b/base/abstractdict.jl
@@ -573,7 +573,7 @@ function convert(::Type{T}, x::AbstractDict) where T<:AbstractDict
 end
 
 # hashing objects by identity
-_tablesz(x::Integer) = x < 16 ? 16 : one(x)<<((sizeof(x)<<3)-leading_zeros(x-1))
+_tablesz(x::T) where T <: Integer = x < 16 ? T(16) : one(T)<<((sizeof(T)<<3)-leading_zeros(x-one(T)))
 
 TP{K,V} = Union{Type{Tuple{K,V}},Type{Pair{K,V}}}