Skip to content

Conversation

@barrbrain
Copy link
Collaborator

No description provided.

@barrbrain barrbrain force-pushed the lossless branch 4 times, most recently from 7020e1b to 2efb217 Compare October 19, 2023 06:39
@codecov
Copy link

codecov bot commented Oct 19, 2023

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Files Coverage Δ
src/asm/x86/transform/forward.rs 97.83% <ø> (ø)
src/encoder.rs 86.95% <100.00%> (+0.01%) ⬆️
src/transform/forward.rs 100.00% <100.00%> (ø)
src/transform/forward_shared.rs 99.86% <100.00%> (+<0.01%) ⬆️
src/transform/inverse.rs 99.68% <100.00%> (+0.01%) ⬆️
src/transform/mod.rs 90.65% <100.00%> (+0.68%) ⬆️
src/asm/x86/transform/inverse.rs 96.49% <84.61%> (-3.51%) ⬇️

📢 Thoughts on this report? Let us know!.

@barrbrain barrbrain force-pushed the lossless branch 2 times, most recently from 64ef14c to 636fe9b Compare October 19, 2023 08:00
@barrbrain barrbrain marked this pull request as ready for review October 19, 2023 08:01
Copy link
Collaborator

@lu-zero lu-zero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems fine, I now wonder if the various dispatching functions could be marked inline.

@barrbrain barrbrain merged commit 47a9d1b into xiph:master Oct 19, 2023
@barrbrain barrbrain deleted the lossless branch October 19, 2023 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants