Actually copying a structure this size is unnecessarily expensive given how it's actually used. Remove copy-ability and replace with move semantics. Remove swap() method as well, replacing the one usage with a move since it's a local variable that's about to die anyway.