[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