Thread overview
[Issue 23143] ImportC: forward enum declarations need to be supported
May 28, 2022
Max Samukha
Jul 07, 2022
Walter Bright
Jul 09, 2022
Walter Bright
Jul 09, 2022
Dlang Bot
Jul 12, 2022
Dlang Bot
May 28, 2022
https://issues.dlang.org/show_bug.cgi?id=23143

Max Samukha <maxsamukha@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|ImportC: forward enum       |ImportC: forward enum
                   |declarations result in      |declarations need to be
                   |uninformative errors        |supported
           Severity|enhancement                 |normal

--- Comment #1 from Max Samukha <maxsamukha@gmail.com> ---
I'd rather qualify this as a bug. If ImportC can't handle system headers, it's a failure. This issue prevents Windows DDK's wdm.h from being imported.

--
July 07, 2022
https://issues.dlang.org/show_bug.cgi?id=23143

Walter Bright <bugzilla@digitalmars.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bugzilla@digitalmars.com

--- Comment #2 from Walter Bright <bugzilla@digitalmars.com> ---
It seems both gcc and clang support it unless the -Wpedantic is supplied:

fix23143> gcc -c test.c -std=c11 -Wpedantic
test.c:3:6: warning: ISO C forbids forward references to âenumâ types
[-Wpedantic]
 enum _POOL_TYPE;
      ^
fix23143> clang -c test.c -std=c11 -Wpedantic
test.c:3:6: warning: ISO C forbids forward references to 'enum' types
[-Wpedantic]
enum _POOL_TYPE;
     ^
1 warning generated.

So I suppose we are doomed to support it.

--
July 09, 2022
https://issues.dlang.org/show_bug.cgi?id=23143

Walter Bright <bugzilla@digitalmars.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Hardware|x86                         |All
                 OS|Windows                     |All
           Severity|normal                      |enhancement

--
July 09, 2022
https://issues.dlang.org/show_bug.cgi?id=23143

Dlang Bot <dlang-bot@dlang.rocks> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |pull

--- Comment #3 from Dlang Bot <dlang-bot@dlang.rocks> ---
@WalterBright created dlang/dmd pull request #14277 "fix Issue 23143 - ImportC: forward enum declarations need to be suppo…" fixing this issue:

- fix Issue 23143 - ImportC: forward enum declarations need to be supported

https://github.com/dlang/dmd/pull/14277

--
July 12, 2022
https://issues.dlang.org/show_bug.cgi?id=23143

Dlang Bot <dlang-bot@dlang.rocks> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #4 from Dlang Bot <dlang-bot@dlang.rocks> ---
dlang/dmd pull request #14277 "fix Issue 23143 - ImportC: forward enum declarations need to be suppo…" was merged into master:

- 65b5bbf146ec7dbd75603a9a9a0627f128cc8c7e by Walter Bright:
  fix Issue 23143 - ImportC: forward enum declarations need to be supported

https://github.com/dlang/dmd/pull/14277

--