commit 99d492c241dcd1451aa9089ac0eabc2bb0b84420
author Corentin Wallez <cwallez@chromium.org> Tue Feb 27 15:17:10 2018 -0500
committer Commit Bot <commit-bot@chromium.org> Tue Mar 06 19:15:33 2018 +0000
tree 68106d90e82b54238569b1ae42c548f35d91a2a3
parent 98d0a2b1313d59050f76861cbef8bee4d70896ad

Use packed enums for the texture types and targets, part 2

This completes the refactor by using the packed enums in the gl:: layer
and in the backends.

The packed enum code generation is modified to support explicitly
assigning values to the packed enums so that the TextureTarget cube map
faces are in the correct order and easy to iterate over.

BUG=angleproject:2169

Change-Id: I5903235e684ccf382e92a8a1e10c5c85b4b16a04
Reviewed-on: https://chromium-review.googlesource.com/939994
Commit-Queue: Corentin Wallez <cwallez@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>

src/libANGLE/ImageIndexIterator_unittest.cpp

diff --git a/src/libANGLE/ImageIndexIterator_unittest.cpp b/src/libANGLE/ImageIndexIterator_unittest.cpp
index 83894df..2b656e2 100644
--- a/src/libANGLE/ImageIndexIterator_unittest.cpp
+++ b/src/libANGLE/ImageIndexIterator_unittest.cpp
@@ -32,8 +32,8 @@
         ImageIndex current = iter.current();
         ImageIndex nextIndex = iter.next();
 
-        EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_2D), nextIndex.type);
-        EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_2D), nextIndex.target);
+        EXPECT_EQ(TextureType::_2D, nextIndex.type);
+        EXPECT_EQ(TextureTarget::_2D, nextIndex.target);
         EXPECT_EQ(mip, nextIndex.mipIndex);
         EXPECT_FALSE(nextIndex.hasLayer());
 
@@ -54,13 +54,12 @@
 
     for (GLint mip = minMip; mip < maxMip; mip++)
     {
-        for (GLenum target = FirstCubeMapTextureTarget; target <= LastCubeMapTextureTarget;
-             target++)
+        for (TextureTarget target : AllCubeFaceTextureTargets())
         {
             EXPECT_TRUE(iter.hasNext());
             ImageIndex nextIndex = iter.next();
 
-            EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_CUBE_MAP), nextIndex.type);
+            EXPECT_EQ(TextureType::CubeMap, nextIndex.type);
             EXPECT_EQ(target, nextIndex.target);
             EXPECT_EQ(mip, nextIndex.mipIndex);
             EXPECT_FALSE(nextIndex.hasLayer());
@@ -83,8 +82,8 @@
             EXPECT_TRUE(iter.hasNext());
             ImageIndex nextIndex = iter.next();
 
-            EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_3D), nextIndex.type);
-            EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_3D), nextIndex.target);
+            EXPECT_EQ(TextureType::_3D, nextIndex.type);
+            EXPECT_EQ(TextureTarget::_3D, nextIndex.target);
             EXPECT_EQ(mip, nextIndex.mipIndex);
             EXPECT_EQ(layer, nextIndex.layerIndex);
             EXPECT_TRUE(nextIndex.hasLayer());
@@ -110,8 +109,8 @@
             EXPECT_TRUE(iter.hasNext());
             ImageIndex nextIndex = iter.next();
 
-            EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_2D_ARRAY), nextIndex.type);
-            EXPECT_EQ(static_cast<GLenum>(GL_TEXTURE_2D_ARRAY), nextIndex.target);
+            EXPECT_EQ(TextureType::_2DArray, nextIndex.type);
+            EXPECT_EQ(TextureTarget::_2DArray, nextIndex.target);
             EXPECT_EQ(mip, nextIndex.mipIndex);
             EXPECT_EQ(layer, nextIndex.layerIndex);
             EXPECT_TRUE(nextIndex.hasLayer());