[OSRM-talk] Compiler warnings policy
Daniel Hofmann
hofmann at mapbox.com
Mon Sep 11 10:34:35 UTC 2017
This Wiki page is a bit outdated; for example there are a few compiler
warnings on old compilers in combination with specific boost versions.
That said we should check warnings for Travis (and the Windows CI ..) and
especially our production builds.
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
for everything else someone will flag it on the pull request.
HTH,
Daniel J H
On Sun, Sep 10, 2017 at 9:01 PM, Mateusz Loskot <mateusz at loskot.net> wrote:
> Hi,
>
> Is this policy still respected?
>
> "Treat Compiler Warning as Errors!" [1]
>
> [1] https://github.com/Project-OSRM/osrm-backend/wiki/Coding-Standards
>
> Best regards,
> --
> Mateusz Loskot, http://mateusz.loskot.net
>
> _______________________________________________
> OSRM-talk mailing list
> OSRM-talk at openstreetmap.org
> https://lists.openstreetmap.org/listinfo/osrm-talk
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/osrm-talk/attachments/20170911/76c4b7d4/attachment.html>
More information about the OSRM-talk
mailing list