[PATCH] Replace OwningPtr<T> with std::unique_ptr<T>.
Chandler Carruth
chandlerc at gmail.com
Wed Mar 5 02:14:05 PST 2014
I've skimmed / spot checked this patch. Let me know when its updated with the two pulled-out bits? And if you have any other ideas for pulling out bits?
================
Comment at: include/llvm/Support/YAMLTraits.h:856-857
@@ -855,4 +855,4 @@
llvm::SourceMgr SrcMgr; // must be before Strm
- OwningPtr<llvm::yaml::Stream> Strm;
- OwningPtr<HNode> TopNode;
+ std::unique_ptr<llvm::yaml::Stream> Strm;
+ std::unique_ptr<HNode> TopNode;
llvm::error_code EC;
----------------
I would use clang-format on the diff for this (it has a diff-only processing mode). I usually use it via git-clang-format, but YMMV.
http://llvm-reviews.chandlerc.com/D2926
More information about the llvm-commits
mailing list