search

briansmith / ring

Safe, fast, small crypto using Rust
Other
3.65k stars 684 forks source link

cpu: Clarify `cfg(...)` logic by removing redundant checks. #1985

Closed briansmith closed 4 months ago

briansmith commented 4 months ago

Make it clearer when reading arm.rs and intel.rs tthat these modules are only compiled for the relevant target architectures. Remove redundant conditional logic that was confusing regarding this.

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.29%. Comparing base (5fb6541) to head (57ffafc).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1985 +/- ## ========================================== + Coverage 96.28% 96.29% +0.01% ========================================== Files 137 137 Lines 20723 20699 -24 Branches 226 226 ========================================== - Hits 19953 19933 -20 + Misses 735 731 -4 Partials 35 35 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.