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

Add SRTO_IPV6ONLY option #608

Merged
merged 15 commits into from
Apr 18, 2019
Merged

Add SRTO_IPV6ONLY option #608

merged 15 commits into from
Apr 18, 2019

Conversation

thomasjammet
Copy link
Contributor

Add SRTO_IPV6ONLY to SRT_SOCKOPT for setting the IPV6_V6ONLY sockoption before the ::bind call
Also delete a "performance warning" in Visual Studio (srt.h)

@maxsharabayko maxsharabayko added this to the v.1.3.3 milestone Mar 28, 2019
srtcore/core.cpp Show resolved Hide resolved
srtcore/core.cpp Show resolved Hide resolved
srtcore/core.h Outdated Show resolved Hide resolved
srtcore/channel.cpp Outdated Show resolved Hide resolved
- correction of ipv6only default value
- srt.h : correction of the operator& return condition, more readable
- now ipv6only is used for checking if an address is reusable
Copy link
Collaborator

@maxsharabayko maxsharabayko left a comment

Choose a reason for hiding this comment

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

New socket option SRTO_IPV6ONLY needs to be described in API.md.

@maxsharabayko maxsharabayko added [core] Area: Changes in SRT library core Type: Enhancement Indicates new feature requests labels Apr 18, 2019
thomasjammet added a commit to MonaSolutions/srt that referenced this pull request Apr 18, 2019
small correction of SRTO_IPV6ONLY option description, reported from PR Haivision#608
@rndi rndi merged commit d1ad82c into Haivision:master Apr 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[core] Area: Changes in SRT library core Type: Enhancement Indicates new feature requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants