remove more deprecations (#28457)

Author: JeffBezanson

base/abstractdict.jl

@@ -554,7 +554,7 @@ function IdDict(kv)
     try
         dict_with_eltype((K, V) -> IdDict{K, V}, kv, eltype(kv))
     catch e
-        if !applicable(start, kv) || !all(x->isa(x,Union{Tuple,Pair}),kv)
+        if !applicable(iterate, kv) || !all(x->isa(x,Union{Tuple,Pair}),kv)
             throw(ArgumentError(
                 "IdDict(kv): kv needs to be an iterator of tuples or pairs"))
         else

base/deprecated.jl

@@ -837,72 +837,6 @@ next element and the new iteration state should be returned.
 """
 function iterate end
 
-# Compatibility with old iteration protocol
-function iterate(x, state)
-    @_inline_meta
-    done(x, state) && return nothing
-    return next(x, state)
-end
-const old_iterate_line_prev = (@__LINE__)
-function iterate(x)
-    r = iterate(x, start(x))
-    depwarn("The start/next/done iteration protocol is deprecated. Implement `iterate(::$(typeof(x)))`.", :start)
-    r
-end
-
-struct LegacyIterationCompat{I,T,S}
-    done::Bool
-    nextval::T
-    state::S
-    LegacyIterationCompat{I,T,S}() where {I,T,S} = new{I,T,S}(true)
-    LegacyIterationCompat{I,T,S}(nextval::T, state::S) where {I,T,S} = new{I,T,S}(false, nextval, state)
-end
-
-function has_non_default_iterate(T)
-    world = ccall(:jl_get_world_counter, UInt, ())
-    mt = Base._methods(iterate, Tuple{T}, -1, world)
-    # Check if this is the above method
-    if (mt[1][3].file == @__FILE_SYMBOL__) && (mt[1][3].line == old_iterate_line_prev + 2)
-        return false
-    end
-    return true
-end
-
-const compat_start_line_prev = (@__LINE__)
-function start(itr::T) where {T}
-    has_non_default_iterate(T) || throw(MethodError(iterate, (itr,)))
-    depwarn("The start/next/done iteration protocol is deprecated. Use `iterate` instead.", :start)
-    y = iterate(itr)
-    y === nothing && return LegacyIterationCompat{T, Union{}, Union{}}()
-    val, state = y
-    LegacyIterationCompat{T, typeof(val), typeof(state)}(val, state)
-end
-
-function next(itr::I, state::LegacyIterationCompat{I,T,S}) where {I,T,S}
-    val, state = state.nextval, state.state
-    y = iterate(itr, state)
-    if y === nothing
-        return (val, LegacyIterationCompat{I,T,S}())
-    end
-    nextval, state = y
-    val, LegacyIterationCompat{I, typeof(nextval), typeof(state)}(nextval, state)
-end
-
-done(itr::I, state::LegacyIterationCompat{I,T,S}) where {I,T,S} = (@_inline_meta; state.done)
-# This is necessary to support the above compatibility layer,
-# eventually, this should just check for applicability of `iterate`
 function isiterable(T)::Bool
-    if !has_non_default_iterate(T)
-        world = ccall(:jl_get_world_counter, UInt, ())
-        mt = Base._methods(start, Tuple{T}, -1, world)
-        # Check if this is the fallback start method
-        if (mt[1][3].file == @__FILE_SYMBOL__) && (mt[1][3].line == compat_start_line_prev + 2)
-            return false
-        end
-    end
-    return true
+    return hasmethod(iterate, Tuple{T})
 end
-
-# This is required to avoid massive performance problems
-# due to the start(s::AbstractString) deprecation.
-iterate(s::AbstractString) = iterate(s, firstindex(s))

base/essentials.jl

@@ -496,8 +496,6 @@ end
 
 @propagate_inbounds iterate(i::Rest, st=i.st) = iterate(i.itr, st)
 isdone(i::Rest, st...) = isdone(i.itr, st...)
-@propagate_inbounds iterate(i::Rest{I,S}, st::S=i.st) where {I,S<:Base.LegacyIterationCompat{I}} =
-    done(i.itr, st) ? nothing : next(i.itr, st)
 
 eltype(::Type{<:Rest{I}}) where {I} = eltype(I)
 IteratorEltype(::Type{<:Rest{I}}) where {I} = IteratorEltype(I)

base/iterators.jl

@@ -564,6 +564,7 @@ function show(io::IO, tn::Core.TypeName)
 end
 
 show(io::IO, ::Nothing) = print(io, "nothing")
+print(io::IO, ::Nothing) = throw(ArgumentError("`nothing` should not be printed; use `show`, `repr`, or custom output instead."))
 show(io::IO, b::Bool) = print(io, b ? "true" : "false")
 show(io::IO, n::Signed) = (write(io, string(n)); nothing)
 show(io::IO, n::Unsigned) = print(io, "0x", string(n, pad = sizeof(n)<<1, base = 16))

base/show.jl

@@ -6,7 +6,7 @@ import ..@__MODULE__, ..parentmodule
 const Base = parentmodule(@__MODULE__)
 using .Base.Order
 using .Base: copymutable, LinearIndices, length, (:),
-    eachindex, axes, first, last, similar, start, next, done, zip, OrdinalRange,
+    eachindex, axes, first, last, similar, zip, OrdinalRange,
     AbstractVector, @inbounds, AbstractRange, @eval, @inline, Vector, @noinline,
     AbstractMatrix, AbstractUnitRange, isless, identity, eltype, >, <, <=, >=, |, +, -, *, !,
     extrema, sub_with_overflow, add_with_overflow, oneunit, div, getindex, setindex!,

base/sort.jl

@@ -462,9 +462,6 @@ function __init__()
             ENV["OPENBLAS_NUM_THREADS"] = "8"
         elseif haskey(ENV, "JULIA_CPU_THREADS") # or exactly as specified
             ENV["OPENBLAS_NUM_THREADS"] = cpu_threads
-        elseif haskey(ENV, "JULIA_CPU_CORES") # TODO: delete in 1.0 (deprecation)
-            Core.print("JULIA_CPU_CORES is deprecated, use JULIA_CPU_THREADS instead.\n")
-            ENV["OPENBLAS_NUM_THREADS"] = cpu_threads
         end # otherwise, trust that openblas will pick CPU_THREADS anyways, without any intervention
     end
     # for the few uses of Libc.rand in Base:
@@ -545,85 +542,6 @@ let
 
     print_time("Stdlibs total", Base.tot_time_stdlib[])
 end
-
-@eval Base begin
-    @deprecate_binding Test root_module(Base, :Test) true ", run `using Test` instead"
-    @deprecate_binding Distributed root_module(Base, :Distributed) true ", run `using Distributed` instead"
-    @deprecate_binding Random root_module(Base, :Random) true ", run `using Random` instead"
-    @deprecate_binding Serializer root_module(Base, :Serialization) true ", run `using Serialization` instead"
-
-    # PR #25571
-    @deprecate_binding LinAlg root_module(Base, :LinearAlgebra) true ", run `using LinearAlgebra` instead"
-    @deprecate_binding(I, root_module(Base, :LinearAlgebra).I, true,
-        ", run `using LinearAlgebra` to load linear algebra functionality.")
-
-    @deprecate_binding Pkg root_module(Base, :Pkg) true ", run `using Pkg` instead"
-    @deprecate_binding LibGit2 root_module(Base, :LibGit2) true ", run `import LibGit2` instead"
-
-    @eval @deprecate_stdlib $(Symbol("@spawn")) Distributed true
-    @eval @deprecate_stdlib $(Symbol("@spawnat")) Distributed true
-    @eval @deprecate_stdlib $(Symbol("@fetch")) Distributed true
-    @eval @deprecate_stdlib $(Symbol("@fetchfrom")) Distributed true
-    @eval @deprecate_stdlib $(Symbol("@everywhere")) Distributed true
-    @eval @deprecate_stdlib $(Symbol("@parallel")) Distributed true
-
-    @deprecate_stdlib addprocs Distributed true
-    @deprecate_stdlib CachingPool Distributed true
-    @deprecate_stdlib clear! Distributed true
-    @deprecate_stdlib ClusterManager Distributed true
-    @deprecate_stdlib default_worker_pool Distributed true
-    @deprecate_stdlib init_worker Distributed true
-    @deprecate_stdlib interrupt Distributed true
-    @deprecate_stdlib launch Distributed true
-    @deprecate_stdlib manage Distributed true
-    @deprecate_stdlib myid Distributed true
-    @deprecate_stdlib nprocs Distributed true
-    @deprecate_stdlib nworkers Distributed true
-    @deprecate_stdlib pmap Distributed true
-    @deprecate_stdlib procs Distributed true
-    @deprecate_stdlib remote Distributed true
-    @deprecate_stdlib remotecall Distributed true
-    @deprecate_stdlib remotecall_fetch Distributed true
-    @deprecate_stdlib remotecall_wait Distributed true
-    @deprecate_stdlib remote_do Distributed true
-    @deprecate_stdlib rmprocs Distributed true
-    @deprecate_stdlib workers Distributed true
-    @deprecate_stdlib WorkerPool Distributed true
-    @deprecate_stdlib RemoteChannel Distributed true
-    @deprecate_stdlib Future Distributed true
-    @deprecate_stdlib WorkerConfig Distributed true
-    @deprecate_stdlib RemoteException Distributed true
-    @deprecate_stdlib ProcessExitedException Distributed true
-
-    # PR #24874
-    @deprecate_stdlib rand! Random true
-    @deprecate_stdlib srand Random true
-    @deprecate_stdlib AbstractRNG Random true
-    @deprecate_stdlib randcycle  Random true
-    @deprecate_stdlib randcycle!  Random true
-    @deprecate_stdlib randperm  Random true
-    @deprecate_stdlib randperm! Random true
-    @deprecate_stdlib shuffle  Random true
-    @deprecate_stdlib shuffle! Random true
-    @deprecate_stdlib randsubseq Random true
-    @deprecate_stdlib randsubseq! Random true
-    @deprecate_stdlib randstring Random true
-    @deprecate_stdlib MersenneTwister  Random true
-    @deprecate_stdlib RandomDevice  Random true
-    @deprecate_stdlib randn! Random true
-    @deprecate_stdlib randexp Random true
-    @deprecate_stdlib randexp! Random true
-    @deprecate_stdlib bitrand Random true
-    @deprecate_stdlib randjump Random true
-    @deprecate_stdlib GLOBAL_RNG Random false
-
-    # PR #25571: LinearAlgebra to stdlib
-
-    # PR #25021
-    @deprecate_stdlib normalize_string Unicode true
-    @deprecate_stdlib graphemes Unicode true
-    @deprecate_stdlib is_assigned_char Unicode true
-end
 end
 
 # Clear global state

base/sysimg.jl

@@ -92,9 +92,6 @@ function __init__()
     env_threads = nothing
     if haskey(ENV, "JULIA_CPU_THREADS")
         env_threads = ENV["JULIA_CPU_THREADS"]
-    elseif haskey(ENV, "JULIA_CPU_CORES") # TODO: delete in 1.0 (deprecation)
-        Core.print("JULIA_CPU_CORES is deprecated, use JULIA_CPU_THREADS instead.\n")
-        env_threads = ENV["JULIA_CPU_CORES"]
     end
     global CPU_THREADS = if env_threads !== nothing
         env_threads = tryparse(Int, env_threads)
@@ -106,7 +103,6 @@ function __init__()
     else
         Int(ccall(:jl_cpu_threads, Int32, ()))
     end
-    global CPU_CORES = CPU_THREADS # TODO: delete in 1.0 (deprecation)
     global SC_CLK_TCK = ccall(:jl_SC_CLK_TCK, Clong, ())
     global CPU_NAME = ccall(:jl_get_cpu_name, Ref{String}, ())
     global JIT = ccall(:jl_get_JIT, Ref{String}, ())

base/sysinfo.jl

@@ -244,7 +244,6 @@ Base.process_exited
 Base.kill(::Base.Process, ::Integer)
 Base.Sys.set_process_title
 Base.Sys.get_process_title
-Base.readandwrite
 Base.ignorestatus
 Base.detach
 Base.Cmd

doc/src/base/base.md

@@ -92,14 +92,6 @@ include("pmap.jl")
 include("managers.jl")    # LocalManager and SSHManager
 include("precompile.jl")
 
-# Deprecations
-
-@eval @deprecate $(Symbol("@parallel")) $(Symbol("@distributed"))
-
-# PR 26783
-@deprecate pmap(p::AbstractWorkerPool, f, c; kwargs...) pmap(f, p, c; kwargs...)
-@deprecate pmap(p::AbstractWorkerPool, f, c1, c...; kwargs...) pmap(f, p, c1, c...; kwargs...)
-
 function __init__()
     init_parallel()
 end