public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [wwwdocs] gcc-14: Mention that some warnings are now errors
@ 2024-04-13  9:50 Sebastian Huber
  2024-04-13 17:44 ` Eric Gallager
  2024-04-14 15:55 ` Gerald Pfeifer
  0 siblings, 2 replies; 5+ messages in thread
From: Sebastian Huber @ 2024-04-13  9:50 UTC (permalink / raw)
  To: GCC Patches; +Cc: Sam James

---
 htdocs/gcc-14/changes.html | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
index 8ac08e9a..a183fad8 100644
--- a/htdocs/gcc-14/changes.html
+++ b/htdocs/gcc-14/changes.html
@@ -231,6 +231,17 @@ a work-in-progress.</p>
   previous options <code>-std=c2x</code>, <code>-std=gnu2x</code>
   and <code>-Wc11-c2x-compat</code>, which are deprecated but remain
   supported.</li>
+  <li>The following warnings are now errors (see also
+    <a href="porting_to.html">Porting to GCC 14</a>):
+    <ul>
+      <li><code>-Werror=declaration-missing-parameter-type</code></li>
+      <li><code>-Werror=implicit-function-declaration</code></li>
+      <li><code>-Werror=implicit-int</code></li>
+      <li><code>-Werror=incompatible-pointer-types</code></li>
+      <li><code>-Werror=int-conversion</code></li>
+      <li><code>-Werror=return-mismatch</code></li>
+    </ul>
+  </li>
 </ul>
 
 <h3 id="cxx">C++</h3>
-- 
2.35.3


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [wwwdocs] gcc-14: Mention that some warnings are now errors
  2024-04-13  9:50 [wwwdocs] gcc-14: Mention that some warnings are now errors Sebastian Huber
@ 2024-04-13 17:44 ` Eric Gallager
  2024-04-13 17:53   ` Sam James
  2024-04-14 15:55 ` Gerald Pfeifer
  1 sibling, 1 reply; 5+ messages in thread
From: Eric Gallager @ 2024-04-13 17:44 UTC (permalink / raw)
  To: Sebastian Huber; +Cc: GCC Patches, Sam James

On Sat, Apr 13, 2024 at 5:51 AM Sebastian Huber
<sebastian.huber@embedded-brains.de> wrote:
>
> ---
>  htdocs/gcc-14/changes.html | 11 +++++++++++
>  1 file changed, 11 insertions(+)
>
> diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
> index 8ac08e9a..a183fad8 100644
> --- a/htdocs/gcc-14/changes.html
> +++ b/htdocs/gcc-14/changes.html
> @@ -231,6 +231,17 @@ a work-in-progress.</p>
>    previous options <code>-std=c2x</code>, <code>-std=gnu2x</code>
>    and <code>-Wc11-c2x-compat</code>, which are deprecated but remain
>    supported.</li>
> +  <li>The following warnings are now errors (see also
> +    <a href="porting_to.html">Porting to GCC 14</a>):
> +    <ul>
> +      <li><code>-Werror=declaration-missing-parameter-type</code></li>
> +      <li><code>-Werror=implicit-function-declaration</code></li>
> +      <li><code>-Werror=implicit-int</code></li>
> +      <li><code>-Werror=incompatible-pointer-types</code></li>
> +      <li><code>-Werror=int-conversion</code></li>
> +      <li><code>-Werror=return-mismatch</code></li>
> +    </ul>
> +  </li>
>  </ul>
>

I'd suggest adding the words "by default" to emphasize that they can
be turned back into warnings, and/or disabled. Perhaps others will
have other wording suggestions.

>  <h3 id="cxx">C++</h3>
> --
> 2.35.3
>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [wwwdocs] gcc-14: Mention that some warnings are now errors
  2024-04-13 17:44 ` Eric Gallager
@ 2024-04-13 17:53   ` Sam James
  0 siblings, 0 replies; 5+ messages in thread
From: Sam James @ 2024-04-13 17:53 UTC (permalink / raw)
  To: Eric Gallager; +Cc: Sebastian Huber, GCC Patches

[-- Attachment #1: Type: text/plain, Size: 1546 bytes --]

Eric Gallager <egall@gwmail.gwu.edu> writes:

> On Sat, Apr 13, 2024 at 5:51 AM Sebastian Huber
> <sebastian.huber@embedded-brains.de> wrote:
>>
>> ---
>>  htdocs/gcc-14/changes.html | 11 +++++++++++
>>  1 file changed, 11 insertions(+)
>>
>> diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html
>> index 8ac08e9a..a183fad8 100644
>> --- a/htdocs/gcc-14/changes.html
>> +++ b/htdocs/gcc-14/changes.html
>> @@ -231,6 +231,17 @@ a work-in-progress.</p>
>>    previous options <code>-std=c2x</code>, <code>-std=gnu2x</code>
>>    and <code>-Wc11-c2x-compat</code>, which are deprecated but remain
>>    supported.</li>
>> +  <li>The following warnings are now errors (see also
>> +    <a href="porting_to.html">Porting to GCC 14</a>):
>> +    <ul>
>> +      <li><code>-Werror=declaration-missing-parameter-type</code></li>
>> +      <li><code>-Werror=implicit-function-declaration</code></li>
>> +      <li><code>-Werror=implicit-int</code></li>
>> +      <li><code>-Werror=incompatible-pointer-types</code></li>
>> +      <li><code>-Werror=int-conversion</code></li>
>> +      <li><code>-Werror=return-mismatch</code></li>
>> +    </ul>
>> +  </li>
>>  </ul>
>>
>
> I'd suggest adding the words "by default" to emphasize that they can
> be turned back into warnings, and/or disabled. Perhaps others will
> have other wording suggestions.

Works for me with that added, although I can't formally approve.

It's fine without it though too.

>
>>  <h3 id="cxx">C++</h3>
>> --
>> 2.35.3
>>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 377 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [wwwdocs] gcc-14: Mention that some warnings are now errors
  2024-04-13  9:50 [wwwdocs] gcc-14: Mention that some warnings are now errors Sebastian Huber
  2024-04-13 17:44 ` Eric Gallager
@ 2024-04-14 15:55 ` Gerald Pfeifer
  2024-04-14 16:02   ` Jakub Jelinek
  1 sibling, 1 reply; 5+ messages in thread
From: Gerald Pfeifer @ 2024-04-14 15:55 UTC (permalink / raw)
  To: Sebastian Huber; +Cc: GCC Patches, Sam James

Hi Sebastian,

On Sat, 13 Apr 2024, Sebastian Huber wrote:
> +  <li>The following warnings are now errors (see also
> +    <a href="porting_to.html">Porting to GCC 14</a>):
> +    <ul>
> +      <li><code>-Werror=declaration-missing-parameter-type</code></li>
> +      <li><code>-Werror=implicit-function-declaration</code></li>
> +      <li><code>-Werror=implicit-int</code></li>
> +      <li><code>-Werror=incompatible-pointer-types</code></li>
> +      <li><code>-Werror=int-conversion</code></li>
> +      <li><code>-Werror=return-mismatch</code></li>
> +    </ul>

thanks for putting this together. Just a question, and maybe I'm confused:
these don't look like warnings to me?

Are you saying that what used to be -Wint-conversion is now an error by 
default and does not need to activated as such with any further options?

(That is, it appears as if those -Werror=... flags were used?)

And does this apply to all C versions/dialects?

Gerald

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [wwwdocs] gcc-14: Mention that some warnings are now errors
  2024-04-14 15:55 ` Gerald Pfeifer
@ 2024-04-14 16:02   ` Jakub Jelinek
  0 siblings, 0 replies; 5+ messages in thread
From: Jakub Jelinek @ 2024-04-14 16:02 UTC (permalink / raw)
  To: Gerald Pfeifer; +Cc: Sebastian Huber, GCC Patches, Sam James

On Sun, Apr 14, 2024 at 05:55:03PM +0200, Gerald Pfeifer wrote:
> Hi Sebastian,
> 
> On Sat, 13 Apr 2024, Sebastian Huber wrote:
> > +  <li>The following warnings are now errors (see also
> > +    <a href="porting_to.html">Porting to GCC 14</a>):
> > +    <ul>
> > +      <li><code>-Werror=declaration-missing-parameter-type</code></li>
> > +      <li><code>-Werror=implicit-function-declaration</code></li>
> > +      <li><code>-Werror=implicit-int</code></li>
> > +      <li><code>-Werror=incompatible-pointer-types</code></li>
> > +      <li><code>-Werror=int-conversion</code></li>
> > +      <li><code>-Werror=return-mismatch</code></li>
> > +    </ul>
> 
> thanks for putting this together. Just a question, and maybe I'm confused:
> these don't look like warnings to me?
> 
> Are you saying that what used to be -Wint-conversion is now an error by 
> default and does not need to activated as such with any further options?
> 
> (That is, it appears as if those -Werror=... flags were used?)
> 
> And does this apply to all C versions/dialects?

No, so I think the change is certainly not appropriate in that form.
The porting_to.html, in particular porting_to.html#warnings-as-errors
has detailed explanation on what actually changed etc., so I think we just
want one short sentence that certain C warnings are now errors and refer
to particular porting_to.html#warnings-as-errors for details.
Because the above shortcut is certainly not true.

	Jakub


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2024-04-14 16:02 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-04-13  9:50 [wwwdocs] gcc-14: Mention that some warnings are now errors Sebastian Huber
2024-04-13 17:44 ` Eric Gallager
2024-04-13 17:53   ` Sam James
2024-04-14 15:55 ` Gerald Pfeifer
2024-04-14 16:02   ` Jakub Jelinek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).