Revert "make last work on any reversible collection (#42991)"

This reverts commit 1f484c3 since it
seems to cause excess breakage for the benefits it brings.

NEWS.md

@@ -25,8 +25,6 @@ Language changes
 * `@time` and `@timev` now take an optional description to allow annotating the source of time reports.
   i.e. `@time "Evaluating foo" foo()` ([#42431])
 * New `@showtime` macro to show both the line being evaluated and the `@time` report ([#42431])
-* `last(collection)` will now work on any collection that supports `Iterators.reverse` and `first`, rather than being
-  restricted to indexable collections.
 * Unbalanced Unicode bidirectional formatting directives are now disallowed within strings and comments,
   to mitigate the ["trojan source"](https://www.trojansource.codes) vulnerability ([#42918]).
 

base/abstractarray.jl

@@ -462,15 +462,11 @@ end
     last(coll)
 
 Get the last element of an ordered collection, if it can be computed in O(1) time. This is
-accomplished by calling [`Iterators.reverse`](@ref) and then [`first`](@ref) on that reversed iterator.
-Return the end point of an [`AbstractRange`](@ref) even if it is empty.
+accomplished by calling [`lastindex`](@ref) to get the last index. Return the end
+point of an [`AbstractRange`](@ref) even if it is empty.
 
 See also [`first`](@ref), [`endswith`](@ref).
 
-!!! compat "Julia 1.8"
-    For versions of julia older than 1.8, `last(x)` will only work on collections that support indexing and
-    [`lastindex`](@ref).
-
 # Examples
 ```jldoctest
 julia> last(1:2:10)
@@ -480,8 +476,7 @@ julia> last([1; 2; 3; 4])
 4
 ```
 """
-last(a) = first(Iterators.reverse(a))
-last(a::AbstractVector) = a[end]
+last(a) = a[end]
 
 """
     last(itr, n::Integer)

base/iterators.jl

@@ -103,6 +103,7 @@ size(r::Reverse) = size(r.itr)
 IteratorSize(::Type{Reverse{T}}) where {T} = IteratorSize(T)
 IteratorEltype(::Type{Reverse{T}}) where {T} = IteratorEltype(T)
 last(r::Reverse) = first(r.itr) # the first shall be last
+first(r::Reverse) = last(r.itr) # and the last shall be first
 
 # reverse-order array iterators: assumes more-specialized Reverse for eachindex
 @propagate_inbounds function iterate(A::Reverse{<:AbstractArray}, state=(reverse(eachindex(A.itr)),))

test/iterators.jl

@@ -887,8 +887,3 @@ end
     @test Iterators.peel(x^2 for x in 2:4)[1] == 4
     @test Iterators.peel(x^2 for x in 2:4)[2] |> collect == [9, 16]
 end
-
-@testset "last for iterators" begin
-    @test last(Iterators.map(identity, 1:3)) == 3
-    @test last(Iterators.filter(iseven, (Iterators.map(identity, 1:3)))) == 2
-end