TemporaryFile - only use move semantic

justsmth · Nov 13, 2024 · 010d0c1 · 010d0c1
1 parent 096f73a
commit 010d0c1
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/crypto/test/file_util.h b/crypto/test/file_util.h
@@ -85,7 +85,7 @@ class TemporaryFile {
   TemporaryFile() = default;
   ~TemporaryFile();
 
-  TemporaryFile(TemporaryFile &other) { *this = std::move(other); }
+  TemporaryFile(TemporaryFile&& other) noexcept { *this = std::move(other); }
   TemporaryFile& operator=(TemporaryFile&&other) {
     // Ensure |path_| is empty so it doesn't try to delete the File.
     auto old_other_path = other.path_;
@@ -94,6 +94,9 @@ class TemporaryFile {
     return *this;
   }
 
+  TemporaryFile(const TemporaryFile&) = delete;
+  TemporaryFile& operator=(const TemporaryFile&) = delete;
+
   // Init initializes the temporary file with the specified content. It returns
   // true on success and false on error. On error, callers should call
   // |IgnoreTempFileErrors| to determine whether to ignore the error.