<div dir="ltr"><div>That's great, thank you and others involved for working on this, it is an impressive amount of work!</div><div><br></div><div>Anna</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Aug 26, 2021 at 4:22 PM Nicklas Larsson via grass-dev <<a href="mailto:grass-dev@lists.osgeo.org">grass-dev@lists.osgeo.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi all!<br>
<br>
At the beginning of this year, compiling GRASS raised a very high number of compiler warnings. For example with GCC there were about 140 and with Clang about 300 warnings. This was reported with GitHub issue #1247 [1]. Since then these warnings has been addressed in a number of PRs. Today we have reached the point where there are no warnings at all (!) for either GCC and Clang (with default settings) on the main branch.<br>
<br>
To keep this level of warning frequency to zero, the compiler flag -Werror has been added to the GitHub "GCC C/C++ standards check" CI builds. This will treat a compiler warning as an error, and will cause the build to fail.<br>
<br>
For those who have a PR in the pipeline affecting C or C++ code, should consider making a rebase to main (or otherwise trigger a CI check) before merging.<br>
<br>
Cheers,<br>
Nicklas<br>
<br>
<br>
[1] <a href="https://github.com/OSGeo/grass/issues/1247" rel="noreferrer" target="_blank">https://github.com/OSGeo/grass/issues/1247</a><br>
_______________________________________________<br>
grass-dev mailing list<br>
<a href="mailto:grass-dev@lists.osgeo.org" target="_blank">grass-dev@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/grass-dev" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/grass-dev</a><br>
</blockquote></div></div>