Thread overview
[Issue 19685] Nested aggregate overlaps not detected
Feb 18, 2019
kinke@gmx.net
Feb 19, 2019
Dlang Bot
Feb 24, 2019
Dlang Bot
Feb 24, 2019
Dlang Bot
February 18, 2019
https://issues.dlang.org/show_bug.cgi?id=19685

kinke@gmx.net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |kinke@gmx.net

--- Comment #1 from kinke@gmx.net ---
https://run.dlang.io/is/p9cpXq:

```
struct S
{
    union
    {
        struct { int a = 123; }
        struct { int b = 456; }
    }
}

void main()
{
    S s;
    assert(s.b == 123);
}
```

Compiles successfully with 2.085.0-beta1, earlier versions rightfully complain:

Error: overlapping default initialization for field b and a Error: overlapping default initialization for field a and b

--
February 19, 2019
https://issues.dlang.org/show_bug.cgi?id=19685

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

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

--- Comment #2 from Dlang Bot <dlang-bot@dlang.rocks> ---
@Basile-z created dlang/dmd pull request #9373 "fix issue 19685 - Revert "aggregate, reduce the complexity of the loo…" fixing this issue:

- fix issue 19685 - Revert "aggregate, reduce the complexity of the loop checking the overlapped fields"

  This reverts commit 2ada2803f40cb59e4528dfadcf824d24e269d1f5.

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

--
February 24, 2019
https://issues.dlang.org/show_bug.cgi?id=19685

--- Comment #3 from Dlang Bot <dlang-bot@dlang.rocks> ---
@MartinNowak created dlang/dmd pull request #9387 "Merge remote-tracking branch 'upstream/stable' into merge_stable" fixing this issue:

- Merge pull request #9373 from Basile-z/issue-19685

  fix issue 19685 - Revert "aggregate, reduce the complexity of the loo…
  merged-on-behalf-of: Nicholas Wilson <thewilsonator@users.noreply.github.com>

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

--
February 24, 2019
https://issues.dlang.org/show_bug.cgi?id=19685

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 #9387 "Merge remote-tracking branch 'upstream/stable' into merge_stable" was merged into master:

- 1ed87aae3546f4d71df1d9e2e85c93d29d07a2b0 by The Dlang Bot:
  Merge pull request #9373 from Basile-z/issue-19685

  fix issue 19685 - Revert "aggregate, reduce the complexity of the loo…
  merged-on-behalf-of: Nicholas Wilson <thewilsonator@users.noreply.github.com>

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

--