Skip to content

Commit 9f6dba6

Browse files
wip
1 parent 37d1e8e commit 9f6dba6

File tree

6 files changed

+9
-12
lines changed

6 files changed

+9
-12
lines changed

base/REPLCompletions.jl

+1-1
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ function complete_symbol(sym, ffunc)
4747
lookup_module = false
4848
t, found = get_type(ex, context_module)
4949
end
50-
found || return UTF8String[]
50+
found || return String[]
5151
# Ensure REPLCompletion do not crash when asked to complete a tuple, #15329
5252
!lookup_module && t <: Tuple && return String[]
5353
end

base/show.jl

+6-6
Original file line numberDiff line numberDiff line change
@@ -210,9 +210,9 @@ end
210210

211211
function lambdainfo_slotnames(l::LambdaInfo)
212212
slotnames = l.slotnames
213-
isa(slotnames, Array) || return UTF8String[]
214-
names = Dict{UTF8String,Int}()
215-
printnames = Vector{UTF8String}(length(slotnames))
213+
isa(slotnames, Array) || return String[]
214+
names = Dict{String,Int}()
215+
printnames = Vector{String}(length(slotnames))
216216
for i in eachindex(slotnames)
217217
name = string(slotnames[i])
218218
idx = get!(names, name, i)
@@ -533,9 +533,9 @@ function show_unquoted(io::IO, ex::Slot, ::Int, ::Int)
533533
end
534534
end
535535
slotnames = get(io, :LAMBDA_SLOTNAMES, false)
536-
if (isa(slotnames, Vector{UTF8String}) &&
537-
slotid <= length(slotnames::Vector{UTF8String}))
538-
print(io, (slotnames::Vector{UTF8String})[slotid])
536+
if (isa(slotnames, Vector{String}) &&
537+
slotid <= length(slotnames::Vector{String}))
538+
print(io, (slotnames::Vector{String})[slotid])
539539
else
540540
print(io, "_", slotid)
541541
end

base/strings/generic.jl

+1-1
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ string(s::AbstractString) = s
1212

1313
bytestring() = ""
1414
bytestring(s::Vector{UInt8}) =
15-
ccall(:jl_pchar_to_string, Ref{ByteString}, (Ptr{UInt8},Int), s, length(s))
15+
ccall(:jl_pchar_to_string, Ref{String}, (Ptr{UInt8},Int), s, length(s))
1616

1717
convert(::Type{Vector{UInt8}}, s::AbstractString) = bytestring(s).data
1818
convert(::Type{Array{UInt8}}, s::AbstractString) = bytestring(s).data

src/init.c

-1
Original file line numberDiff line numberDiff line change
@@ -838,7 +838,6 @@ void jl_get_builtin_hooks(void)
838838
#endif
839839

840840
jl_string_type = (jl_datatype_t*)core("String");
841-
jl_symbolnode_type = (jl_datatype_t*)core("SymbolNode");
842841
jl_weakref_type = (jl_datatype_t*)core("WeakRef");
843842
}
844843

test/replcompletions.jl

+1-1
Original file line numberDiff line numberDiff line change
@@ -295,7 +295,7 @@ s = "CompletionFoo.test4(CompletionFoo.test_y_array[1]()[1], CompletionFoo.test_
295295
c, r, res = test_complete(s)
296296
@test !res
297297
@test length(c) == 1
298-
@test c[1] == string(methods(CompletionFoo.test4, Tuple{ASCIIString, ASCIIString})[1])
298+
@test c[1] == string(methods(CompletionFoo.test4, Tuple{String, String})[1])
299299

300300
########## Test where the current inference logic fails ########
301301
# Fails due to inferrence fails to determine a concrete type for arg 1

test/strings/basic.jl

-2
Original file line numberDiff line numberDiff line change
@@ -93,8 +93,6 @@ end
9393
@test SubString("hello", 1, 0)[10:9] == ""
9494
@test SubString("hellø", 1, 6)[10:9] == ""
9595
@test SubString("hellø", 1, 0)[10:9] == ""
96-
@test ASCIIString("")[10:9] == ""
97-
@test UTF8String("")[10:9] == ""
9896
@test SubString("", 1, 6)[10:9] == ""
9997
@test SubString("", 1, 0)[10:9] == ""
10098

0 commit comments

Comments
 (0)