Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove redundant #[cfg] conditions from backend/direct.rs. #2811

Merged
merged 1 commit into from
Jun 26, 2022

Conversation

jimblandy
Copy link
Member

The entire wgpu::backend::direct module is included only under the
condition:

#[cfg(any(not(target_arch = "wasm32"), feature = "webgl"))]

So there is no need to apply this condition to any item within that
module.

The entire `wgpu::backend::direct` module is included only under the
condition:

    #[cfg(any(not(target_arch = "wasm32"), feature = "webgl"))]

So there is no need to apply this condition to any item within that
module.
@jimblandy jimblandy requested a review from cwfitzgerald June 26, 2022 22:38
Copy link
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.

Dope!

@cwfitzgerald cwfitzgerald merged commit 788cd7d into gfx-rs:master Jun 26, 2022
@jimblandy jimblandy deleted the backend-redundant-cfg branch June 28, 2022 00:39
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