Move constructors/operators should be marked noexcept to avoid the compiler picking the copy constructor instead (performance-noexcept-move-constructor)

src/Tag.cc

@@ -65,7 +65,7 @@ Tag& Tag::operator=(const Tag& other)
 	return *this;
 	}
 
-Tag& Tag::operator=(const Tag&& other)
+Tag& Tag::operator=(const Tag&& other) noexcept
 	{
 	if ( this != &other )
 		{

src/Tag.h

@@ -82,7 +82,7 @@ protected:
 	/**
 	 * Move assignment operator.
 	 */
-	Tag& operator=(const Tag&& other);
+	Tag& operator=(const Tag&& other) noexcept;
 
 	/**
 	 * Compares two tags for equality.

src/logging/Tag.cc

@@ -16,7 +16,7 @@ logging::Tag& logging::Tag::operator=(const logging::Tag& other)
 	return *this;
 	}
 
-logging::Tag& logging::Tag::operator=(const logging::Tag&& other)
+logging::Tag& logging::Tag::operator=(const logging::Tag&& other) noexcept
 	{
 	::Tag::operator=(other);
 	return *this;

src/logging/Tag.h

@@ -56,7 +56,7 @@ public:
 	/**
 	 * Move assignment operator.
 	 */
-	Tag& operator=(const Tag&& other);
+	Tag& operator=(const Tag&& other) noexcept;
 
 	/**
 	 * Compares two tags for equality.

src/probabilistic/CardinalityCounter.cc

@@ -76,7 +76,7 @@ CardinalityCounter::CardinalityCounter(CardinalityCounter& other)
 	p = other.p;
 	}
 
-CardinalityCounter::CardinalityCounter(CardinalityCounter&& o)
+CardinalityCounter::CardinalityCounter(CardinalityCounter&& o) noexcept
 	{
 	V = o.V;
 	alpha_m = o.alpha_m;

src/probabilistic/CardinalityCounter.h

@@ -43,7 +43,7 @@ public:
 	/**
 	 * Move-Constructor
 	 */
-	CardinalityCounter(CardinalityCounter&& o);
+	CardinalityCounter(CardinalityCounter&& o) noexcept;
 
 	/**
 	 * Constructor for a known number of buckets.