Skip to content

Conversation

tbolon
Copy link
Contributor

@tbolon tbolon commented Sep 21, 2023

Summary

Add clarifications about how parameters are used when using Control.Invoke and parameters are supplied.

See Control.InvokeMarshaledCallbackDo for code.

Found when investigating for dotnet/winforms#9965

@ghost ghost added community-contribution Indicates that the PR has been added by a community member area-WinForms Issues or PRs that relate to WinForms. labels Sep 21, 2023
@learn-build-service-prod
Copy link

Learn Build status updates of commit 0de4ce0:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Windows.Forms/Control.xml ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

@gewarren gewarren left a comment

Choose a reason for hiding this comment

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

Some grammatical suggestions. Still needs technical review.

@learn-build-service-prod
Copy link

Learn Build status updates of commit 3f0fbec:

✅ Validation status: passed

File Status Preview URL Details
xml/System.Windows.Forms/Control.xml ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

Copy link
Member

@dreddy-work dreddy-work left a comment

Choose a reason for hiding this comment

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

LGTM.

@gewarren gewarren enabled auto-merge (squash) September 28, 2023 22:41
@gewarren gewarren merged commit 542cdb3 into dotnet:main Sep 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-WinForms Issues or PRs that relate to WinForms. community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants