[OSRM-talk] Compiler warnings policy
mateusz at loskot.net
Mon Sep 11 11:25:51 UTC 2017
On 11 September 2017 at 12:34, Daniel Hofmann <hofmann at mapbox.com> wrote:
> This Wiki page is a bit outdated; for example there are a few compiler
> warnings on old compilers in combination with specific boost versions.
AFAICT, compiling with VS 2017 for 64-bit target gives me ~3.5K warnings,
though from narrow set of issues:
> That said we should check warnings for Travis (and the Windows CI ..) and
> especially our production builds.
Unless there is -Werror policy, warnings are (almost) never checked :-)
> I think a "coding standard" boils down to:
> - adapt your naming convention to what's already present in the code base
> - use clang-format for formatting
> - include what you use
Anyway, I asked out of curiosity (seeing largish number of warnings w/ VS2017).
Mateusz Loskot, http://mateusz.loskot.net
More information about the OSRM-talk