<div dir="ltr"><div>If you're familiar with VS2017 and development on Windows, feel free to enable all warnings but then disable warnings that are not critical (think: indentation, etc.).<br><br><br></div>I think our AppVeyor build is also quite spammy in that regard. The main problem is the core devs don't run Windows :)<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 11, 2017 at 1:25 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"><span class="">On 11 September 2017 at 12:34, Daniel Hofmann <<a href="mailto:hofmann@mapbox.com">hofmann@mapbox.com</a>> wrote:<br>
> This Wiki page is a bit outdated; for example there are a few compiler<br>
> warnings on old compilers in combination with specific boost versions.<br>
<br>
</span>AFAICT, compiling with VS 2017 for 64-bit target gives me ~3.5K warnings,<br>
though from narrow set of issues:<br>
<br>
C4067<br>
C4101<br>
C4146<br>
C4244<br>
C4267<br>
C4305<br>
C4309<br>
C4373<br>
C4396<br>
C4717<br>
C4800<br>
<span class=""><br>
> That said we should check warnings for Travis (and the Windows CI ..) and<br>
> especially our production builds.<br>
<br>
</span>Unless there is -Werror policy, warnings are (almost) never checked :-)<br>
<span class=""><br>
> I think a "coding standard" boils down to:<br>
> - adapt your naming convention to what's already present in the code base<br>
> - use clang-format for formatting<br>
> - include what you use<br>
<br>
</span>Sounds good.<br>
<br>
<br>
Anyway, I asked out of curiosity (seeing largish number of warnings w/ VS2017).<br>
<div class="HOEnZb"><div class="h5"><br>
Best regards,<br>
--<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>
</div></div></blockquote></div><br></div>