this copy constructor should copy the vector, this uses the same memory which is not what you want there. In fact, you can just avoid copy constructor for now