safer

Author: nsajko

stdlib/Random/src/misc.jl

@@ -185,7 +185,14 @@ ltm52(n::Int, mask::Int=nextpow(2, n)-1) = LessThan(n-1, Masked(mask, UInt52Raw(
 
 function shuffle(rng::AbstractRNG, tup::(Tuple{Vararg{T, N}} where {T})) where {N}
     @inline let  # `@inline` and `@inbounds` are here to help escape analysis
-        Ind = (N <= typemax(UInt8)) ? UInt8 : UInt16  # use a narrow integer type to save stack space and prevent heap allocation
+        # use a narrow integer type to save stack space and prevent heap allocation
+        Ind = if N ≤ typemax(UInt8)
+            UInt8
+        elseif N ≤ typemax(UInt16)
+            UInt16
+        else
+            UInt
+        end
         clo = @inbounds let mem = Memory{Ind}(undef, N)
             randperm!(rng, mem)
             let mem = mem, tup = tup