yeah that's a no op assignment.  So node1 = node2 doesn't do anything.  Not sure a memcpy(this, &copy, sizeof(Node)) is legal, probably better to just assign all the fields.  You don't need a deep copy though, at least not for this purpose, you could just copy the pointers, since the old copy is going to be dead in a nanosecond.  

Edit:

whoops, I realized that was bad advice.  If the destructor of frees an allocated resource, you are going to be dereferencing an invalid pointer (probably, unless that memory gets reused) when you later try to use the copy.  Sorry about that, hope it didn't cause any problems.  

yeah that's a no op assignment.  So node1 = node2 doesn't do anything.  Not sure a memcpy(this, &copy, sizeof(Node)) is legal, probably better to just assign all the fields.  You don't need a deep copy though, at least not for this purpose, you could just copy the pointers, since the old copy is going to be dead in a nanosecond.  

yeah that's a no op assignment.  So node1 = node2 doesn't do anything.  Not sure a memcpy(this, &copy, sizeof(Node)) is legal, probably better to just assign all the fields.  You don't need a deep copy though, at least not here for this, you could just copy the pointers, since the old copy is going to be dead in a nanosecond.