I think it's good. Since the behavior is pretty wrapped up now I think unique_ptr would be a little overkill.