Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Simplified IPC code. #277

Merged
merged 1 commit into from
Aug 12, 2021
Merged

Simplified IPC code. #277

merged 1 commit into from
Aug 12, 2021

Conversation

jorgecarleitao
Copy link
Owner

Makes it more obvious why unsafe there is sound.

@jorgecarleitao jorgecarleitao added the enhancement An improvement to an existing feature label Aug 11, 2021
@codecov
Copy link

codecov bot commented Aug 11, 2021

Codecov Report

Merging #277 (70825fd) into main (4c18c0a) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #277   +/-   ##
=======================================
  Coverage   77.14%   77.15%           
=======================================
  Files         254      254           
  Lines       20648    20649    +1     
=======================================
+ Hits        15929    15931    +2     
+ Misses       4719     4718    -1     
Impacted Files Coverage Δ
src/io/ipc/read/read_basic.rs 88.50% <100.00%> (+0.13%) ⬆️
src/compute/arithmetics/mod.rs 51.87% <0.00%> (+0.75%) ⬆️

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 4c18c0a...70825fd. Read the comment docs.

@jorgecarleitao jorgecarleitao merged commit c7e1a4d into main Aug 12, 2021
@jorgecarleitao jorgecarleitao deleted the improved_code branch August 12, 2021 06:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement An improvement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant