Skip to content

Fix warnings in rustc nightly due to introduction of From<f16> for f32#23971

Merged
alice-i-cecile merged 1 commit intobevyengine:mainfrom
HackerFoo:fix_f32_warnings
Apr 24, 2026
Merged

Fix warnings in rustc nightly due to introduction of From<f16> for f32#23971
alice-i-cecile merged 1 commit intobevyengine:mainfrom
HackerFoo:fix_f32_warnings

Conversation

@HackerFoo
Copy link
Copy Markdown
Contributor

@HackerFoo HackerFoo commented Apr 24, 2026

Objective

There's a new warning for code that uses literals with Into<f32> because the compiler can no longer infer that the literal should be an f32 after From<f16> for f32 was introduced.

See: rust-lang/rust#154024

Solution

Explicit f32 literals.

Note

This behavior might make Into<f32> arguments inconvenient in the future.

@alice-i-cecile alice-i-cecile added D-Trivial Nice and easy! A great choice to get started with Bevy A-UI Graphical user interfaces, styles, layouts, and widgets C-Code-Quality A section of code that is hard to understand or change S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it labels Apr 24, 2026
@github-project-automation github-project-automation Bot moved this to Needs SME Triage in UI Apr 24, 2026
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Apr 24, 2026
Merged via the queue into bevyengine:main with commit 9b8a234 Apr 24, 2026
47 checks passed
@github-project-automation github-project-automation Bot moved this from Needs SME Triage to Done in UI Apr 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-UI Graphical user interfaces, styles, layouts, and widgets C-Code-Quality A section of code that is hard to understand or change D-Trivial Nice and easy! A great choice to get started with Bevy S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants