Looks like the entire destructor should just go. Doesn't do anything now and isn't virtual. The default implicit constructor is good enough.