Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PCM client delay reporting for A2DP sink profile #516

Merged
merged 1 commit into from
Oct 30, 2024
Merged

Conversation

arkq
Copy link
Owner

@arkq arkq commented Dec 9, 2021

This feature requires BlueZ >= 5.79

Copy link

codecov bot commented Oct 3, 2024

Codecov Report

Attention: Patch coverage is 84.74576% with 9 lines in your changes missing coverage. Please review.

Project coverage is 70.55%. Comparing base (f93a6e8) to head (9d37f0e).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
utils/aplay/aplay.c 78.26% 5 Missing ⚠️
src/ba-transport-pcm.c 85.71% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #516      +/-   ##
==========================================
+ Coverage   70.30%   70.55%   +0.25%     
==========================================
  Files          96       96              
  Lines       15799    16000     +201     
  Branches     2502     2509       +7     
==========================================
+ Hits        11107    11289     +182     
- Misses       4575     4594      +19     
  Partials      117      117              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@arkq arkq force-pushed the client-delay branch 2 times, most recently from ebb6794 to f7e303e Compare October 3, 2024 21:47
@arkq arkq force-pushed the client-delay branch 2 times, most recently from 6f52ccc to 6363778 Compare October 26, 2024 19:17
This feature requires BlueZ >= 5.79
@arkq arkq merged commit 9d37f0e into master Oct 30, 2024
39 checks passed
@arkq arkq deleted the client-delay branch October 30, 2024 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant