Merge pull request InsightSoftwareConsortium#5963 from hjmjohnson/revert-forward-move

Revert "BUG: Use std::forward instead of std::move on forwarding references"

Author: hjmjohnson

Modules/Core/Common/test/itkArray2DGTest.cxx

@@ -70,7 +70,7 @@ TEST(Array2D, MoveConstruct)
     const auto * const * const originalDataArray{ original.data_array() };
     const unsigned int         originalSize{ original.size() };
 
-    const auto moveConstructed = std::forward<decltype(original)>(original);
+    const auto moveConstructed = std::move(original);
 
     // After the "move", the move-constructed object has retrieved the original data.
     EXPECT_EQ(moveConstructed.data_array(), originalDataArray);

Modules/Core/Common/test/itkRangeGTestUtilities.h

@@ -94,7 +94,7 @@ class RangeGTestUtilities
   ExpectMoveConstructedRangeHasSameIteratorsAsOriginalBeforeMove(TRange && originalRange)
   {
     const TRange originalRangeBeforeMove = originalRange;
-    TRange       moveConstructedRange(std::forward<TRange>(originalRange));
+    TRange       moveConstructedRange(std::move(originalRange));
 
     ExpectRangesHaveEqualBeginAndEnd(moveConstructedRange, originalRangeBeforeMove);
   }
@@ -107,7 +107,7 @@ class RangeGTestUtilities
     const TRange originalRangeBeforeMove = originalRange;
 
     TRange moveAssignedRange;
-    moveAssignedRange = std::forward<TRange>(originalRange);
+    moveAssignedRange = std::move(originalRange);
 
     ExpectRangesHaveEqualBeginAndEnd(moveAssignedRange, originalRangeBeforeMove);
   }