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

D3D12_PIPELINE_STATE_STREAM_DESC's pPipelineStateSubobjectStream should probably be a const pointer #119

Open
MarijnS95 opened this issue Jan 29, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@MarijnS95
Copy link
Contributor

https://learn.microsoft.com/en-us/windows/win32/api/d3d12/ns-d3d12-d3d12_pipeline_state_stream_desc should be input-only data, and is more accurately described by taking const void *pPipelineStateSubobjectStream;.

@walbourn walbourn added the enhancement New feature or request label Nov 7, 2024
@walbourn
Copy link
Member

walbourn commented Nov 7, 2024

For most of the history of Win32 APIs, structs don't use const void*. I've seen them showing up in newer DirectML.h headers. I suspect an issue here is that d3d12 core platform headers are generated using MIDL which doesn't support this annotation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants