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

Compiling chrono generates an inordinate number of warning messages #77

Closed
DavidHammen opened this issue Dec 16, 2015 · 1 comment
Closed

Comments

@DavidHammen
Copy link

My bare minimum for a quality C++ software library is that it must compile clean with -Wall using either a GNU or LLVM compiler. The chrono code fails to meet this minimal standard. With -Wall, I get several thousands of lines of warning messages. In fact, the chrono code does not compile clean even without -Wall.

@thepianoboy
Copy link
Contributor

thepianoboy commented Mar 2, 2017

Partially fixed with commit db1ad88
Compiling cleanly with -Wall is somewhat unrealistic for a package with external dependencies like Irrlicht or CUDA. The above commit updates several modules and the core package to build on either GCC or LLVM with typical options and -Werror enabled. This deals with the most common and egregious warnings, and keeps the package clean enough for most open source compiler users. Additional warnings will be addressed in the future, possibly including -Wpedantic to ensure proper cross-compiler compatibility.

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

No branches or pull requests

2 participants