commit 05a449a795258e34d2451b9390b6021c7a4a2f28
author Rafael Cintron <rafael.cintron@microsoft.com> Wed Jun 20 18:08:04 2018 -0700
committer Commit Bot <commit-bot@chromium.org> Thu Jun 21 18:42:27 2018 +0000
tree 3b3f65df1e18604aa972e91c34b9ac1233de39bb
parent 5fd3693e1137cb651dc3e7f5d088a621e4efec1f

Replace reinterpret_cast with safer or no cast

When casting types to one another in C++, the weaker the cast,
the better.

This change replaces instances of reinterpret_cast with static_cast
or no cast where it safe and correct to do so.

BUG=angleproject:2683

Change-Id: I99c9033614a65282ae1d78cf0f4b80fabd75877a
Reviewed-on: https://chromium-review.googlesource.com/1109396
Commit-Queue: Rafael Cintron <rafael.cintron@microsoft.com>
Reviewed-by: Jamie Madill <jmadill@chromium.org>

src/libANGLE/renderer/vulkan/VertexArrayVk.cpp

diff --git a/src/libANGLE/renderer/vulkan/VertexArrayVk.cpp b/src/libANGLE/renderer/vulkan/VertexArrayVk.cpp
index 5cff70a..7c70fbb 100644
--- a/src/libANGLE/renderer/vulkan/VertexArrayVk.cpp
+++ b/src/libANGLE/renderer/vulkan/VertexArrayVk.cpp
@@ -522,7 +522,7 @@
             void *srcDataMapping = nullptr;
             ASSERT(!glBuffer->isMapped());
             ANGLE_TRY(bufferVk->map(context, 0, &srcDataMapping));
-            uint8_t *srcData           = reinterpret_cast<uint8_t *>(srcDataMapping);
+            uint8_t *srcData           = static_cast<uint8_t *>(srcDataMapping);
             intptr_t offsetIntoSrcData = reinterpret_cast<intptr_t>(drawCallParams.indices());
             srcData += offsetIntoSrcData;