<div dir="ltr"><div><div><div><div><div>This Wiki page is a bit outdated; for example there are a few compiler warnings on old compilers in combination with specific boost versions.<br></div><div>That said we should check warnings for Travis (and the Windows CI ..) and especially our production builds.<br></div><div><br></div>I think a "coding standard" boils down to:<br></div>- adapt your naming convention to what's already present in the code base<br></div>- use clang-format for formatting<br></div>- include what you use</div><div><br></div><div>for everything else someone will flag it on the pull request.</div><div><br></div><div>HTH,</div><div>Daniel J H<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Sep 10, 2017 at 9:01 PM, Mateusz Loskot <span dir="ltr"><<a href="mailto:mateusz@loskot.net" target="_blank">mateusz@loskot.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
Is this policy still respected?<br>
<br>
"Treat Compiler Warning as Errors!" [1]<br>
<br>
[1] <a href="https://github.com/Project-OSRM/osrm-backend/wiki/Coding-Standards" rel="noreferrer" target="_blank">https://github.com/Project-<wbr>OSRM/osrm-backend/wiki/Coding-<wbr>Standards</a><br>
<br>
Best regards,<br>
<span class="HOEnZb"><font color="#888888">--<br>
Mateusz Loskot, <a href="http://mateusz.loskot.net" rel="noreferrer" target="_blank">http://mateusz.loskot.net</a><br>
<br>
______________________________<wbr>_________________<br>
OSRM-talk mailing list<br>
<a href="mailto:OSRM-talk@openstreetmap.org">OSRM-talk@openstreetmap.org</a><br>
<a href="https://lists.openstreetmap.org/listinfo/osrm-talk" rel="noreferrer" target="_blank">https://lists.openstreetmap.<wbr>org/listinfo/osrm-talk</a><br>
</font></span></blockquote></div><br></div>