r215853 - Convert a few ownership comments with std::unique_ptr.
Rafael EspĂndola
rafael.espindola at gmail.com
Sun Aug 17 16:37:20 PDT 2014
> Directly constructing temporary smart pointers is an eye-brow raising
> thing (though not applicable to LLVM - it's not exception safe). I'd
> probably be inclined to take ownership on the line before (with the
> result of getMemBuffer) into a named unique_ptr, then move into the
> call got getVFSFromYAML instead.
Fixed in r215865.
> (& perhaps one day we'll have MemoryBuffer's factories return by
> unique_ptr... -
That is where I am going :-)
Cheers,
Rafael
More information about the cfe-commits
mailing list