virtuals removed and replaced with override where necessary on the rest of the code base, clang-tidy to the rescue.
The uninitialized variable 'box' is assigned to itself. Judging from the fact that this is a copy constructor, I think that it is necessary to store in the 'box' the value of 'cv.box'