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.

Sign up for GitHub

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

Jump to bottom

Add Null static field to handles #640

Merged
merged 2 commits into from
Aug 9, 2022
Merged

Add Null static field to handles #640

merged 2 commits into from
Aug 9, 2022

Conversation

elachlan
Copy link
Contributor

@elachlan elachlan commented Aug 5, 2022

Fixes #639

@elachlan
Copy link
Contributor Author

elachlan commented Aug 5, 2022

@fernandanavamoya

@elachlan elachlan force-pushed the NullFieldOnHandles branch 2 times, most recently from fdee5c2 to cdee3e4 Compare August 5, 2022 21:48
jnm2
jnm2 reviewed Aug 6, 2022
View reviewed changes
src/Microsoft.Windows.CsWin32/Generator.cs Outdated Show resolved Hide resolved
src/Microsoft.Windows.CsWin32/Generator.cs Outdated Show resolved Hide resolved
@AArnott AArnott self-assigned this Aug 9, 2022
@AArnott
Copy link
Member

AArnott commented Aug 9, 2022

Thanks for contributing, @elachlan!

@AArnott AArnott enabled auto-merge August 9, 2022 04:05
@AArnott AArnott merged commit 2dc756c into microsoft:main Aug 9, 2022
@elachlan elachlan deleted the NullFieldOnHandles branch August 9, 2022 05:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jnm2 jnm2 jnm2 left review comments

Assignees

@AArnott AArnott

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Null value for HINSTANCE
3 participants
@elachlan @AArnott @jnm2