Skip to content
This repository was archived by the owner on Oct 28, 2021. It is now read-only.

Fix failing to create DAG file larger than 2 Gb #4023

Merged
merged 1 commit into from
Mar 30, 2017
Merged

Conversation

gumb0
Copy link
Member

@gumb0 gumb0 commented Mar 29, 2017

Fixes #4021

fseek has 32-bit offset parameter, and there's no fully-portable way in C to do 64-bit seek, so we resort to different version for Windows & POSIX.

@codecov-io
Copy link

Codecov Report

Merging #4023 into develop will decrease coverage by 0.04%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##           develop    #4023      +/-   ##
===========================================
- Coverage    65.05%   65.01%   -0.05%     
===========================================
  Files          306      306              
  Lines        21350    21352       +2     
===========================================
- Hits         13889    13881       -8     
- Misses        7461     7471      +10
Impacted Files Coverage Δ
libethash/io.h 100% <ø> (ø) ⬆️
libethash/io_posix.c 85.36% <0%> (-4.38%) ⬇️
libethash/io.c 46.34% <0%> (ø) ⬆️
test/tools/fuzzTesting/fuzzHelper.cpp 31.97% <0%> (-4.07%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7cadd20...8209f08. Read the comment docs.

@gumb0 gumb0 merged commit 7d8a1b9 into develop Mar 30, 2017
@gumb0 gumb0 removed the in progress label Mar 30, 2017
@gumb0 gumb0 deleted the fix-mining-fseek branch June 8, 2017 14:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants