Skip to content

Make ComputeCommand public#496

Closed
imiklos wants to merge 1 commit intogfx-rs:masterfrom
imiklos:command_encoder_public
Closed

Make ComputeCommand public#496
imiklos wants to merge 1 commit intogfx-rs:masterfrom
imiklos:command_encoder_public

Conversation

@imiklos
Copy link
Copy Markdown

@imiklos imiklos commented Feb 18, 2020

Set public the ComputeCommand, PhantomSlice and some underlying functions.

@imiklos imiklos changed the title Make CommandEncoder public Make ComputeCommand public Feb 18, 2020
Set public the `ComputeCommand`, `PhantomSlice` and some underlying functions.
Copy link
Copy Markdown
Member

@kvark kvark left a comment

Choose a reason for hiding this comment

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

Do we really need to use this directly? wgpu-core exposes a bunch of easy to use C functions that can be used to encode passes.

@imiklos
Copy link
Copy Markdown
Author

imiklos commented Feb 19, 2020

Thank you for pointing that out! We can definitely use the exposed C functions, as you mentioned. So I'm closing this pull request.

@imiklos imiklos closed this Feb 19, 2020
kvark pushed a commit to kvark/wgpu that referenced this pull request Jun 3, 2021
496: Replace u32 with NonZeroU32 in TextureView r=kvark a=kunalmohan

Includes gfx-rs#865

Co-authored-by: Kunal Mohan <kunalmohan99@gmail.com>
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.

2 participants