[clang] 68ef916 - Delete PrintingPolicy's copy constructor/operator.
Erich Keane via cfe-commits
cfe-commits at lists.llvm.org
Fri Jul 23 11:23:59 PDT 2021
Author: Erich Keane
Date: 2021-07-23T11:22:52-07:00
New Revision: 68ef916659b9cd4127276f9502fc2870ca6cfdea
URL: https://github.com/llvm/llvm-project/commit/68ef916659b9cd4127276f9502fc2870ca6cfdea
DIFF: https://github.com/llvm/llvm-project/commit/68ef916659b9cd4127276f9502fc2870ca6cfdea.diff
LOG: Delete PrintingPolicy's copy constructor/operator.
This type is 'fat' now thanks to the callbacks, so it should never be
copied as far as I know. Delete the copy operations so that we don't do
so accidentially.
Added:
Modified:
clang/include/clang/AST/PrettyPrinter.h
Removed:
################################################################################
diff --git a/clang/include/clang/AST/PrettyPrinter.h b/clang/include/clang/AST/PrettyPrinter.h
index 3baf2b2ba94d6..3b09c604b47b0 100644
--- a/clang/include/clang/AST/PrettyPrinter.h
+++ b/clang/include/clang/AST/PrettyPrinter.h
@@ -76,6 +76,9 @@ struct PrintingPolicy {
SuppressImplicitBase(false), FullyQualifiedName(false),
PrintCanonicalTypes(false), PrintInjectedClassNameWithArguments(true) {}
+ PrintingPolicy(const PrintingPolicy&) = delete;
+ PrintingPolicy &operator=(const PrintingPolicy &) = delete;
+
/// Adjust this printing policy for cases where it's known that we're
/// printing C++ code (for instance, if AST dumping reaches a C++-only
/// construct). This should not be used if a real LangOptions object is
More information about the cfe-commits
mailing list