Fix redundant draw in DrawBaseVertexBaseInstanceTest

Discovered this redundant draw issue when writing javascript webgl conformance test. Together fix some broken formatting. TBR=geofflang@chromium.org Bug: 3402 Change-Id: I29dbcd04da7e47d9e0bc79074ff69202378c1214 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/1816822Reviewed-by: Shrek Shao <shrekshao@google.com> Commit-Queue: Shrek Shao <shrekshao@google.com>

Fix redundant draw in DrawBaseVertexBaseInstanceTest
c847a75f · shrekshao · Commit Bot · 1850492e · c847a75f
Commit c847a75f authored Sep 20, 2019 by shrekshao Committed by Commit Bot Oct 03, 2019
Hide whitespace changes
Inline Side-by-side

Showing with 18 additions and 18 deletions

DrawBaseVertexBaseInstanceTest.cpp src/tests/gl_tests/DrawBaseVertexBaseInstanceTest.cpp +18 -18

No files found.
--- a/src/tests/gl_tests/DrawBaseVertexBaseInstanceTest.cpp
+++ b/src/tests/gl_tests/DrawBaseVertexBaseInstanceTest.cpp
@@ -309,18 +309,18 @@ void main()

        const uint32_t countPerDraw = kCountY * 6;

-            const GLsizei drawCount = kCountX;
-            const std::vector<GLsizei> counts(drawCount, countPerDraw);
-            const std::vector<GLsizei> firsts(drawCount, 0);
-            const std::vector<GLsizei> instanceCounts(drawCount, 2);
-            std::vector<GLuint> baseInstances(drawCount);
-            for (size_t i = 0; i < drawCount; i++)
-            {
-                baseInstances[i] = i * 2;
-            }
-            glMultiDrawArraysInstancedBaseInstanceANGLE(GL_TRIANGLES, drawCount, counts.data(),
-                                                        instanceCounts.data(), firsts.data(),
-                                                        baseInstances.data());
+        const GLsizei drawCount = kCountX / 2;
+        const std::vector<GLsizei> counts(drawCount, countPerDraw);
+        const std::vector<GLsizei> firsts(drawCount, 0);
+        const std::vector<GLsizei> instanceCounts(drawCount, 2);
+        std::vector<GLuint> baseInstances(drawCount);
+        for (size_t i = 0; i < drawCount; i++)
+        {
+            baseInstances[i] = i * 2;
+        }
+        glMultiDrawArraysInstancedBaseInstanceANGLE(GL_TRIANGLES, drawCount, counts.data(),
+                                                    instanceCounts.data(), firsts.data(),
+                                                    baseInstances.data());
    }

    void doDrawElementsInstancedBaseVertexBaseInstance()
@@ -331,12 +331,12 @@ void main()

        for (uint32_t v = 0; v < kCountY; v++)
        {
-                for (uint32_t i = 0; i < kCountX; i += 2)
-                {
-                    glDrawElementsInstancedBaseVertexBaseInstanceANGLE(
-                        GL_TRIANGLES, countPerDraw, GL_UNSIGNED_SHORT,
-                        reinterpret_cast<GLvoid *>(static_cast<uintptr_t>(0)), 2, v * 4, i);
-                }
+            for (uint32_t i = 0; i < kCountX; i += 2)
+            {
+                glDrawElementsInstancedBaseVertexBaseInstanceANGLE(
+                    GL_TRIANGLES, countPerDraw, GL_UNSIGNED_SHORT,
+                    reinterpret_cast<GLvoid *>(static_cast<uintptr_t>(0)), 2, v * 4, i);
+            }
        }
    }