gauteh / dars

𓃢 A fast and light OPeNDAP server
MIT License
20 stars 4 forks source link

Simple cleanup #25

Closed magnusuMET closed 2 years ago

magnusuMET commented 2 years ago

Updating the hdf5-rust branch, formatting, clippy lints, and removal of some unsafe in this crate

magnusuMET commented 2 years ago

Still a small memory leak, but number of leaks has gone from 2464 byte(s) in 308 object(s) to 4 byte(s) in 2 object(s)

codecov-commenter commented 2 years ago

Codecov Report

Merging #25 (9328760) into master (da5f016) will increase coverage by 0.12%. The diff coverage is 81.30%.

@@            Coverage Diff             @@
##           master      #25      +/-   ##
==========================================
+ Coverage   74.46%   74.58%   +0.12%     
==========================================
  Files          24       24              
  Lines        1167     1157      -10     
==========================================
- Hits          869      863       -6     
+ Misses        298      294       -4     
Impacted Files Coverage Δ
dap2/src/dods/xdr.rs 21.05% <0.00%> (+3.66%) :arrow_up:
dap2/src/lib.rs 54.54% <ø> (ø)
dars/src/config.rs 15.38% <0.00%> (ø)
dars/src/main.rs 5.26% <ø> (ø)
dars-catalog/src/handlers.rs 60.86% <50.00%> (ø)
dars/src/data/handlers.rs 78.57% <50.00%> (ø)
dap2/src/dods/mod.rs 61.76% <66.66%> (-0.74%) :arrow_down:
dap2/src/constraint.rs 95.83% <100.00%> (-0.60%) :arrow_down:
dap2/src/dds.rs 86.97% <100.00%> (+0.46%) :arrow_up:
dap2/src/hyperslab.rs 84.00% <100.00%> (ø)
... and 12 more

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 da5f016...9328760. Read the comment docs.