Handle gcc 7 warnings
authorCaio Marcelo de Oliveira Filho <caio.oliveira@intel.com>
Thu, 22 Feb 2018 01:54:57 +0000 (17:54 -0800)
committerChris Forbes <chrisforbes@google.com>
Thu, 8 Mar 2018 23:39:05 +0000 (15:39 -0800)
commit7fb99bc0541b2ba9d2b08ddafce5c3eb819ea4fd
tree638577fd73ce9b6381dcde7619116d9d50b2582a
parenta76eb94eda712267ccded7d3dfe5065fb6d15e37
Handle gcc 7 warnings

- "-Wimplicit-fallthrough" warns on case conditions that
  fallthrough. GCC does have a way to accept conditions that have a
  comment about falling through inside the cases, but the codebase has
  other cases where a single comment explains the fallthrough for the
  whole switch. This warning is being ignored. It was not possible to
  use "-Wno-error=WARNING" because older GCCs fail -- but they do
  accept "-Wno-WARNING" where WARNING is unknown (since at least gcc
  4.6.4).

- "-Wint-in-bool-context" warns (among other things) about result of
  multiplication being used in boolean context. Code was changed to
  use de::max() instead.

- "-Wmaybe-uninitialized" warned about some attributes of a struct not
  being initialized depending on the case. Code was changed to
  initialize the struct in question.

Components: Framework, Vulkan
(Cherry-picked from Khronos change c3342293855b8cdb093408d867b44d0cc6b42155)
Test: python scripts/check_build_sanity.py with gcc-7
Bug: b/74405145

Change-Id: Iabb26f01e047353c6e2a704ab27b673e85b185be
external/vulkancts/modules/vulkan/shaderexecutor/vktShaderExecutor.cpp
external/vulkancts/modules/vulkan/texture/vktTextureFilteringExplicitLodTests.cpp
scripts/check_build_sanity.py