Thread overview
[Issue 8704] Invalid nested struct constructions should be detected
Apr 11, 2014
Kenji Hara
Apr 11, 2014
Kenji Hara
April 11, 2014
https://issues.dlang.org/show_bug.cgi?id=8704

Kenji Hara <k.hara.pg@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|WORKSFORME                  |---

--- Comment #4 from Kenji Hara <k.hara.pg@gmail.com> ---
(In reply to yebblies from comment #3)
> The pull request wasn't merged, but both cases now error.

The opening test case still does not cause front-end errors (no error occur
with -o- switch).

And T t; should also be rejected if T is nested struct. Updated test case is:

void check(T)()
{
    // nested struct constructions outside valid frame scope
    T t0;       // new!
    T t1 = T();
    T t2 = T(1);
}

void main()
{
    struct S
    {
        int n;
        void foo(){}
    }
    check!S();
}

(In reply to Kenji Hara from comment #1)
> https://github.com/D-Programming-Language/dmd/pull/1133

I reopened the pull request to fix this issue perfectly.

--
April 11, 2014
https://issues.dlang.org/show_bug.cgi?id=8704

Kenji Hara <k.hara.pg@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |kennytm@gmail.com

--- Comment #5 from Kenji Hara <k.hara.pg@gmail.com> ---
*** Issue 5902 has been marked as a duplicate of this issue. ***

--
May 14, 2014
https://issues.dlang.org/show_bug.cgi?id=8704

github-bugzilla@puremagic.com changed:

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

--
May 14, 2014
https://issues.dlang.org/show_bug.cgi?id=8704

--- Comment #6 from github-bugzilla@puremagic.com ---
Commits pushed to master at https://github.com/D-Programming-Language/dmd

https://github.com/D-Programming-Language/dmd/commit/da0be7592a903af33da5d65792e38e265f560bb3 fix Issue 8704 - Invalid nested struct constructions should be detected

https://github.com/D-Programming-Language/dmd/commit/3232f56a34900bd4228c22134b637464bb37ed0c Merge pull request #1133 from 9rnsr/fix8704

Issue 8704 - Invalid nested struct constructions should be detected

--