Skip to content

Commit 24cdf34

Browse files
committed
add back isvalid(::String), fixes JuliaLang#17467
1 parent c7ca319 commit 24cdf34

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

base/strings/basic.jl

+1
Original file line numberDiff line numberDiff line change
@@ -289,6 +289,7 @@ byte_string_classify(s::String) = byte_string_classify(s.data)
289289
# 2: valid UTF-8
290290

291291
isvalid(::Type{String}, s::Union{Vector{UInt8},String}) = byte_string_classify(s) != 0
292+
isvalid(s::String) = isvalid(String, s)
292293

293294
## uppercase and lowercase transformations ##
294295
uppercase(s::AbstractString) = map(uppercase, s)

test/strings/basic.jl

+2-2
Original file line numberDiff line numberDiff line change
@@ -287,11 +287,11 @@ for (val, pass) in (
287287
(b"\udc00\u0100", false),
288288
(b"\udc00\ud800", false)
289289
)
290-
@test isvalid(String, val) == pass
290+
@test isvalid(String, val) == pass == isvalid(String(val))
291291
end
292292

293293
# Issue #11203
294-
@test isvalid(String, UInt8[]) == true
294+
@test isvalid(String, UInt8[]) == true == isvalid("")
295295

296296
# Check UTF-8 characters
297297
# Check ASCII range (true),

0 commit comments

Comments
 (0)