Skip to content
This repository has been archived by the owner on Jun 18, 2021. It is now read-only.

msaa-line example fixes #28

Merged
merged 1 commit into from
Jun 24, 2019
Merged

msaa-line example fixes #28

merged 1 commit into from
Jun 24, 2019

Conversation

rukai
Copy link
Contributor

@rukai rukai commented Jun 22, 2019

This PR fixes the msaa-line example in addition to the fixes in gfx-rs/wgpu#235

If gfx-rs/gfx#2853 is merged first we can remove the crates.io patch.

bors bot added a commit to gfx-rs/wgpu that referenced this pull request Jun 23, 2019
235: Full MSAA handling r=kvark a=rukai

These changes fix the msaa-line wgpu example, along with a PR to wgpu-native gfx-rs/wgpu-rs#28

Concerns:
*   webgpu does not expose a way for users to query limits, how are they supposed to choose a suitable sample_count?
*   I think `attachment_unused` should be moved into gfx-hal. Where abouts?
*   Is a sample mask of `:u64 = !0` suitable?

Co-authored-by: Rukai <rubickent@gmail.com>
@rukai
Copy link
Contributor Author

rukai commented Jun 24, 2019

Updated the commit, should be good to go.

Copy link
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.

bors r+

bors bot added a commit that referenced this pull request Jun 24, 2019
28: msaa-line example fixes r=kvark a=rukai

This PR fixes the msaa-line example in addition to the fixes in gfx-rs/wgpu#235

If gfx-rs/gfx#2853 is merged first we can remove the crates.io patch.

Co-authored-by: Rukai <rubickent@gmail.com>
@bors
Copy link
Contributor

bors bot commented Jun 24, 2019

Build succeeded

@bors bors bot merged commit b235c96 into gfx-rs:master Jun 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants