Fix debug build on Linux

Compiling an assert in Program.cpp was failing at least on GCC 4.8 because it compared unsigned size_t to zero, a comparison that was always true. The C standard defines size_t as unsigned, so the assert shouldn't be necessary on other platforms either. There was already a commit on top of the patch that added the bug, so it is simpler to fix the problem by changing the assert rather than reverting. TEST=standalone debug build on Linux Change-Id: Ifd910332a770f7360a15c31706beca740d0f289d Reviewed-on: https://chromium-review.googlesource.com/294971Reviewed-by: Olli Etuaho <oetuaho@nvidia.com> Tested-by: Olli Etuaho <oetuaho@nvidia.com>

Fix debug build on Linux
401d9fe6 · Olli Etuaho · d7a25243 · 401d9fe6
Commit 401d9fe6 authored Aug 26, 2015 by Olli Etuaho
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 2 deletions

Program.cpp src/libANGLE/Program.cpp +1 -2

No files found.
--- a/src/libANGLE/Program.cpp
+++ b/src/libANGLE/Program.cpp
@@ -615,8 +615,7 @@ GLuint Program::getAttributeLocation(const std::string &name)
 bool Program::isAttribLocationActive(size_t attribLocation) const
 {
-    ASSERT(attribLocation >= 0 &&
+    ASSERT(attribLocation < mData.mActiveAttribLocationsMask.size());
-           static_cast<size_t>(attribLocation) < mData.mActiveAttribLocationsMask.size());
    return mData.mActiveAttribLocationsMask[attribLocation];
 }