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.

Sign up for GitHub

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

Jump to bottom

[core] A change in setupCC to access CUnitQueue via CRcvQueue instead of CRcvBuffer #1681

Merged
merged 2 commits into from
Dec 1, 2020
Merged

[core] A change in setupCC to access CUnitQueue via CRcvQueue instead of CRcvBuffer #1681

merged 2 commits into from
Dec 1, 2020

Conversation

maxsharabayko
Copy link
Collaborator

@maxsharabayko maxsharabayko commented Nov 30, 2020 •
edited
Loading

A work within #1674 to reduce changes when integrating the new receiver buffer.

TODO:

  • Delete CRcvBuffer::getUnitQueue()

@maxsharabayko
[core] A change in setupCC to access CUnitQueue
a98d6ac 
via CRcvQueue instead of CRcvBuffer
@maxsharabayko maxsharabayko added Type: Maintenance Work required to maintain or clean up the code [core] Area: Changes in SRT library core labels Nov 30, 2020
@maxsharabayko maxsharabayko added this to the v1.4.3 milestone Nov 30, 2020
@maxsharabayko maxsharabayko mentioned this pull request Nov 30, 2020
Closed
9 tasks
@codecov

This comment has been minimized.

Sign in to view
@maxsharabayko maxsharabayko marked this pull request as ready for review December 1, 2020 10:34
@maxsharabayko maxsharabayko merged commit c42bc13 into Haivision:master Dec 1, 2020
@maxsharabayko maxsharabayko deleted the develop/access-unit-queue branch December 1, 2020 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ethouris ethouris ethouris approved these changes

Assignees
No one assigned
Labels
[core] Area: Changes in SRT library core Type: Maintenance Work required to maintain or clean up the code
Projects
None yet
Milestone
v1.4.3
Development

Successfully merging this pull request may close these issues.
2 participants
@maxsharabayko @ethouris