Ogeon / palette

A Rust library for linear color calculations and conversion
Apache License 2.0
755 stars 60 forks source link

Fix upcoming Clippy lifetime warnings #419

Closed Ogeon closed 1 month ago

Ogeon commented 1 month ago

Clarifies implicitly named lifetimes, and elides named lifetimes that only appear once. Shouldn't affect the type checking.

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 0% with 14 lines in your changes missing coverage. Please review.

Project coverage is 82.83%. Comparing base (e37e2fe) to head (d78e89d). Report is 2 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #419 +/- ## ========================================== - Coverage 82.87% 82.83% -0.04% ========================================== Files 130 131 +1 Lines 20199 20162 -37 Branches 20199 20162 -37 ========================================== - Hits 16739 16702 -37 Misses 3337 3337 Partials 123 123 ``` | [Components](https://app.codecov.io/gh/Ogeon/palette/pull/419/components?src=pr&el=components&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Erik+Hedvall) | Coverage Δ | | |---|---|---| | [palette](https://app.codecov.io/gh/Ogeon/palette/pull/419/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Erik+Hedvall) | `82.90% <0.00%> (-0.04%)` | :arrow_down: | | [palette_derive](https://app.codecov.io/gh/Ogeon/palette/pull/419/components?src=pr&el=component&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Erik+Hedvall) | `81.98% <ø> (ø)` | |
codspeed-hq[bot] commented 1 month ago

CodSpeed Performance Report

Merging #419 will improve performances by 14.99%

Comparing clippy_fixes (d78e89d) with master (e37e2fe)

Summary

⚡ 2 improvements ✅ 45 untouched benchmarks

Benchmarks breakdown

Benchmark master clippy_fixes Change
matrix_inverse 438.9 ns 381.7 ns +14.99%
multiply_3x3 316.7 ns 287.5 ns +10.14%