Skip to content

Remove is_integer_const#17204

Open
Jarcho wants to merge 2 commits into
rust-lang:masterfrom
Jarcho:int_const
Open

Remove is_integer_const#17204
Jarcho wants to merge 2 commits into
rust-lang:masterfrom
Jarcho:int_const

Conversation

@Jarcho

@Jarcho Jarcho commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Most instances of is_integer_const were used by style lints which never should have used it in the first place. e.g. range_plus_one will no longer suggest changing x..y + C to x..=y just because C is currently defined as one; it's a thing which can change. The final uses of it have been replaced with eval_int which gives a notably better API in one case a mildly worse one in the other case.

changelog: none

@rustbot

rustbot commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

r? @dswij

rustbot has assigned @dswij.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: 8 candidates
  • 8 candidates expanded to 8 candidates
  • Random selection from dswij, llogiq, samueltardieu

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.

3 participants