It wasn't crashing but it's defenitely undefined behaviour.

Now instead of calling copy that calls the constructor we call the
constructor directly, which deals fine with a null "other" to be created
from

Also some const