Skip to content

[wgpu] correct doc for resolve query set#4849

Merged
Wumpf merged 3 commits intogfx-rs:trunkfrom
valaphee:resolve-query-set-doc
Dec 9, 2023
Merged

[wgpu] correct doc for resolve query set#4849
Wumpf merged 3 commits intogfx-rs:trunkfrom
valaphee:resolve-query-set-doc

Conversation

@valaphee
Copy link
Copy Markdown
Contributor

@valaphee valaphee commented Dec 8, 2023

Connections
Closes #4838

Description
Moved resolve_query_set to the general command encoder impl, as it technically doesn't require any feature, is not meaningful to the end user, and it's not possible to create a query set with an unsupported type anyway.

And added info about how many bytes to expect.

Testing

Checklist

  • Run cargo fmt.
  • Run cargo clippy. If applicable, add:
    • --target wasm32-unknown-unknown
    • --target wasm32-unknown-emscripten
  • Run cargo xtask test to run tests.
  • Add change to CHANGELOG.md. See simple instructions inside file.

@valaphee valaphee requested a review from a team as a code owner December 8, 2023 17:15
Copy link
Copy Markdown
Member

@Wumpf Wumpf left a comment

Choose a reason for hiding this comment

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

nice, thank you!

@Wumpf Wumpf enabled auto-merge (squash) December 9, 2023 11:07
@Wumpf Wumpf linked an issue Dec 9, 2023 that may be closed by this pull request
@cwfitzgerald
Copy link
Copy Markdown
Member

CI is failing

@Wumpf
Copy link
Copy Markdown
Member

Wumpf commented Dec 9, 2023

oops, my fault!

@Wumpf Wumpf merged commit 6c4128b into gfx-rs:trunk Dec 9, 2023
@valaphee valaphee deleted the resolve-query-set-doc branch December 21, 2023 22:13
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.

Incorrect docs for resolve_query_set

3 participants