Consider using unique_ptr or shared_ptr instead of raw pointer access

[samhatchett]

For owned or transferred pointers, we should be using std::unique_ptr for RAII and memory management. Especially in context of exception handling and potentially returning new allocations to client code which may not assign and own the pointer. For example, all factory methods leak by default.