Skip to content

Add mip level count and array layer count validation to texture_create_view#844

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

Add mip level count and array layer count validation to texture_create_view#844
bors[bot] merged 1 commit intogfx-rs:masterfrom
kunalmohan:create-view

Conversation

@kunalmohan
Copy link
Copy Markdown
Contributor

@kunalmohan kunalmohan commented Jul 28, 2020

Connections
Link to the issues addressed by this PR, or dependent PRs in other repositories

Description
Describe what problem this is solving, and how it's solved.
Catches the error when mip_level_count = 0 and base_mip_level > texture.mip_level_count (same with array_layer_count)

Testing
Explain how this change is tested.
Haven't been tested yet

@kunalmohan
Copy link
Copy Markdown
Contributor Author

I have tested with wgpu-rs examples. It works fine.

@kunalmohan
Copy link
Copy Markdown
Contributor Author

I'll rebase this once #846 gets merged

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.

Sorry, one last step!

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.

Thank you!
bors r+

@bors
Copy link
Copy Markdown
Contributor

bors bot commented Jul 29, 2020

@bors bors bot merged commit 75292f3 into gfx-rs:master Jul 29, 2020
@kunalmohan kunalmohan deleted the create-view branch July 29, 2020 18:30
kvark added a commit to kvark/wgpu that referenced this pull request Jun 3, 2021
844: Update wgpu to the cross feature fix r=kvark a=kvark



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