Skip to content
New issue

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

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

Already on GitHub? # to your account

NIO: use Posix "namespace" more aggressively #1439

Merged
merged 1 commit into from
Mar 11, 2020
Merged

Conversation

compnerd
Copy link
Contributor

Convert many more cases of the raw address family constant to the
definition in the System version.

[One line description of your change]

Motivation:

[Explain here the context, and why you're making that change. What is the problem you're trying to solve.]

Modifications:

[Describe the modifications you've done.]

Result:

[After your change, what will change.]

Copy link
Contributor

@Lukasa Lukasa left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

Copy link
Contributor

@Lukasa Lukasa left a comment

Choose a reason for hiding this comment

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

Oh, right: this needs some SocketOptionLevel conversions at all of these usage sites.

@compnerd
Copy link
Contributor Author

Im very confused why I didn't need the conversion :/

Convert many more cases of the raw address family constant to the
definition in the System version.
@Lukasa
Copy link
Contributor

Lukasa commented Mar 11, 2020

@swift-nio-bot test this please

@Lukasa
Copy link
Contributor

Lukasa commented Mar 11, 2020

I think they're only necessary on Linux due to a mismatch in the way the constants are declared and the way they're used.

@Lukasa Lukasa added the 🔨 semver/patch No public API change. label Mar 11, 2020
@Lukasa Lukasa added this to the 2.15.0 milestone Mar 11, 2020
Copy link
Contributor

@Lukasa Lukasa left a comment

Choose a reason for hiding this comment

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

LGTM, thanks.

@Lukasa Lukasa merged commit f44d524 into apple:master Mar 11, 2020
pull bot pushed a commit to scope-demo/swift-nio that referenced this pull request Mar 11, 2020
Convert many more cases of the raw address family constant to the
definition in the System version.
@compnerd compnerd deleted the namespace branch March 11, 2020 18:55
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
🔨 semver/patch No public API change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants