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

Uppercase numeric constants #11790

Merged
merged 1 commit into from
Jan 25, 2014
Merged

Uppercase numeric constants #11790

merged 1 commit into from
Jan 25, 2014

Conversation

lambda-fairy
Copy link
Contributor

The following are renamed:

  • min_value => MIN
  • max_value => MAX
  • bits => BITS
  • bytes => BYTES

All tests pass, except for run-pass/phase-syntax-link-does-resolve.rs. I doubt that failure is related, though.

Fixes #10010.

The following are renamed:

* `min_value` => `MIN`
* `max_value` => `MAX`
* `bits` => `BITS`
* `bytes` => `BYTES`

Fixes #10010.
@alexcrichton
Copy link
Member

You get a gold star for closing a 1.0 milestone issue, thank you!

@huonw
Copy link
Member

huonw commented Jan 25, 2014

bors added a commit that referenced this pull request Jan 25, 2014
The following are renamed:

* `min_value` => `MIN`
* `max_value` => `MAX`
* `bits` => `BITS`
* `bytes` => `BYTES`

All tests pass, except for `run-pass/phase-syntax-link-does-resolve.rs`. I doubt that failure is related, though.

Fixes #10010.
@bors bors closed this Jan 25, 2014
@bors bors merged commit 988e4f0 into rust-lang:master Jan 25, 2014
@snowe2010 snowe2010 mentioned this pull request Nov 15, 2015
flip1995 pushed a commit to flip1995/rust that referenced this pull request Nov 16, 2023
…arth

Destructure `Conf` in `register_lints`

And some other miscellaneous clean-up in the area

changelog: none
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.

std::int::min_value, max_value, etc. should be capitalized
4 participants