[clang-tools-extra] [NFC] Complete proper copying and resource cleanup in classes. (PR #118655)

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 5 05:13:49 PST 2024


================
@@ -81,6 +81,9 @@ class ClangTidyContext {
 
   ~ClangTidyContext();
 
+  ClangTidyContext(const ClangTidyContext &) = default;
+  ClangTidyContext &operator=(const ClangTidyContext &) = default;
----------------
AaronBallman wrote:

I think these would make more sense as `= delete;` so that we prevent accidental copies of the context object. (We want this passed around by reference or pointer only.)

https://github.com/llvm/llvm-project/pull/118655


More information about the cfe-commits mailing list