search

JuliaIO / JpegTurbo.jl

Julia interface to libjpeg-turbo
MIT License
15 stars 7 forks source link

fix: removing C stdio usage #21

Closed johnnychen94 closed 2 years ago

johnnychen94 commented 2 years ago

Without investigating why stdio is broken in x86 system, this commit replaces the C fopen with julia read so that it works on x86 system.

The decoding performance is still the same, which indicates that libjpeg-turbo has no stremline support for FILE.

codecov[bot] commented 2 years ago

Codecov Report

Merging #21 (0649093) into master (faaabb9) will decrease coverage by 6.41%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #21      +/-   ##
==========================================
- Coverage   87.32%   80.90%   -6.42%     
==========================================
  Files           5        5              
  Lines         213      199      -14     
==========================================
- Hits          186      161      -25     
- Misses         27       38      +11
Impacted Files Coverage Δ
src/decode.jl 87.15% <100.00%> (-10.41%) :arrow_down:

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 faaabb9...0649093. Read the comment docs.