Skip to content

Conversation

@riverar
Copy link
Collaborator

@riverar riverar commented Jan 5, 2024

Fixes: #1726

Add a new Retained attribute to indicate when pointers to structures must remain allocated throughout the lifetime of an asynchronous operation.

@riverar riverar requested a review from mikebattista as a code owner January 5, 2024 09:01
@riverar riverar requested a review from AArnott January 5, 2024 09:02
AArnott
AArnott previously approved these changes Jan 5, 2024
Copy link
Member

@AArnott AArnott left a comment

Choose a reason for hiding this comment

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

This looks good. But what about DeviceIoControl? Are there other APIs that use OVERLAPPED pointers that we should include in this?

@mikebattista
Copy link
Collaborator

@riverar can you respond to @AArnott's questions?

@riverar
Copy link
Collaborator Author

riverar commented Feb 16, 2024

Thanks for the ping. I can add those to this PR too. Standby.

@AArnott
Copy link
Member

AArnott commented May 3, 2024

@riverar any update?

@riverar
Copy link
Collaborator Author

riverar commented May 3, 2024

Catching up on PRs, good timing. Adding now.

@riverar
Copy link
Collaborator Author

riverar commented May 3, 2024

Updated. If I missed any APIs, happy to add them.

AArnott
AArnott previously approved these changes May 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HttpReceiveHttpRequest OVERLAPPED* parameter should be marked as a long-pin pointer

4 participants