Recommendations for enabling more compiler checks

Compilers enable by default, a set of warnings they will emit on dodgy or less-than-ideal code. These defaults:

• a. change over time
• b. vary between compiler versions
• c. do not include some of the most valuable warnings that might help prevent bugs

This issue is designed to focus on the problem of c from the perspective of the latest clang++ 4.x versions.

Enabling more warnings than compilers do by default can be an important way of catching bugs early. For example integer truncation bugs can often be detected by enabling -Wconversion, which is not on by default.

Enabling more warnings is very difficult to do after a project is big (e.g. https://github.com/Project-OSRM/osrm-backend/pull/4495 and https://github.com/mapnik/mapnik/issues/2907 and https://github.com/mapnik/mapnik/issues/3204). It is best not to wait and rather to start a project with aggressive warnings from the beginning.

So, the question then becomes: what is a good set of aggressive warnings to enable at the start of a project (or to try to integrate into existing projects)?

In particular:

1. 🍇 Which flags we should always enable for all code no matter what?

2. 🍊 What additional flags may be very useful in some scenarios/some code bases?

3. 🍏 For small projects where it is feasible, can we actually start with clang++s -Weverything? This, when feasible, might be ideal. How to do it?

4. 🍎 What compiler specific flags should we recommend (that only currently work for clang++ or g++)?

首先，检查 issue 中关于 clang++ 4.x 中激进警告的四个问题，包括 -Wconversion 和 -Weverything。研究哪些 flags 广泛有用、取决于场景或特定于编译器，然后记录一项明确的建议，并解释小型项目可以如何应用它；当警告指南回答了全部四个问题时，即视为完成。