public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug c/96842] New: enhancement: copy clang Wheader-guard
@ 2020-08-29 13:06 dcb314 at hotmail dot com
  2020-08-30 18:12 ` [Bug c/96842] " egallager at gcc dot gnu.org
  2021-07-23 18:07 ` [Bug preprocessor/96842] " pinskia at gcc dot gnu.org
  0 siblings, 2 replies; 3+ messages in thread
From: dcb314 at hotmail dot com @ 2020-08-29 13:06 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96842

            Bug ID: 96842
           Summary: enhancement: copy clang Wheader-guard
           Product: gcc
           Version: unknown
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c
          Assignee: unassigned at gcc dot gnu.org
          Reporter: dcb314 at hotmail dot com
  Target Milestone: ---

clang checks header guard macros and gcc doesn't.

So code in <stdio.h> like

#ifndef _STDIO_H
#define _STDIO_H_        1

where the define is different to the previous test, gets caught.

For example:

AIPlayerFactory.h:19:2: warning: 'AMEOBAX_AI_PLAYER_FACTORY_H' is used as a
header guard here, followed by #define of a different macro [-Wheader-guard]

There are about 500 examples of this in Fedora Linux distribution.

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

* [Bug c/96842] enhancement: copy clang Wheader-guard
  2020-08-29 13:06 [Bug c/96842] New: enhancement: copy clang Wheader-guard dcb314 at hotmail dot com
@ 2020-08-30 18:12 ` egallager at gcc dot gnu.org
  2021-07-23 18:07 ` [Bug preprocessor/96842] " pinskia at gcc dot gnu.org
  1 sibling, 0 replies; 3+ messages in thread
From: egallager at gcc dot gnu.org @ 2020-08-30 18:12 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96842

Eric Gallager <egallager at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |diagnostic
             Blocks|                            |87403
   Last reconfirmed|                            |2020-08-30
                 CC|                            |egallager at gcc dot gnu.org
             Status|UNCONFIRMED                 |NEW
     Ever confirmed|0                           |1

--- Comment #1 from Eric Gallager <egallager at gcc dot gnu.org> ---
confirmed that this warning would be nice to have.


Referenced Bugs:

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87403
[Bug 87403] [Meta-bug] Issues that suggest a new warning

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

* [Bug preprocessor/96842] enhancement: copy clang Wheader-guard
  2020-08-29 13:06 [Bug c/96842] New: enhancement: copy clang Wheader-guard dcb314 at hotmail dot com
  2020-08-30 18:12 ` [Bug c/96842] " egallager at gcc dot gnu.org
@ 2021-07-23 18:07 ` pinskia at gcc dot gnu.org
  1 sibling, 0 replies; 3+ messages in thread
From: pinskia at gcc dot gnu.org @ 2021-07-23 18:07 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96842

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |enhancement
          Component|c                           |preprocessor

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

end of thread, other threads:[~2021-07-23 18:07 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-29 13:06 [Bug c/96842] New: enhancement: copy clang Wheader-guard dcb314 at hotmail dot com
2020-08-30 18:12 ` [Bug c/96842] " egallager at gcc dot gnu.org
2021-07-23 18:07 ` [Bug preprocessor/96842] " pinskia at gcc dot gnu.org

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).