Изменения

* В c++ нет гарантий на то, как будет соотносится размер указателя и какого-нибудь интегрального типа. Поэтому для swap'а union'ов недостаточно обменть те члены, которые, как вам кажется, не меньше по размеру остальных. Наиболее правильным решением является создание именованного union'а, к которому можно применить std:swap.

* Конструктор копирования и оператор присваивания умеют создаваться автоматически. Если для класса нет необходимости писать специальную версию этих функций и нет каких-то других причин для этого, то лучше и не делать.