Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

clean up and export crc32c function #22274

Merged
merged 14 commits into from
Jun 13, 2017
Merged

clean up and export crc32c function #22274

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
643b1e3
clean up and export crc32c function
stevengj Jun 7, 2017
721bbf5
added PR to NEWS
stevengj Jun 7, 2017
184acd1
use read, not mmap, for crc
stevengj Jun 7, 2017
c3f5d91
slight optimization (don't open cachefile twice for checksum)
stevengj Jun 7, 2017
bd64651
avoid resizing buffer in readbytes
stevengj Jun 7, 2017
62d7dc4
add an assertion
stevengj Jun 7, 2017
ceda720
more crc32c tests
stevengj Jun 7, 2017
67ef143
docs typo
stevengj Jun 8, 2017
f34d08c
crc32c(IOBuffer) bugfix and test
stevengj Jun 8, 2017
38f6fc0
restore crc32 of String, add crc32c(io) to read all of a stream, add …
stevengj Jun 8, 2017
1126d0f
use crc32c block size of 8192*3, matching the underling C library
stevengj Jun 9, 2017
da2fba1
optimized IOBuffer crc32c
stevengj Jun 9, 2017
a1b0431
slight simplification/generalization
stevengj Jun 12, 2017
8e058f3
eliminate theoretical race condition in crc32c if file is being writt…
stevengj Jun 12, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
slight simplification/generalization
  • Loading branch information
@stevengj
stevengj committed Jun 12, 2017
commit a1b0431b1222dfe023948690c1fd98ddc9ea8aa3
2 changes: 1 addition & 1 deletion base/loading.jl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -683,7 +683,7 @@ function compilecache(name::String)
if success(create_expr_cache(path, cachefile, concrete_deps))
# append checksum to the end of the .ji file:
open(cachefile, "a+") do f
write(f, hton(crc32c(seekstart(f), filesize(f))))
write(f, hton(crc32c(seekstart(f))))
end
else
error("Failed to precompile $name to $cachefile.")
Expand Down
7 changes: 1 addition & 6 deletions base/util.jl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -817,12 +817,7 @@ function crc32c(io::IO, nb::Integer, crc::UInt32=0x00000000)
return unsafe_crc32c(buf, readbytes!(io, buf, nb), crc)
end
crc32c(io::IO, crc::UInt32=0x00000000) = crc32c(io, typemax(Int64), crc)

# optimization for `open(crc, filename)` to use the size of the file
open(::typeof(crc32c), filename::AbstractString) =
open(filename, "r") do f
crc32c(f, filesize(f))
end
crc32c(io::IOStream, crc::UInt32=0x00000000) = crc32c(io, filesize(io)-position(io), crc)


"""
Expand Down