Skip to content

Add #![deny(warnings)] to the crate #83

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
Closed
alexcrichton opened this issue Sep 30, 2016 · 2 comments
Closed

Add #![deny(warnings)] to the crate #83

alexcrichton opened this issue Sep 30, 2016 · 2 comments

Comments

@alexcrichton
Copy link
Member

With lots of platform specific code it's pretty likely that a whole bunch of various warnings crop up here or there. What do you think about trying to reel these in with a blanket denial of all warnings?
@japaric
Copy link
Member

japaric commented Oct 1, 2016

What do you think about trying to reel these in with a blanket denial of all warnings?

Or ... we could just set #![allow(warnings)]!

Sounds like a chore. But I'm in favor.

@alexcrichton
Copy link
Member Author

Done!

tgross35 pushed a commit to tgross35/compiler-builtins that referenced this issue Feb 23, 2025
@bors @erikdesjardins
Merge rust-lang#83
7bedb61 
83: implement log10 and log10f r=japaric a=erikdesjardins

closes rust-lang#24, closes rust-lang#25

Co-authored-by: Erik <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@alexcrichton @japaric