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

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

src/Utils/Mempool.hpp

@@ -46,11 +46,7 @@ class MempoolStats {
 	void statDelBlock() { --m_BlockCount; }
 public:
 	MempoolStats() = default;
-	MempoolStats(MempoolStats &&other) noexcept
-	{
-		/* Call move assignment operator. */
-		*this = std::forward<MempoolStats>(other);
-	}
+	MempoolStats(MempoolStats &&other) noexcept { *this = std::move<MempoolStats>(other); }
 	MempoolStats &operator=(MempoolStats &&other) noexcept
 	{
 		std::swap(m_SlabCount, other.m_SlabCount);
@@ -134,17 +130,13 @@ class MempoolInstance : public MempoolStats<ENABLE_STATS> {
 	MempoolInstance() = default;
 	MempoolInstance(const MempoolInstance &other) = delete;
 	MempoolInstance &operator=(const MempoolInstance &other) = delete;
-	MempoolInstance(MempoolInstance &&other) noexcept
-	{
-		/* Call move assignment operator. */
-		*this = std::forward<MempoolInstance>(other);
-	}
+	MempoolInstance(MempoolInstance &&other) noexcept { *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);