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

config: allow for custom SSRC. Added thread safety #185

Merged
merged 1 commit into from
Dec 21, 2022

Conversation

wowaser
Copy link
Contributor

@wowaser wowaser commented Dec 7, 2022

An alternative solution for #180. Here, we share an SSRC between the objects in a thread-safe way. No intrusion on the API, as opposed to #182.

@wowaser
Copy link
Contributor Author

wowaser commented Dec 7, 2022

I corrected the error that the circleci is giving, but it's not updating

@jrsnen jrsnen added the safe to test Run tests on CI label Dec 13, 2022
@jrsnen
Copy link
Member

jrsnen commented Dec 13, 2022

Hi,

I tested and this does not compile on Linux due to warnings, if you could fix those, otherwise this looks acceptable.

BR, Joni

@wowaser wowaser force-pushed the refSSRC branch 3 times, most recently from 65d4445 to db19b52 Compare December 13, 2022 13:36
@wowaser
Copy link
Contributor Author

wowaser commented Dec 13, 2022

I think now it is good

@jrsnen
Copy link
Member

jrsnen commented Dec 21, 2022

Looks good!

@jrsnen jrsnen merged commit 176e855 into ultravideo:master Dec 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
safe to test Run tests on CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants