From bc70a93bba6c6f63486df55295145d043604a228 Mon Sep 17 00:00:00 2001
From: Neil MacIntosh <neilmac@microsoft.com>
Date: Mon, 12 Sep 2016 19:06:45 -0700
Subject: [PATCH] Removed unnecessary reinterpret_cast<> from span
 implementation.

---
 gsl/span | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gsl/span b/gsl/span
index cce8253..d2e16dc 100644
--- a/gsl/span
+++ b/gsl/span
@@ -420,7 +420,7 @@ public:
             details::is_allowed_extent_conversion<OtherExtent, Extent>::value &&
             details::is_allowed_element_type_conversion<OtherElementType, element_type>::value>>
     constexpr span(const span<OtherElementType, OtherExtent>& other)
-        : storage_(reinterpret_cast<pointer>(other.data()),
+        : storage_(static_cast<pointer>(other.data()),
                    details::extent_type<OtherExtent>(other.size()))
     {
     }
@@ -431,7 +431,7 @@ public:
             details::is_allowed_extent_conversion<OtherExtent, Extent>::value &&
             details::is_allowed_element_type_conversion<OtherElementType, element_type>::value>>
     constexpr span(span<OtherElementType, OtherExtent>&& other)
-        : storage_(reinterpret_cast<pointer>(other.data()),
+        : storage_(static_cast<pointer>(other.data()),
                    details::extent_type<OtherExtent>(other.size()))
     {
     }