From 98f920527e2cd4e8d6f0c9ddb7ea9ebbf1d7843b Mon Sep 17 00:00:00 2001
From: Criss <crisserpl2@gmail.com>
Date: Mon, 4 Sep 2017 22:13:50 +0200
Subject: [PATCH] Math: get rid of needless GenerateReverseSequence utility.

---
 src/Magnum/Math/BoolVector.h        | 7 -------
 src/Magnum/Math/RectangularMatrix.h | 4 ++--
 src/Magnum/Math/Vector.h            | 4 ++--
 3 files changed, 4 insertions(+), 11 deletions(-)

diff --git a/src/Magnum/Math/BoolVector.h b/src/Magnum/Math/BoolVector.h
index 63924981e..2b2bf4195 100644
--- a/src/Magnum/Math/BoolVector.h
+++ b/src/Magnum/Math/BoolVector.h
@@ -49,13 +49,6 @@ namespace Implementation {
     template<std::size_t ...sequence> struct GenerateSequence<0, sequence...> {
         typedef Sequence<sequence...> Type;
     };
-
-    template<std::size_t N, std::size_t ...sequence> struct GenerateReverseSequence:
-        GenerateReverseSequence<N-1, sequence..., N-1> {};
-
-    template<std::size_t ...sequence> struct GenerateReverseSequence<0, sequence...> {
-        typedef Sequence<sequence...> Type;
-    };
     #endif
 
     template<class T> constexpr T repeat(T value, std::size_t) { return value; }
diff --git a/src/Magnum/Math/RectangularMatrix.h b/src/Magnum/Math/RectangularMatrix.h
index 4881246ea..43231c165 100644
--- a/src/Magnum/Math/RectangularMatrix.h
+++ b/src/Magnum/Math/RectangularMatrix.h
@@ -400,7 +400,7 @@ template<std::size_t cols, std::size_t rows, class T> class RectangularMatrix {
          * @see @ref transposed(), @ref flippedRows(), @ref Vector::flipped()
          */
         constexpr RectangularMatrix<cols, rows, T> flippedCols() const {
-            return flippedColsInternal(typename Implementation::GenerateReverseSequence<cols>::Type{});
+            return flippedColsInternal(typename Implementation::GenerateSequence<cols>::Type{});
         }
 
         /**
@@ -454,7 +454,7 @@ template<std::size_t cols, std::size_t rows, class T> class RectangularMatrix {
         template<class U, std::size_t ...sequence> constexpr explicit RectangularMatrix(Implementation::Sequence<sequence...>, U) noexcept: _data{Vector<rows, T>((static_cast<void>(sequence), U{typename U::Init{}}))...} {}
 
         template<std::size_t ...sequence> constexpr RectangularMatrix<cols, rows, T> flippedColsInternal(Implementation::Sequence<sequence...>) const {
-            return {(*this)[sequence]...};
+            return {(*this)[cols - 1 - sequence]...};
         }
 
         template<std::size_t ...sequence> constexpr RectangularMatrix<cols, rows, T> flippedRowsInternal(Implementation::Sequence<sequence...>) const {
diff --git a/src/Magnum/Math/Vector.h b/src/Magnum/Math/Vector.h
index 0f17e44f5..c6ac19da8 100644
--- a/src/Magnum/Math/Vector.h
+++ b/src/Magnum/Math/Vector.h
@@ -537,7 +537,7 @@ template<std::size_t size, class T> class Vector {
          *      @ref RectangularMatrix::flippedRows()
          */
         constexpr Vector<size, T> flipped() const {
-            return flippedInternal(typename Implementation::GenerateReverseSequence<size>::Type{});
+            return flippedInternal(typename Implementation::GenerateSequence<size>::Type{});
         }
 
         /**
@@ -587,7 +587,7 @@ template<std::size_t size, class T> class Vector {
         }
 
         template<std::size_t ...sequence> constexpr Vector<size, T> flippedInternal(Implementation::Sequence<sequence...>) const {
-            return {(*this)[sequence]...};
+            return {(*this)[size - 1 - sequence]...};
         }
 
         T _data[size];