[GRASS-dev] No more compiler warnings

[Anna Petrášová]

That's great, thank you and others involved for working on this, it is an
impressive amount of work!

Anna

On Thu, Aug 26, 2021 at 4:22 PM Nicklas Larsson via grass-dev <
grass-dev at lists.osgeo.org> wrote:

> Hi all!
>
> 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.
>
> 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.
>
> 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.
>
> Cheers,
> Nicklas
>
>
> [1] https://github.com/OSGeo/grass/issues/1247
> _______________________________________________
> grass-dev mailing list
> grass-dev at lists.osgeo.org
> https://lists.osgeo.org/mailman/listinfo/grass-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/grass-dev/attachments/20210826/dee255d5/attachment.html>