On Wednesday, 11 July 2018 at 20:10:17 UTC, Meta wrote:
I hate to say I told you so, but...
https://github.com/dlang/phobos/pull/5855#issuecomment-345783238
Just joking, of course =)
Nullable has needed to be completely overhauled for a long time
because it was only really designed with POD types in mind.
Good news is the union hack seems to be working actually (
https://github.com/dlang/phobos/pull/6619 ). See the Turducken
post in General (
https://forum.dlang.org/thread/ekbxqxhnttihkoszz...@forum.dlang.org ); I didn't use the full Turducken in the PR for Phobos because I don't think Nullable is specced to be able to handle reassigning const data anyway, so the union assign in moveEmplace seems sufficient.
