I think it's good enough. It would have prevented the not-emptying the RHS bug and eliminate the need for = nullptr at the end. Since T is a trivial type, no custom deleter should be needed / that bit should just work right I think. Would you rather I merge it or would you like to try unique_ptr / update the PR?