[PATCH] [C++11] Remove the remaining uses of OwningPtr.

Ahmed Charles acharles at outlook.com
Sat Mar 8 22:40:52 PST 2014



================
Comment at: examples/clang-interpreter/main.cpp:19
@@ -18,3 +18,2 @@
 #include "clang/Frontend/TextDiagnosticPrinter.h"
-#include "llvm/ADT/OwningPtr.h"
 #include "llvm/ADT/SmallString.h"
----------------
David Blaikie wrote:
> It'd be nice to include <memory> whenever it's needed rather than relying on indirect inclusion.
I could replace each include of OwningPtr with <memory> but that wouldn't actually make your desire come true, it would only reduce the likelihood that I introduce transient build breaks with this change that are easy to fix.

Unless there's a tool I can run to verify this, it's just going to result in me sprinkling the code with lots of <memory> 'love'. :)


http://llvm-reviews.chandlerc.com/D3015



More information about the cfe-commits mailing list