deprecate replace(s::String, pat, r, n) to replace(s, pat=>r, count=n)

base/deprecated.jl

@@ -1344,14 +1344,19 @@ end
 function replace(s::AbstractString, pat, f, n::Integer)
     if n <= 0
         depwarn(string("`replace(s, pat, r, count)` with `count <= 0` is deprecated, use ",
-                       "`replace(s, pat, r, typemax(Int))` or `replace(s, pat, r)` instead"),
+                       "`replace(s, pat=>r, count=typemax(Int))` or `replace(s, pat=>r)` instead"),
                 :replace)
-        replace(s, pat, f)
+        replace(s, pat=>f)
     else
-        replace_new(String(s), pat, f, n)
+        depwarn(string("`replace(s, pat, r, count)` is deprecated, use ",
+                       "`replace(s, pat=>r, count=count)`"),
+                :replace)
+        replace(String(s), pat=>f, count=n)
     end
 end
 
+@deprecate replace(s::AbstractString, pat, f) replace(s, pat=>f)
+
 # PR #22475
 @deprecate ntuple(f, ::Type{Val{N}}) where {N}  ntuple(f, Val(N))
 @deprecate fill_to_length(t, val, ::Type{Val{N}}) where {N} fill_to_length(t, val, Val(N)) false

base/libgit2/utils.jl

@@ -83,7 +83,7 @@ Standardise the path string `path` to use POSIX separators.
 """
 function posixpath end
 if Sys.iswindows()
-    posixpath(path) = replace(path,'\\','/')
+    posixpath(path) = replace(path,'\\' => '/')
 elseif Sys.isunix()
     posixpath(path) = path
 end

base/markdown/GitHub/table.jl

@@ -11,7 +11,7 @@ function parserow(stream::IO)
         row = split(line, r"(?<!\\)\|")
         length(row) == 1 && return
         isempty(row[1]) && shift!(row)
-        map!(x -> strip(replace(x, "\\|", "|")), row, row)
+        map!(x -> strip(replace(x, "\\|" => "|")), row, row)
         isempty(row[end]) && pop!(row)
         return row
     end
@@ -104,7 +104,7 @@ _dash(width, align) =
 
 function plain(io::IO, md::Table)
     cells = mapmap(md.rows) do each
-        replace(plaininline(each), "|", "\\|")
+        replace(plaininline(each), "|" => "\\|")
     end
     padcells!(cells, md.align, len = length, min = 3)
     for i = axes(cells,1)

base/markdown/render/html.jl

@@ -31,7 +31,7 @@ for ch in "'`!\$%()=+{}[]"
 end
 
 function htmlesc(io::IO, s::AbstractString)
-    # s1 = replace(s, r"&(?!(\w+|\#\d+);)", "&")
+    # s1 = replace(s, r"&(?!(\w+|\#\d+);)" => "&")
     for ch in s
         print(io, get(_htmlescape_chars, ch, ch))
     end

base/markdown/render/rst.jl

@@ -124,7 +124,7 @@ end
 
 rstinline(io::IO, f::Footnote) = print(io, "[", f.id, "]_")
 
-rstescape(s) = replace(s, "\\", "\\\\")
+rstescape(s) = replace(s, "\\" => "\\\\")
 
 rstinline(io::IO, s::AbstractString) = print(io, rstescape(s))
 

base/markdown/render/terminal/formatting.jl

@@ -51,7 +51,7 @@ end
 # Wrapping
 
 function ansi_length(s)
-    replace(s, r"\e\[[0-9]+m", "") |> length
+    replace(s, r"\e\[[0-9]+m" => "") |> length
 end
 
 words(s) = split(s, " ")

base/markdown/render/terminal/render.jl

@@ -114,7 +114,7 @@ function terminline(io::IO, content::Vector)
 end
 
 function terminline(io::IO, md::AbstractString)
-    print(io, replace(md, r"[\s\t\n]+", " "))
+    print(io, replace(md, r"[\s\t\n]+" => " "))
 end
 
 function terminline(io::IO, md::Bold)

base/methodshow.jl

@@ -203,7 +203,7 @@ function url(m::Method)
     file = string(m.file)
     line = m.line
     line <= 0 || ismatch(r"In\[[0-9]+\]", file) && return ""
-    Sys.iswindows() && (file = replace(file, '\\', '/'))
+    Sys.iswindows() && (file = replace(file, '\\' => '/'))
     if inbase(M)
         if isempty(Base.GIT_VERSION_INFO.commit)
             # this url will only work if we're on a tagged release

base/pkg/reqs.jl

@@ -19,7 +19,7 @@ struct Requirement <: Line
     system::Vector{AbstractString}
 
     function Requirement(content::AbstractString)
-        fields = split(replace(content, r"#.*$", ""))
+        fields = split(replace(content, r"#.*$" => ""))
         system = AbstractString[]
         while !isempty(fields) && fields[1][1] == '@'
             push!(system,shift!(fields)[2:end])

base/process.jl

@@ -107,7 +107,7 @@ function show(io::IO, cmd::Cmd)
             with_output_color(:underline, io) do io
                 print_shell_word(io, arg, shell_special)
             end
-        end, '`', "\\`")
+        end, '`' => "\\`")
     end, ' '))
     print(io, '`')
     print_env && (print(io, ","); show(io, cmd.env))

base/repl/LineEdit.jl

@@ -1883,12 +1883,12 @@ function bracketed_paste(s; tabwidth=options(s).tabwidth)
     ps = state(s, mode(s))
     str = readuntil(ps.terminal, "\e[201~")
     input = str[1:prevind(str, end-5)]
-    input = replace(input, '\r', '\n')
+    input = replace(input, '\r' => '\n')
     if position(buffer(s)) == 0
         indent = Base.indentation(input; tabwidth=tabwidth)[1]
         input = Base.unindent(input, indent; tabwidth=tabwidth)
     end
-    return replace(input, '\t', " "^tabwidth)
+    return replace(input, '\t' => " "^tabwidth)
 end
 
 function tab_should_complete(s)

base/repl/REPL.jl

@@ -450,7 +450,7 @@ function add_history(hist::REPLHistoryProvider, s)
     entry = """
     # time: $(Libc.strftime("%Y-%m-%d %H:%M:%S %Z", time()))
     # mode: $mode
-    $(replace(str, r"^"ms, "\t"))
+    $(replace(str, r"^"ms => "\t"))
     """
     # TODO: write-lock history file
     seekend(hist.history_file)
@@ -946,7 +946,7 @@ function setup_interface(
                         tail = lstrip(tail)
                     end
                     if isprompt_paste # remove indentation spaces corresponding to the prompt
-                        tail = replace(tail, r"^ {7}"m, "") # 7: jl_prompt_len
+                        tail = replace(tail, r"^ {7}"m => "") # 7: jl_prompt_len
                     end
                     LineEdit.replace_line(s, tail, true)
                     LineEdit.refresh_line(s)
@@ -956,7 +956,7 @@ function setup_interface(
                 line = strip(input[oldpos:prevind(input, pos)])
                 if !isempty(line)
                     if isprompt_paste # remove indentation spaces corresponding to the prompt
-                        line = replace(line, r"^ {7}"m, "") # 7: jl_prompt_len
+                        line = replace(line, r"^ {7}"m => "") # 7: jl_prompt_len
                     end
                     # put the line on the screen and history
                     LineEdit.replace_line(s, line)

base/repl/REPLCompletions.jl

@@ -192,7 +192,7 @@ function complete_path(path::AbstractString, pos; use_envpath=false)
         end
     end
 
-    matchList = String[replace(s, r"\s", "\\ ") for s in matches]
+    matchList = String[replace(s, r"\s" => "\\ ") for s in matches]
     startpos = pos - endof(prefix) + 1 - length(matchall(r" ", prefix))
     # The pos - endof(prefix) + 1 is correct due to `endof(prefix)-endof(prefix)==0`,
     # hence we need to add one to get the first index. This is also correct when considering
@@ -503,15 +503,15 @@ function completions(string, pos)
         startpos = nextind(partial, reverseind(partial, m.offset))
         r = startpos:pos
 
-        expanded = complete_expanduser(replace(string[r], r"\\ ", " "), r)
+        expanded = complete_expanduser(replace(string[r], r"\\ " => " "), r)
         expanded[3] && return expanded  # If user expansion available, return it
 
-        paths, r, success = complete_path(replace(string[r], r"\\ ", " "), pos)
+        paths, r, success = complete_path(replace(string[r], r"\\ " => " "), pos)
 
         if inc_tag == :string &&
            length(paths) == 1 &&  # Only close if there's a single choice,
            !isdir(expanduser(replace(string[startpos:prevind(string, start(r))] * paths[1],
-                                     r"\\ ", " "))) &&  # except if it's a directory
+                                     r"\\ " => " "))) &&  # except if it's a directory
            (length(string) <= pos ||
             string[nextind(string,pos)] != '"')  # or there's already a " at the cursor.
             paths[1] *= "\""

base/repl/latex_symbols.jl

@@ -53,7 +53,7 @@ const latex_strings = Set(values(Base.REPLCompletions.latex_symbols))
 open(fname) do f
     for L in eachline(f)
         x = map(s -> rstrip(s, [' ','\t','\n']),
-                split(replace(L, r"[{}\"]+", "\t"), "\t"))
+                split(replace(L, r"[{}\"]+" => "\t"), "\t"))
         c = Char(parse(Int, x[2], 16))
         if (Base.is_id_char(c) || Base.isoperator(Symbol(c))) &&
            string(c) ∉ latex_strings && !isascii(c)

base/shell.jl

@@ -222,11 +222,11 @@ function print_shell_escaped_posixly(io::IO, args::AbstractString...)
             return true
         end
         if all(isword, arg)
-            have_single && (arg = replace(arg, '\'', "\\'"))
-            have_double && (arg = replace(arg, '"', "\\\""))
+            have_single && (arg = replace(arg, '\'' => "\\'"))
+            have_double && (arg = replace(arg, '"' => "\\\""))
             print(io, arg)
         else
-            print(io, '\'', replace(arg, '\'', "'\\''"), '\'')
+            print(io, '\'', replace(arg, '\'' => "'\\''"), '\'')
         end
         first = false
     end

base/strings/util.jl

@@ -373,8 +373,8 @@ _replace(io, repl, str, r, pattern) = print(io, repl)
 _replace(io, repl::Function, str, r, pattern) =
     print(io, repl(SubString(str, first(r), last(r))))
 
-# TODO: rename to `replace` when `replace` is removed from deprecated.jl
-function replace_new(str::String, pattern, repl, count::Integer)
+function replace(str::String, pat_repl::Pair; count::Integer=typemax(Int))
+    pattern, repl = pat_repl
     count == 0 && return str
     count < 0 && throw(DomainError(count, "`count` must be non-negative."))
     n = 1
@@ -407,11 +407,11 @@ function replace_new(str::String, pattern, repl, count::Integer)
 end
 
 """
-    replace(s::AbstractString, pat, r, [count::Integer])
+    replace(s::AbstractString, pat=>r; [count::Integer])
 
 Search for the given pattern `pat` in `s`, and replace each occurrence with `r`.
 If `count` is provided, replace at most `count` occurrences.
-As with [`search`](@ref), the second argument may be a
+As with [`search`](@ref), `pat` may be a
 single character, a vector or a set of characters, a string, or a regular expression. If `r`
 is a function, each occurrence is replaced with `r(s)` where `s` is the matched substring.
 If `pat` is a regular expression and `r` is a `SubstitutionString`, then capture group
@@ -420,20 +420,18 @@ To remove instances of `pat` from `string`, set `r` to the empty `String` (`""`)
 
 # Examples
 ```jldoctest
-julia> replace("Python is a programming language.", "Python", "Julia")
+julia> replace("Python is a programming language.", "Python" => "Julia")
 "Julia is a programming language."
 
-julia> replace("The quick foxes run quickly.", "quick", "slow", 1)
+julia> replace("The quick foxes run quickly.", "quick" => "slow", count=1)
 "The slow foxes run quickly."
 
-julia> replace("The quick foxes run quickly.", "quick", "", 1)
+julia> replace("The quick foxes run quickly.", "quick" => "", count=1)
 "The  foxes run quickly."
 ```
 """
-replace(s::AbstractString, pat, f) = replace_new(String(s), pat, f, typemax(Int))
-# TODO: change this to the following when `replace` is removed from deprecated.jl:
-# replace(s::AbstractString, pat, f, count::Integer=typemax(Int)) =
-#     replace(String(s), pat, f, count)
+replace(s::AbstractString, pat_f::Pair; count=typemax(Int)) =
+    replace(String(s), pat_f, count=count)
 
 # TODO: allow transform as the first argument to replace?
 

doc/src/stdlib/strings.md

@@ -38,7 +38,7 @@ Base.searchindex
 Base.rsearchindex
 Base.contains(::AbstractString, ::AbstractString)
 Base.reverse(::Union{String,SubString{String}})
-Base.replace(s::AbstractString, pat, f)
+Base.replace(s::AbstractString, ::Pair)
 Base.split
 Base.rsplit
 Base.strip

test/libgit2.jl

@@ -2550,7 +2550,7 @@ mktempdir() do dir
             try
                 # In some environments, namely Macs, the hostname "macbook.local" is bound
                 # to the external address while "macbook" is bound to the loopback address.
-                unshift!(hostnames, replace(gethostname(), r"\..*$", ""))
+                unshift!(hostnames, replace(gethostname(), r"\..*$" => ""))
             end
 
             loopback = ip"127.0.0.1"

test/logging.jl

@@ -241,7 +241,7 @@ end
         s = String(take!(io))
         # Remove the small amount of color, as `Base.print_with_color` can't be
         # simply controlled.
-        s = replace(s, r"^\e\[1m\e\[..m(.*)\e\[39m\e\[22m"m, s"\1")
+        s = replace(s, r"^\e\[1m\e\[..m(.*)\e\[39m\e\[22m"m => s"\1")
         # println(s)
         s
     end

test/operators.jl

@@ -110,8 +110,8 @@ Base.promote_rule(::Type{T19714}, ::Type{Int}) = T19714
 end
 @testset "function negation" begin
     str = randstring(20)
-    @test filter(!Base.Unicode.isupper, str) == replace(str, r"[A-Z]", "")
-    @test filter(!Base.Unicode.islower, str) == replace(str, r"[a-z]", "")
+    @test filter(!Base.Unicode.isupper, str) == replace(str, r"[A-Z]" => "")
+    @test filter(!Base.Unicode.islower, str) == replace(str, r"[a-z]" => "")
 end
 
 # issue #19891

test/perf/shootout/regex_dna.jl

@@ -36,7 +36,7 @@ function regex_dna(infile="regexdna-input.txt")
     seq = read(infile, String)
     l1 = length(seq)
 
-    seq = replace(seq, r">.*\n|\n", "")
+    seq = replace(seq, r">.*\n|\n" => "")
     l2 = length(seq)
 
     for v in variants
@@ -48,12 +48,11 @@ function regex_dna(infile="regexdna-input.txt")
     end
 
     for (u, v) in subs
-        seq = replace(seq, u, v)
+        seq = replace(seq, u => v)
     end
 
 #    println()
 #    println(l1)
 #    println(l2)
 #    println(length(seq))
 end
-

test/regex.jl

@@ -45,7 +45,7 @@ let m = match(r"(?<a>.)(.)(?<b>.)", "xyz")
 end
 
 # Backcapture reference in substitution string
-@test replace("abcde", r"(..)(?P<byname>d)", s"\g<byname>xy\\\1") == "adxy\\bce"
+@test replace("abcde", r"(..)(?P<byname>d)" => s"\g<byname>xy\\\1") == "adxy\\bce"
 @test_throws ErrorException replace("a", r"(?P<x>)", s"\g<y>")
 
 # Proper unicode handling

test/replcompletions.jl

@@ -704,7 +704,7 @@ end
 let path = tempdir(),
     space_folder = randstring() * " α",
     dir = joinpath(path, space_folder),
-    dir_space = replace(space_folder, " ", "\\ ")
+    dir_space = replace(space_folder, " " => "\\ ")
 
     mkdir(dir)
     cd(path) do

test/show.jl

@@ -302,7 +302,7 @@ end
 
 # issue #7921
 @test replace(sprint(show, Expr(:function, :(==(a, b)), Expr(:block,:(return a == b)))),
-              r"\s+", " ") == ":(function ==(a, b) return a == b end)"
+              r"\s+" => " ") == ":(function ==(a, b) return a == b end)"
 
 # unicode operator printing
 @test sprint(show, :(1 ⊕ (2 ⊗ 3))) == ":(1 ⊕ 2 ⊗ 3)"
@@ -581,15 +581,15 @@ let ex,
     @test string(l2)  == "#= myfile:42 =#"
     @test string(l1)  == string(l2n)
     ex = Expr(:block, l1, :x, l2, :y, l2n, :z)
-    @test replace(string(ex)," ","") == replace("""
+    @test replace(string(ex)," " => "") == replace("""
     begin
         #= line 42 =#
         x
         #= myfile:42 =#
         y
         #= line 42 =#
         z
-    end""", " ", "")
+    end""", " " => "")
 end
 # Test the printing of whatever form of line number representation
 # that is used in the arguments to a macro looks the same as for