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/FramebufferVk.cpp

diff --git a/src/libANGLE/renderer/vulkan/FramebufferVk.cpp b/src/libANGLE/renderer/vulkan/FramebufferVk.cpp
index c22b2f9..a86da49 100644
--- a/src/libANGLE/renderer/vulkan/FramebufferVk.cpp
+++ b/src/libANGLE/renderer/vulkan/FramebufferVk.cpp
@@ -315,7 +315,7 @@
     params.pack        = glState.getPackState();
 
     ANGLE_TRY(readPixelsImpl(context, clippedArea, params,
-                             reinterpret_cast<uint8_t *>(pixels) + outputSkipBytes));
+                             static_cast<uint8_t *>(pixels) + outputSkipBytes));
     mReadPixelsBuffer.releaseRetainedBuffers(renderer);
     return gl::NoError();
 }
@@ -836,7 +836,7 @@
     ANGLE_TRY(mReadPixelsBuffer.invalidate(renderer->getDevice()));
 
     PackPixels(packPixelsParams, angleFormat, area.width * angleFormat.pixelBytes, readPixelBuffer,
-               reinterpret_cast<uint8_t *>(pixels));
+               static_cast<uint8_t *>(pixels));
 
     return vk::NoError();
 }