Thanks, lgtm! ================ Comment at: include/clang/AST/ASTContext.h:2360-2361 @@ -2360,1 +2359,4 @@ +/// explicitly freed, as ASTContext will free all of this memory when it gets +/// destroyed. Please note that you cannot use delete on the pointer; it must +/// be deallocated using an explicit destructor call followed by /// @c Context.Deallocate(Ptr). ---------------- Context.Deallocate() is a noop, so I think this last sentence is incorrect. I'd delete it or shorten to "Note that you cannot use delete on the pointer."
================ Comment at: include/clang/AST/ASTContext.h:2399-2400 @@ -2397,1 +2398,4 @@ +/// explicitly freed, as ASTContext will free all of this memory when it gets +/// destroyed. Please note that you cannot use delete on the pointer; it must +/// be deallocated using an explicit destructor call followed by /// @c Context.Deallocate(Ptr). ---------------- Ditto http://reviews.llvm.org/D5392 _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
