JuliaLang / MbedTLS.jl

Wrapper around mbedtls
Other
41 stars 50 forks source link

Use Base.IOError in closewrite #260

Closed gustafsson closed 2 years ago

gustafsson commented 2 years ago

Instead of MbedException, in line with #257

Fixes #259

codecov[bot] commented 2 years ago

Codecov Report

Merging #260 (6633354) into master (ecf7e5b) will decrease coverage by 0.20%. The diff coverage is 14.28%.

@@            Coverage Diff             @@
##           master     #260      +/-   ##
==========================================
- Coverage   73.77%   73.56%   -0.21%     
==========================================
  Files          12       12              
  Lines         713      715       +2     
==========================================
  Hits          526      526              
- Misses        187      189       +2     
Impacted Files Coverage Δ
src/error.jl 33.33% <0.00%> (-2.39%) :arrow_down:
src/ssl.jl 72.37% <16.66%> (-0.26%) :arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

gustafsson commented 2 years ago

I stumbled upon a related question when making this change: https://github.com/JuliaLang/MbedTLS.jl/pull/257/files#r997901371