Skip to content

Commit d2eddaf

Browse files
author
Andy Ferris
committed
Fixed ASCII vs UTF8 errors with Julia v0.5 String
1 parent 3218c83 commit d2eddaf

File tree

1 file changed

+6
-10
lines changed

1 file changed

+6
-10
lines changed

src/plain.jl

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1284,11 +1284,7 @@ function read{S<:String}(obj::DatasetOrAttribute, ::Type{S})
12841284
buf = Ptr{UInt8}[C_NULL]
12851285
memtype_id = h5t_copy(H5T_C_S1)
12861286
h5t_set_size(memtype_id, H5T_VARIABLE)
1287-
if isleaftype(S)
1288-
h5t_set_cset(memtype_id, cset(S))
1289-
else
1290-
h5t_set_cset(memtype_id, h5t_get_cset(datatype(obj)))
1291-
end
1287+
h5t_set_cset(memtype_id, h5t_get_cset(datatype(obj)))
12921288
readarray(obj, memtype_id, buf)
12931289
ret = @compat unsafe_string(buf[1])
12941290
else
@@ -2144,7 +2140,7 @@ function h5a_get_name(attr_id::Hid)
21442140
len = h5a_get_name(attr_id, 0, C_NULL) # order of args differs from {f,i}_get_name
21452141
buf = Array(UInt8, len+1)
21462142
h5a_get_name(attr_id, len+1, buf)
2147-
@compat unsafe_string(buf[1:len])
2143+
@compat String(buf[1:len])
21482144
end
21492145
function h5f_get_name(loc_id::Hid)
21502146
len = h5f_get_name(loc_id, C_NULL, 0)
@@ -2393,15 +2389,15 @@ function __init__()
23932389
# h5e_set_auto(H5E_DEFAULT, C_NULL, C_NULL)
23942390

23952391
ASCII_LINK_PROPERTIES[] = p_create(H5P_LINK_CREATE)
2396-
h5p_set_char_encoding(ASCII_LINK_PROPERTIES[].id, cset(Compat.ASCIIString))
2392+
h5p_set_char_encoding(ASCII_LINK_PROPERTIES[].id, H5T_CSET_ASCII)
23972393
h5p_set_create_intermediate_group(ASCII_LINK_PROPERTIES[].id, 1)
23982394
UTF8_LINK_PROPERTIES[] = p_create(H5P_LINK_CREATE)
2399-
h5p_set_char_encoding(UTF8_LINK_PROPERTIES[].id, cset(Compat.UTF8String))
2395+
h5p_set_char_encoding(UTF8_LINK_PROPERTIES[].id, H5T_CSET_UTF8)
24002396
h5p_set_create_intermediate_group(UTF8_LINK_PROPERTIES[].id, 1)
24012397
ASCII_ATTRIBUTE_PROPERTIES[] = p_create(H5P_ATTRIBUTE_CREATE)
2402-
h5p_set_char_encoding(ASCII_ATTRIBUTE_PROPERTIES[].id, cset(Compat.ASCIIString))
2398+
h5p_set_char_encoding(ASCII_ATTRIBUTE_PROPERTIES[].id, H5T_CSET_ASCII)
24032399
UTF8_ATTRIBUTE_PROPERTIES[] = p_create(H5P_ATTRIBUTE_CREATE)
2404-
h5p_set_char_encoding(UTF8_ATTRIBUTE_PROPERTIES[].id, cset(Compat.UTF8String))
2400+
h5p_set_char_encoding(UTF8_ATTRIBUTE_PROPERTIES[].id, H5T_CSET_UTF8)
24052401

24062402
rehash!(hdf5_type_map, length(hdf5_type_map.keys))
24072403
rehash!(hdf5_prop_get_set, length(hdf5_prop_get_set.keys))

0 commit comments

Comments
 (0)