www.digitalmars.com         C & C++   DMDScript  

digitalmars.D.bugs - [Issue 10017] New: Can not assign to a Variant another Variant holding a bigger structure

reply d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=10017

           Summary: Can not assign to a Variant another Variant holding a
                    bigger structure
           Product: D
           Version: D2
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Phobos
        AssignedTo: nobody puremagic.com
        ReportedBy: octavian.cacina outlook.com
            Blocks: 9122


--- Comment #0 from Tavi Cacina <octavian.cacina outlook.com> 2013-05-02
10:31:15 PDT ---
---
import std.variant;

struct S
{
  int[9] s;
}

void main()
{
  Variant v1, v2; // maximum payload size 32 bytes
  v1 = S(); // OK, the payload is allocated with new
  v2 = v1; // AssertError: target must be non-null
}
---


This error affects also the max size of a std.concurrency message, see #9122.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
May 02 2013
next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=10017


Tavi Cacina <octavian.cacina outlook.com> changed:

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


--- Comment #1 from Tavi Cacina <octavian.cacina outlook.com> 2013-05-02
10:55:55 PDT ---
https://github.com/D-Programming-Language/phobos/pull/1281

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
May 02 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=10017



--- Comment #2 from github-bugzilla puremagic.com 2013-05-27 12:04:13 PDT ---
Commits pushed to master at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/2dba41d0323f518ad41300e5bd3cf0ff5665029d
fix issue 10017

https://github.com/D-Programming-Language/phobos/commit/5ddf3bc19a240b77255c9583aee3d35a9157cd8c
Merge pull request #1312 from dawgfoto/fix10017

fix issue 10017

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
May 27 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=10017


David Eagen <david eagen.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |david eagen.com


--- Comment #3 from David Eagen <david eagen.com> 2013-05-27 18:08:14 PDT ---
This fixes the problem for me on x86_64 Linux.

Thank you!

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
May 27 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=10017


Martin Nowak <code dawg.eu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |code dawg.eu
         Resolution|                            |FIXED


-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
May 28 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=10017


Martin Krejcirik <mk krej.cz> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mk krej.cz


--- Comment #4 from Martin Krejcirik <mk krej.cz> 2013-05-29 17:23:12 CEST ---
Still doesn't work on linux 32 bit.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
May 29 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=10017



--- Comment #5 from Martin Nowak <code dawg.eu> 2013-05-29 09:04:11 PDT ---
(In reply to comment #4)
 Still doesn't work on linux 32 bit.
Do you still get the "target must be non-null" error? Are you sure you are using dmd's HEAD version? -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: -------
May 29 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=10017



--- Comment #6 from Martin Nowak <code dawg.eu> 2013-05-29 09:55:40 PDT ---
This was only merged into the master branch but not into 2.063.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
May 29 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=10017



--- Comment #7 from Martin Krejcirik <mk krej.cz> 2013-05-30 00:47:28 CEST ---
You are right, sorry for the confusion.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
May 29 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=10017



--- Comment #8 from Martin Nowak <code dawg.eu> 2013-05-29 17:59:37 PDT ---
(In reply to comment #7)
 You are right, sorry for the confusion.
It should have been merged into 2.063 though because bug 9122 was marked as regression. -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: -------
May 29 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=10017



--- Comment #9 from github-bugzilla puremagic.com 2013-05-29 18:56:46 PDT ---
Commit pushed to 2.063 at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/9a7627776e7cffa999f30e5a7450fb7f044a12f7
Merge pull request #1312 from dawgfoto/fix10017

fix issue 10017

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
May 29 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=10017



--- Comment #10 from Kenji Hara <k.hara.pg gmail.com> 2013-05-29 18:57:19 PDT
---
(In reply to comment #8)
 (In reply to comment #7)
 You are right, sorry for the confusion.
It should have been merged into 2.063 though because bug 9122 was marked as regression.
Ok. I pushed the fix to 2.063 branch. -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: -------
May 29 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=10017



--- Comment #11 from David Eagen <david eagen.com> 2013-05-31 19:00:23 PDT ---
(In reply to comment #10)
 (In reply to comment #8)
 (In reply to comment #7)
 You are right, sorry for the confusion.
It should have been merged into 2.063 though because bug 9122 was marked as regression.
Ok. I pushed the fix to 2.063 branch.
Looks like the fix isn't in the 2.063 release binaries. At least not the Linux 64-bit ones. -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: -------
May 31 2013
prev sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=10017


sumitraja gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |sumitraja gmail.com


--- Comment #12 from sumitraja gmail.com 2013-07-29 19:47:30 PDT ---
(In reply to comment #11)
 (In reply to comment #10)
 (In reply to comment #8)
 (In reply to comment #7)
 You are right, sorry for the confusion.
It should have been merged into 2.063 though because bug 9122 was marked as regression.
Ok. I pushed the fix to 2.063 branch.
Looks like the fix isn't in the 2.063 release binaries. At least not the Linux 64-bit ones.
Doesn't seem to be in the Windows 32-bit 2.063 release binaries either. -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: -------
Jul 29 2013