https://issues.dlang.org/show_bug.cgi?id=22981

Basile-z <b2.temp@gmx.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Another forward reference   |premature enum type
                   |bug involving a string      |inference leads to spurious
                   |mixin                       |error message

--

https://issues.dlang.org/show_bug.cgi?id=22981

--- Comment #11 from Max Samukha <maxsamukha@gmail.com> ---
(In reply to Basile-z from comment #7)
> In `S` the non static member `E e` create a cycle but actually with a pass dedicated to "static members - only" , the internal enum `S.a` would not require `S` semantic to be complete.

Yes, I understand this. I meant UDAs are similar *in concept* to static members - they are things associated with the type's name but are not related to values of that type. Before D acquired proper UDAs, we used to emulate them with static fields and introspection.

--

https://issues.dlang.org/show_bug.cgi?id=22981

--- Comment #12 from Max Samukha <maxsamukha@gmail.com> ---
(In reply to Basile-z from comment #10)
> looks like enum being integral is assumed too early ;)
> 
> for example if you change the infered type of S.a
> 
> ```
> enum E { e1 = S.a }
> 
> struct S
> {
>     E e;
>     enum a = 123; // vs enum a = "string"
> }
> ```
> 
> no problems anymore

Seems to be unrelated to the original issue. In the original test case, the enum is integral - it is the member's name that is mixed-in.

--

https://issues.dlang.org/show_bug.cgi?id=22981

Basile-z <b2.temp@gmx.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|premature enum type         |Another forward reference
                   |inference leads to spurious |bug involving a string
                   |error message               |mixin

--- Comment #13 from Basile-z <b2.temp@gmx.com> ---
alright, I'll open another one ;)

--

https://issues.dlang.org/show_bug.cgi?id=22981

--- Comment #14 from Max Samukha <maxsamukha@gmail.com> ---
(In reply to Basile-z from comment #13)
> alright, I'll open another one ;)

Great, thanks!

--

https://issues.dlang.org/show_bug.cgi?id=22981

Iain Buclaw <ibuclaw@gdcproject.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P1                          |P3

--

https://issues.dlang.org/show_bug.cgi?id=22981

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

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

--- Comment #15 from Dlang Bot <dlang-bot@dlang.rocks> ---
@ibuclaw created dlang/dmd pull request #14826 "dmd.aggregate: Define importAll override for AggregateDeclaration" fixing this issue:

- fix Issue 22981 - Another forward reference bug involving a string mixin

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

--

https://issues.dlang.org/show_bug.cgi?id=22981

Iain Buclaw <ibuclaw@gdcproject.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           See Also|                            |https://issues.dlang.org/sh
                   |                            |ow_bug.cgi?id=23598

--