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

Explicit delegate types #5954

Merged
merged 1 commit into from
May 9, 2022
Merged

Conversation

kant2002
Copy link
Contributor

This is done to please AOT compilers.
Removed not used MilGlyphCache_SetCreateGlyphBitmapsCallback
Technically NativeAOT would handle that after dotnet/runtime#63219 but want to cleanup just in case
overall I think using untyped delegates in C# is wrong

Fixes dotnet/runtimelab#1107

This is done to please AOT compilers.

Removed not used MilGlyphCache_SetCreateGlyphBitmapsCallback

Technically NativeAOT would handle that after dotnet/runtime#63219 but want to cleanup just in case

overall I think using untyped delegates in C# is wrong
@kant2002 kant2002 requested a review from a team as a code owner January 16, 2022 04:55
@ghost ghost added the PR metadata: Label to tag PRs, to facilitate with triage label Jan 16, 2022
@ghost ghost requested review from fabiant3, ryalanms and SamBent January 16, 2022 04:55
@dipeshmsft dipeshmsft self-assigned this Jan 17, 2022
@kant2002
Copy link
Contributor Author

@dipeshmsft can I ask, is this kind of changes are welcome here? I probably have bunch of similar one, in order to improve AOT scenarios for WPF

@dipeshmsft
Copy link
Member

Hey @kant2002 these kind of changes are welcome. But at of now, we are working on getting to a stable state after which we will start taking changes.

@kant2002
Copy link
Contributor Author

Can I ask to add this to "Queue to test pass" ?

@dipeshmsft
Copy link
Member

Hey @kant2002 we are still looking at the changes. Once done, we will add the label for this PR.

@ghost ghost assigned kant2002 Apr 5, 2022
@dipeshmsft dipeshmsft merged commit 5c57653 into dotnet:main May 9, 2022
@kant2002 kant2002 deleted the kant/delegate-cleanup branch May 9, 2022 06:49
@ghost ghost locked as resolved and limited conversation to collaborators Jun 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
PR metadata: Label to tag PRs, to facilitate with triage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WPF require MulticastDelegate
2 participants