Skip to content

Commit 42543c1

Browse files
authored
Merge pull request #179 from queryverse/fix-mmap-julia-1.11
Fix mmap finalizer call on Julia 1.11
2 parents 4ec842a + 090f4dd commit 42543c1

File tree

1 file changed

+10
-2
lines changed

1 file changed

+10
-2
lines changed

src/csv.jl

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,11 @@ function csvread(file::IOStream, delim=','; kwargs...)
8787
try
8888
_csvread(VectorBackedUTF8String(mmap_data), delim; kwargs...)
8989
finally
90-
finalize(mmap_data)
90+
@static if VERSION v"1.11"
91+
finalize(mmap_data.ref.mem)
92+
else
93+
finalize(mmap_data)
94+
end
9195
end
9296
end
9397

@@ -116,7 +120,11 @@ function _csvread_f(file::AbstractString, delim=','; kwargs...)
116120
try
117121
_csvread_internal(VectorBackedUTF8String(data), delim; filename=file, kwargs...)
118122
finally
119-
finalize(data)
123+
@static if VERSION v"1.11"
124+
finalize(data.ref.mem)
125+
else
126+
finalize(data)
127+
end
120128
end
121129
end
122130
end

0 commit comments

Comments
 (0)