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.

Sign up for GitHub

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

Jump to bottom

Lint basic arithmetic on integers or floating point arithmetic #887

Closed
llogiq opened this issue Apr 29, 2016 · 2 comments · Fixed by #889
Closed

Lint basic arithmetic on integers or floating point arithmetic #887

llogiq opened this issue Apr 29, 2016 · 2 comments · Fixed by #889
Labels
A-lint Area: New lints good-first-issue These issues are a good way to get started with Clippy

Comments

@llogiq
Copy link
Contributor

llogiq commented Apr 29, 2016

This would be an allow lint that could be activated for code where a more specific kind of arithmetic is wanted (e.g. wrapping or checked).

Courtesy of tomaka on reddit
@mcarton
Copy link
Member

mcarton commented Apr 29, 2016

We could also lint floating point arithmetic. See RFC: Float-free libcore rust-lang/rfcs#1596.

@mcarton mcarton added good-first-issue These issues are a good way to get started with Clippy A-lint Area: New lints labels Apr 29, 2016
@llogiq
Copy link
Contributor Author

llogiq commented Apr 29, 2016

Sure, the more the merrier 😄

@mcarton mcarton changed the title lint basic arithmetic on integers Lint basic arithmetic on integers or floating point arithmetic Apr 29, 2016
llogiq added a commit that referenced this issue Apr 30, 2016
@llogiq
fix #887: New lints for integer/floating-point arithmetic
9a99979
@llogiq llogiq mentioned this issue Apr 30, 2016
Merged
Manishearth added a commit that referenced this issue Apr 30, 2016
@Manishearth
Merge pull request #889 from Manishearth/fix-887
469b3d5 
fix #887: New lints for integer/floating-point arithmetic
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-lint Area: New lints good-first-issue These issues are a good way to get started with Clippy
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix #887: New lints for integer/floating-point arithmetic rust-lang/rust-clippy
2 participants
@mcarton @llogiq