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

Prefer new associated numeric consts in float error messages #78431

Merged
merged 1 commit into from
Oct 30, 2020

Conversation

Rustin170506
Copy link
Member

@Rustin170506 Rustin170506 commented Oct 27, 2020

Fix #78382

@rust-highfive
Copy link
Collaborator

r? @davidtwco

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 27, 2020
@Rustin170506 Rustin170506 changed the title Prefer new associated numeric consts in float error messages WIP: Prefer new associated numeric consts in float error messages Oct 27, 2020
@Rustin170506 Rustin170506 marked this pull request as draft October 27, 2020 06:18
@Rustin170506 Rustin170506 marked this pull request as ready for review October 27, 2020 10:26
@Rustin170506 Rustin170506 changed the title WIP: Prefer new associated numeric consts in float error messages Prefer new associated numeric consts in float error messages Oct 27, 2020
@Rustin170506
Copy link
Member Author

r? @estebank @bstrie

@rust-highfive rust-highfive assigned estebank and unassigned davidtwco Oct 27, 2020
@estebank
Copy link
Contributor

I think there are other cases where we still use the old mod consts. I'll keep the ticket open to check that.

@bors r+

@bors
Copy link
Contributor

bors commented Oct 27, 2020

📌 Commit 650e3cb has been approved by estebank

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 27, 2020
bors added a commit to rust-lang-ci/rust that referenced this pull request Oct 29, 2020
…as-schievink

Rollup of 11 pull requests

Successful merges:

 - rust-lang#75078 (Improve documentation for slice strip_* functions)
 - rust-lang#76138 (Explain fully qualified syntax for `Rc` and `Arc`)
 - rust-lang#78244 (Dogfood {exclusive,half-open} ranges in compiler (nfc))
 - rust-lang#78422 (Do not ICE on invalid input)
 - rust-lang#78423 (rustc_span: improve bounds checks in byte_pos_to_line_and_col)
 - rust-lang#78431 (Prefer new associated numeric consts in float error messages)
 - rust-lang#78462 (Use unwrapDIPtr because the Scope may be null.)
 - rust-lang#78493 (Update cargo)
 - rust-lang#78499 (Prevent String::retain from creating non-utf8 strings when abusing panic)
 - rust-lang#78505 (Update Clippy - temporary_cstring_as_ptr deprecation)
 - rust-lang#78527 (Fix some more typos)

Failed merges:

r? `@ghost`
@bors bors merged commit 31cfe63 into rust-lang:master Oct 30, 2020
@rustbot rustbot added this to the 1.49.0 milestone Oct 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Prefer new associated numeric consts in float error messages
6 participants