utils: std::forward related fixes

std::forward were replace std::move for fixed type context where
it does not make sense. Some redundant comment were removed as they
declare obvious actions under them.

Author: pmelkozer

src/Buffer/Buffer.hpp

@@ -276,8 +276,7 @@ class Buffer {
 	Buffer& operator = (const Buffer& buf) = delete;
 	Buffer(Buffer &&other) noexcept
 	{
-		/* Call move assignment operator. */
-		*this = std::forward<Buffer>(other);
+		*this = std::move(other);
 	}
 	Buffer &operator=(Buffer &&other) noexcept
 	{

src/Utils/Mempool.hpp

@@ -48,8 +48,7 @@ class MempoolStats {
 	MempoolStats() = default;
 	MempoolStats(MempoolStats &&other) noexcept
 	{
-		/* Call move assignment operator. */
-		*this = std::forward<MempoolStats>(other);
+		*this = std::move<MempoolStats>(other);
 	}
 	MempoolStats &operator=(MempoolStats &&other) noexcept
 	{
@@ -136,15 +135,14 @@ class MempoolInstance : public MempoolStats<ENABLE_STATS> {
 	MempoolInstance &operator=(const MempoolInstance &other) = delete;
 	MempoolInstance(MempoolInstance &&other) noexcept
 	{
-		/* Call move assignment operator. */
-		*this = std::forward<MempoolInstance>(other);
+		*this = std::move(other);
 	}
 	MempoolInstance &operator=(MempoolInstance &&other) noexcept
 	{
 		if (this == &other)
 			return *this;
 		/* Move base class. */
-		MempoolStats<ENABLE_STATS>::operator=(std::forward<MempoolInstance>(other));
+		MempoolStats<ENABLE_STATS>::operator=(std::move(other));
 		std::swap(m_SlabList, other.m_SlabList);
 		std::swap(m_FreeList, other.m_FreeList);
 		std::swap(m_SlabDataBeg, other.m_SlabDataBeg);