commit b41a9d6f0bbc891d4d9d7bdf3b349fa2a99dab7e
author Nicolas Capens <capn@google.com> Wed Jun 27 17:15:10 2018 -0400
committer Nicolas Capens <nicolascapens@google.com> Thu Jun 28 00:14:04 2018 +0000
tree 4d0a164a155c2791a141904b67fc2a0e493831bb
parent 65a3264bb0a9a12f4320535af35a4dc3ffbdd4b1

Fix pedantic narrowing conversion error.

Clang treats conversion of an unsigned constant to a signed one that
doesn't fit its range as an error. Make Visual Studio also produce an
error.

Change-Id: I2d392d5ba7765b72aa9e9478c5eee02102bdf286
Reviewed-on: https://swiftshader-review.googlesource.com/19648
Tested-by: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>

src/Reactor/Reactor.vcxproj

diff --git a/src/Reactor/Reactor.vcxproj b/src/Reactor/Reactor.vcxproj
index 6bd1813..788de64 100644
--- a/src/Reactor/Reactor.vcxproj
+++ b/src/Reactor/Reactor.vcxproj
@@ -123,7 +123,7 @@
       <TreatWarningAsError>true</TreatWarningAsError>

       <ExceptionHandling>false</ExceptionHandling>

       <MultiProcessorCompilation>true</MultiProcessorCompilation>

-      <TreatSpecificWarningsAsErrors>4018;5038</TreatSpecificWarningsAsErrors>

+      <TreatSpecificWarningsAsErrors>4018;5038;4838</TreatSpecificWarningsAsErrors>

     </ClCompile>

     <ProjectReference>

       <LinkLibraryDependencies>false</LinkLibraryDependencies>

@@ -149,7 +149,7 @@
       <OmitFramePointers>false</OmitFramePointers>

       <TreatWarningAsError>true</TreatWarningAsError>

       <MultiProcessorCompilation>true</MultiProcessorCompilation>

-      <TreatSpecificWarningsAsErrors>4018;5038</TreatSpecificWarningsAsErrors>

+      <TreatSpecificWarningsAsErrors>4018;5038;4838</TreatSpecificWarningsAsErrors>

     </ClCompile>

     <ProjectReference>

       <LinkLibraryDependencies>false</LinkLibraryDependencies>

@@ -179,7 +179,7 @@
       <FloatingPointExceptions>false</FloatingPointExceptions>

       <TreatWarningAsError>true</TreatWarningAsError>

       <MultiProcessorCompilation>true</MultiProcessorCompilation>

-      <TreatSpecificWarningsAsErrors>4018;5038</TreatSpecificWarningsAsErrors>

+      <TreatSpecificWarningsAsErrors>4018;5038;4838</TreatSpecificWarningsAsErrors>

     </ClCompile>

     <ProjectReference>

       <LinkLibraryDependencies>false</LinkLibraryDependencies>

@@ -207,7 +207,7 @@
       <StringPooling>true</StringPooling>

       <FloatingPointExceptions>false</FloatingPointExceptions>

       <TreatWarningAsError>true</TreatWarningAsError>

-      <TreatSpecificWarningsAsErrors>4018;5038</TreatSpecificWarningsAsErrors>

+      <TreatSpecificWarningsAsErrors>4018;5038;4838</TreatSpecificWarningsAsErrors>

     </ClCompile>

     <ProjectReference>

       <LinkLibraryDependencies>false</LinkLibraryDependencies>

@@ -239,7 +239,7 @@
       <FloatingPointExceptions>false</FloatingPointExceptions>

       <TreatWarningAsError>true</TreatWarningAsError>

       <MultiProcessorCompilation>true</MultiProcessorCompilation>

-      <TreatSpecificWarningsAsErrors>4018;5038</TreatSpecificWarningsAsErrors>

+      <TreatSpecificWarningsAsErrors>4018;5038;4838</TreatSpecificWarningsAsErrors>

     </ClCompile>

     <ProjectReference>

       <LinkLibraryDependencies>false</LinkLibraryDependencies>

@@ -270,7 +270,7 @@
       <StringPooling>true</StringPooling>

       <FloatingPointExceptions>false</FloatingPointExceptions>

       <TreatWarningAsError>true</TreatWarningAsError>

-      <TreatSpecificWarningsAsErrors>4018;5038</TreatSpecificWarningsAsErrors>

+      <TreatSpecificWarningsAsErrors>4018;5038;4838</TreatSpecificWarningsAsErrors>

     </ClCompile>

     <ProjectReference>

       <LinkLibraryDependencies>false</LinkLibraryDependencies>