From: Jamie Madill <jmadill@chromium.org>
Date: Wed, 8 Nov 2017 19:48:32 +0000 (-0500)
Subject: loader: Fix improper cast warning.
X-Git-Tag: upstream/1.1.92~634
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=6d7a500f365ee90584222f3fe332ffd95d1f9c41;p=platform%2Fupstream%2FVulkan-Tools.git

loader: Fix improper cast warning.

warning C4057: 'function': 'LPBYTE' differs in indirection to slightly different base types

This was popping up in some Windows builds. Also add it to the list
of warnings enabled by default with MSVS.

Change-Id: I2703f2e57ba3fb33ec03fbc7b8324c75187abec6
---

diff --git a/CMakeLists.txt b/CMakeLists.txt
index fdc63bd..785d6fc 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -94,6 +94,8 @@ if(WIN32)
     # Warn about potentially uninitialized variables
     add_compile_options("$<$<CXX_COMPILER_ID:MSVC>:/w34701>")
     add_compile_options("$<$<CXX_COMPILER_ID:MSVC>:/w34703>")
+    # Warn about different indirection types.
+    add_compile_options("$<$<CXX_COMPILER_ID:MSVC>:/w34057>")
 endif()
 
 if(NOT WIN32)
diff --git a/loader/loader.c b/loader/loader.c
index 8f954e0..ca6d571 100644
--- a/loader/loader.c
+++ b/loader/loader.c
@@ -465,7 +465,7 @@ bool loaderGetDeviceRegistryEntry(const struct loader_instance *inst, char **reg
         value_name,
         NULL,
         &data_type,
-        manifest_path,
+        (BYTE *)manifest_path,
         &requiredSize
     );