Do warnings as errors always on Linux, not just on gcc 11#2185
Do warnings as errors always on Linux, not just on gcc 11#2185daleglass wants to merge 1 commit intooverte-org:masterfrom
Conversation
|
Maybe we could do it only for GH actions builds? |
|
Or maybe we could do it up to a given, known version so that it doesn't break for end users before we have chance to fix it? |
|
|
|
I'd also like having it only on CI |
My approach is to fix warnings first, debug later if still broken. Warnings are a good sign something isn't quite right.
CI has an old compiler though, it catches less stuff. I want the pickier the better, because the code is crashing as it is, so obviously something isn't quite right still.
Alright, we can bump it to a GCC 15.2.1 because I tested with that. After PR #2184 it'll be clean. |
I think doing it this way is a good idea. Then when GCC 16 comes out with new warnings we will have time to fix them. |
I don't see the reason to restrict warnings as errors to GCC 11 specifically.
New compilers catch more mistakes. The point of warnings as errors is not to add more bad stuff to start with, and to fix errors that were hiding until the compiler got better at catching them.
After PR #2184 we should be back to a fully clean build on GCC 15.2.1.