Skip to content

Use function pointers for interop in System.Net.NetworkInformation - #52192

Merged
jkotas merged 1 commit into
dotnet:mainfrom
jkotas:function-pointers
May 6, 2021
Merged

jkotas merged 1 commit into
dotnet:mainfrom
jkotas:function-pointers

Conversation

@jkotas

@jkotas jkotas commented May 3, 2021

Copy link
Copy Markdown
Member

No description provided.

@ghost ghost added the area-System.Net label May 3, 2021
@ghost

ghost commented May 3, 2021

Copy link
Copy Markdown

Tagging subscribers to this area: @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

Issue Details
Author: jkotas
Assignees: -
Labels:

area-System.Net

Milestone: -

@jkotas

jkotas commented May 3, 2021

Copy link
Copy Markdown
Member Author

Function pointers for interop are more performant and more AOT friendly.

@jkotas
jkotas requested a review from wfurt May 3, 2021 15:54
@jkotas
jkotas force-pushed the function-pointers branch from 7e158a6 to 97ee97d Compare May 4, 2021 00:14
@jkotas
jkotas force-pushed the function-pointers branch from 68275ec to 0b57b57 Compare May 4, 2021 01:28
@jkotas

jkotas commented May 4, 2021

Copy link
Copy Markdown
Member Author

CI is green. This is ready for review.

@wfurt wfurt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM.
We should be able to simplify many places to avoid the callbacks completely but it was always low priority.

@jkotas
jkotas merged commit bf62ae4 into dotnet:main May 6, 2021
@jkotas
jkotas deleted the function-pointers branch May 6, 2021 18:30
@karelz karelz added this to the 6.0.0 milestone May 20, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Jun 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants