These collections can't change size so don't waist time testing for it

Author: Tokazama

src/indexing.jl

@@ -323,39 +323,27 @@ Reconstruct `A` given the values in `data`. New methods using `unsafe_reconstruc
 should only dispatch on `A`.
 """
 function unsafe_reconstruct(A::OneTo, data; kwargs...)
-    if can_change_size(A)
-        return typeof(A)(data)
+    if data isa Slice ||
+       !(known_length(A) === nothing || known_length(A) !== known_length(data))
+        return A
     else
-        if data isa Slice ||
-           !(known_length(A) === nothing || known_length(A) !== known_length(data))
-            return A
-        else
-            return OneTo(data)
-        end
+        return OneTo(data)
     end
 end
 function unsafe_reconstruct(A::UnitRange, data; kwargs...)
-    if can_change_size(A)
-        return typeof(A)(data)
+    if data isa Slice ||
+       !(known_length(A) === nothing || known_length(A) !== known_length(data))
+        return A
     else
-        if data isa Slice ||
-           !(known_length(A) === nothing || known_length(A) !== known_length(data))
-            return A
-        else
-            return UnitRange(data)
-        end
+        return UnitRange(data)
     end
 end
 function unsafe_reconstruct(A::OptionallyStaticUnitRange, data; kwargs...)
-    if can_change_size(A)
-        return typeof(A)(data)
+    if data isa Slice ||
+       !(known_length(A) === nothing || known_length(A) !== known_length(data))
+        return A
     else
-        if data isa Slice ||
-           !(known_length(A) === nothing || known_length(A) !== known_length(data))
-            return A
-        else
-            return OptionallyStaticUnitRange(data)
-        end
+        return OptionallyStaticUnitRange(data)
     end
 end
 function unsafe_reconstruct(A::AbstractUnitRange, data; kwargs...)
@@ -420,6 +408,15 @@ previously executed `to_index(old_axis, arg) -> index`. `to_axis` assumes that
         return to_axis(IndexStyle(axis), axis, inds)
     end
 end
+
+# don't need to check size b/c slice means it's the entire axis
+@inline function to_axis(axis, inds::Slice)
+    if can_change_size(axis)
+        return copy(axis)
+    else
+        return axis
+    end
+end
 @inline function to_axis(S::IndexStyle, axis, inds)
     return unsafe_reconstruct(axis, StaticInt(1):static_length(inds))
 end