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 various unused fields in naga #6025

Merged
merged 3 commits into from
Jul 24, 2024

Conversation

ErichDonGubler
Copy link
Member

@ErichDonGubler ErichDonGubler commented Jul 24, 2024

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.

Testing
Explain how this change is tested.

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.

@ErichDonGubler ErichDonGubler added naga Shader Translator kind: refactor Making existing function faster or nicer labels Jul 24, 2024
@ErichDonGubler ErichDonGubler self-assigned this Jul 24, 2024
@ErichDonGubler ErichDonGubler marked this pull request as ready for review July 24, 2024 13:07
@ErichDonGubler ErichDonGubler requested a review from a team as a code owner July 24, 2024 13:07
@ErichDonGubler
Copy link
Member Author

I'm not sure why CI doesn't catch this, but I see warnings for these unused fields consistently while running cargo clippy --workspace --all-targets --all-features on my M1 Mac workstation.

@waywardmonkeys
Copy link
Contributor

CI is running an older version of Rust (currently 1.76).

@ErichDonGubler ErichDonGubler merged commit 591e1d2 into gfx-rs:trunk Jul 24, 2024
25 checks passed
@ErichDonGubler ErichDonGubler deleted the push-uyuqntysxppy branch July 24, 2024 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind: refactor Making existing function faster or nicer naga Shader Translator
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants