deprecate cpad

JuliaLang · Aug 9, 2017 · 1565093 · 1565093
1 parent 676faec
commit 1565093
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 2 deletions.
diff --git a/NEWS.md b/NEWS.md
@@ -290,6 +290,8 @@ Deprecated or removed
   * Calling `union` with no arguments is deprecated; construct an empty set with an appropriate
     element type using `Set{T}()` instead ([#23144]).
 
+  * `Base.cpad` is removed; use appropriate combination of `rpad` and `lpad` instead ([#23187]).
+
 Julia v0.6.0 Release Notes
 ==========================
 

diff --git a/base/deprecated.jl b/base/deprecated.jl
@@ -1632,6 +1632,9 @@ function SymTridiagonal(dv::AbstractVector{T}, ev::AbstractVector{S}) where {T,S
     SymTridiagonal(convert(Vector{R}, dv), convert(Vector{R}, ev))
 end
 
+# PR #23187
+@deprecate cpad(s, n::Integer, p=" ") rpad(lpad(s,div(n+strwidth(s),2),p),n,p) false
+
 # END 0.7 deprecations
 
 # BEGIN 1.0 deprecations

diff --git a/base/pkg/entry.jl b/base/pkg/entry.jl
@@ -572,7 +572,8 @@ end
 
 function warnbanner(msg...; label="[ WARNING ]", prefix="")
     cols = Base.displaysize(STDERR)[2]
-    warn(prefix="", Base.cpad(label,cols,"="))
+    str = rpad(lpad(label,div(cols+strwidth(label),2),"="),cols,"=")
+    warn(prefix="", str)
     println(STDERR)
     warn(prefix=prefix, msg...)
     println(STDERR)

diff --git a/base/strings/util.jl b/base/strings/util.jl
@@ -256,7 +256,6 @@ julia> rpad("March",20)
 ```
 """
 rpad(s, n::Integer, p=" ") = rpad(string(s),n,string(p))
-cpad(s, n::Integer, p=" ") = rpad(lpad(s,div(n+strwidth(s),2),p),n,p)
 
 # splitter can be a Char, Vector{Char}, AbstractString, Regex, ...
 # any splitter that provides search(s::AbstractString, splitter)