https://issues.dlang.org/show_bug.cgi?id=23598
Max Samukha changed:
What|Removed |Added
Severity|blocker |normal
--- Comment #9 from Max Samukha ---
https://issues.dlang.org/show_bug.cgi?id=23598
Iain Buclaw changed:
What|Removed |Added
See Also||https://issues.dlang.org/sh
https://issues.dlang.org/show_bug.cgi?id=23598
--- Comment #8 from Walter Bright ---
Looking at expandMembers, perhaps this could be done in two passes - once for
the top level conditional compilation, then check oneMember, then again for the
rest.
--
https://issues.dlang.org/show_bug.cgi?id=23598
--- Comment #7 from Walter Bright ---
Perhaps I can make this work by noticing if the `static if` member is the only
member of the template, so it can be eagerly evaluated?
--
https://issues.dlang.org/show_bug.cgi?id=23598
--- Comment #6 from Walter Bright ---
What seems to be happening is the first time it goes through `template sort`,
it cannot set `aliasdecl` because the two `alias sort` assignments are hidden
behind conditional compilation.
Hence `aliasdecl` is
https://issues.dlang.org/show_bug.cgi?id=23598
--- Comment #5 from Walter Bright ---
I suspect that when the sort template is instantiated, it is too eager to
evaluate its arguments. It should just go as far as seeing String is a struct,
and not attempt to semantic its contents.
--
https://issues.dlang.org/show_bug.cgi?id=23598
Walter Bright changed:
What|Removed |Added
CC||bugzi...@digitalmars.com
--- Comment #4
---
@ibuclaw created dlang/dmd pull request #14826 "dmd.aggregate: Define importAll
override for AggregateDeclaration" fixing this issue:
- fix Issue 23598 - Another nasty forward reference bug
https://github.com/dlang/dmd/pull/14826
--
https://issues.dlang.org/show_bug.cgi?id=23598
--- Comment #2 from Max Samukha ---
(In reply to Iain Buclaw from comment #1)
> Related to issue 23595, and a WIP patch I have in the works fixes this test
> case too.
Great! Thanks!
--
https://issues.dlang.org/show_bug.cgi?id=23598
Iain Buclaw changed:
What|Removed |Added
CC||ibuc...@gdcproject.org
See Also|
10 matches
Mail list logo