Skip to content

[core] Fixed the behavior and error code returned by srt_accept_bond (adapted for dev) - #3424

Merged
ethouris merged 15 commits into
Haivision:devfrom
ethouris:dev-dev-fix-accept-bond
Nov 2, 2026
Merged

ethouris merged 15 commits into
Haivision:devfrom
ethouris:dev-dev-fix-accept-bond

Conversation

@ethouris

Copy link
Copy Markdown
Collaborator

Adapted #3335 for dev

@ethouris ethouris added this to the v1.6.0 milestone Oct 29, 2026
@ethouris ethouris changed the title [core] Fixed the behavior and error code returned by srt_accept_bond [core] Fixed the behavior and error code returned by srt_accept_bond (adapted for dev) Oct 29, 2026
@codecov

codecov Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 72.22222% with 20 lines in your changes missing coverage. Please review.
✅ Project coverage is 57.73%. Comparing base (679f83a) to head (6d70d9b).
⚠️ Report is 368 commits behind head on dev.

Files with missing lines Patch % Lines
test/test_bonding.cpp 68.75% 7 Missing and 3 partials ⚠️
srtcore/epoll.cpp 25.00% 6 Missing ⚠️
srtcore/api.cpp 60.00% 2 Missing and 2 partials ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##              dev    #3424       +/-   ##
===========================================
- Coverage   71.12%   57.73%   -13.40%     
===========================================
  Files         101      114       +13     
  Lines       19776    26620     +6844     
  Branches        0     4031     +4031     
===========================================
+ Hits        14065    15368     +1303     
- Misses       5711     9730     +4019     
- Partials        0     1522     +1522     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ethouris
ethouris merged commit 49140b9 into Haivision:dev Nov 2, 2026
18 of 19 checks passed
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.

3 participants