Use l-value reference specifier for assignment operators to prevent m…

…isuse
danmar · Aug 6, 2024 · f1c2344 · f1c2344
1 parent b62cb22
commit f1c2344
Show file tree

Hide file tree

Showing 7 changed files with 6 additions and 7 deletions.
diff --git a/cppcheckpremium-suppressions b/cppcheckpremium-suppressions
@@ -68,7 +68,6 @@ premium-misra-cpp-2023-13.1.2
 premium-misra-cpp-2023-13.3.1
 premium-misra-cpp-2023-13.3.2
 premium-misra-cpp-2023-13.3.3
-premium-misra-cpp-2023-15.0.2
 premium-misra-cpp-2023-15.1.2
 premium-misra-cpp-2023-15.1.3
 premium-misra-cpp-2023-18.1.1

diff --git a/lib/importproject.h b/lib/importproject.h
@@ -74,7 +74,7 @@ class CPPCHECKLIB WARN_UNUSED ImportProject {
     ImportProject() = default;
     virtual ~ImportProject() = default;
     ImportProject(const ImportProject&) = default;
-    ImportProject& operator=(const ImportProject&) = default;
+    ImportProject& operator=(const ImportProject&) & = default;
 
     void selectOneVsConfig(Platform::Type platform);
     void selectVsConfigurations(Platform::Type platform, const std::vector<std::string> &configurations);

diff --git a/lib/library.cpp b/lib/library.cpp
@@ -153,7 +153,7 @@ Library::Library(const Library& other)
     : mData(new LibraryData(*other.mData))
 {}
 
-Library& Library::operator=(const Library& other)
+Library& Library::operator=(const Library& other) &
 {
     mData.reset(new LibraryData(*other.mData));
     return *this;

diff --git a/lib/library.h b/lib/library.h
@@ -57,7 +57,7 @@ class CPPCHECKLIB Library {
     ~Library();
 
     Library(const Library& other);
-    Library& operator=(const Library& other);
+    Library& operator=(const Library& other) &;
 
     enum class ErrorCode : std::uint8_t {
         OK,

diff --git a/lib/symboldatabase.cpp b/lib/symboldatabase.cpp
@@ -2252,7 +2252,7 @@ Variable::~Variable()
     delete mValueType;
 }
 
-Variable& Variable::operator=(const Variable &var)
+Variable& Variable::operator=(const Variable &var) &
 {
     if (this == &var)
         return *this;

diff --git a/lib/symboldatabase.h b/lib/symboldatabase.h
@@ -236,7 +236,7 @@ class CPPCHECKLIB Variable {
 
     ~Variable();
 
-    Variable &operator=(const Variable &var);
+    Variable &operator=(const Variable &var) &;
 
     /**
      * Get name token.

diff --git a/lib/valueptr.h b/lib/valueptr.h
@@ -81,7 +81,7 @@ class CPPCHECKLIB ValuePtr {
         swap(mClone, rhs.mClone);
     }
 
-    ValuePtr<T>& operator=(ValuePtr rhs) {
+    ValuePtr<T>& operator=(ValuePtr rhs) & {
         swap(rhs);
         return *this;
     }