AK: Define conversion from Span<T> to Span<const T> correctly.

I accidently wrote `Span<RemoveConst<T>>` when I meant
`Span<RemoveConst<T>::Type>`.

Changing that wouldn't be enough though, this constructor can only be
defined if T is not const, otherwise it would redefine the copy
constructor.  This can be avoided by overloading the cast operator.

1 files changed, 5 insertions, 5 deletions

diff --git a/AK/Span.h b/AK/Span.h
index bf46986cd0..e974ce8807 100644
--- a/AK/Span.h
+++ b/AK/Span.h
@@ -52,11 +52,6 @@ public:
         , m_size(other.m_size)
     {
     }
-    ALWAYS_INLINE Span(const Span<RemoveConst<T>>& other)
-        : m_values(other.m_values)
-        , m_size(other.m_size)
-    {
-    }
 
     ALWAYS_INLINE const T* data() const { return m_values; }
     ALWAYS_INLINE T* data() { return m_values; }
@@ -115,6 +110,11 @@ public:
         m_values = other.m_values;
     }
 
+    ALWAYS_INLINE operator Span<const T>() const
+    {
+        return { data(), size() };
+    }
+
 protected:
     T* m_values { nullptr };
     size_t m_size { 0 };