Skip to content

Make level and layer count for texture view optional#846

Merged
bors[bot] merged 1 commit intogfx-rs:masterfrom
kvark:tex-view
Jul 29, 2020
Merged

Make level and layer count for texture view optional#846
bors[bot] merged 1 commit intogfx-rs:masterfrom
kvark:tex-view

Conversation

@kvark
Copy link
Copy Markdown
Member

@kvark kvark commented Jul 28, 2020

Connections
implements gpuweb/gpuweb#945

Description
NonZeroU32 is more idiomatic here

Testing
untested, but should work

@kvark kvark requested a review from cwfitzgerald July 28, 2020 19:10
Copy link
Copy Markdown
Member

@cwfitzgerald cwfitzgerald left a comment

Choose a reason for hiding this comment

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

Looks great, a couple of docs fixes needed, but otherwise good.

@kvark
Copy link
Copy Markdown
Member Author

kvark commented Jul 29, 2020

Does are updated
bors r=cwfitzgerald

@bors
Copy link
Copy Markdown
Contributor

bors bot commented Jul 29, 2020

@bors bors bot merged commit 9929b87 into gfx-rs:master Jul 29, 2020
@kvark kvark deleted the tex-view branch July 29, 2020 03:10
bors bot added a commit to gfx-rs/wgpu-rs that referenced this pull request Jul 29, 2020
488: Update wgpu with view descriptor changes r=kvark a=kvark

Includes gfx-rs/wgpu#846

Co-authored-by: Dzmitry Malyshau <kvarkus@gmail.com>
kvark added a commit to kvark/wgpu that referenced this pull request Jun 3, 2021
488: Update wgpu with view descriptor changes r=kvark a=kvark

Includes gfx-rs#846

Co-authored-by: Dzmitry Malyshau <kvarkus@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