On 02/26/2015 06:50 PM, Ilia Mirkin wrote:
Weren't you guys going to flip ARB_dsa to be core-only? Then you'll
need to set supports_gl_core_version = 32 for the test to actually
run... [And remove the compat version support.]

Ah right, then 2.0 would not be not enough.

On Thu, Feb 26, 2015 at 10:41 AM, Martin Peres
<[email protected]> wrote:
Also add the relevant quote from the spec.

Signed-off-by: Martin Peres <[email protected]>
---
  .../arb_direct_state_access/create-transformfeedbacks.c    | 14 +++++++++++++-
  1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/tests/spec/arb_direct_state_access/create-transformfeedbacks.c 
b/tests/spec/arb_direct_state_access/create-transformfeedbacks.c
index f837c13..1419632 100644
--- a/tests/spec/arb_direct_state_access/create-transformfeedbacks.c
+++ b/tests/spec/arb_direct_state_access/create-transformfeedbacks.c
@@ -25,6 +25,14 @@
   *
   * Tests glCreateTransformFeedbacks to see if it behaves in the expected way,
   * throwing the correct errors, etc.
+ *
+ * From OpenGL 4.5, section 13.2.1 "Transform Feedback Objects", page 420:
+ *
+ * "CreateTransformFeedbacks returns n previously unused transform feedback
+ * object names in ids, each representing a new state vector, comprising the
+ * state and with all the same initial values listed in table 23.48.
+ * Errors
+ *   An INVALID_VALUE error is generated if n is negative."
   */

  #include "piglit-util-gl.h"
@@ -32,7 +40,7 @@

  PIGLIT_GL_TEST_CONFIG_BEGIN

-       config.supports_gl_compat_version = 13;
+       config.supports_gl_compat_version = 20;

         config.window_visual = PIGLIT_GL_VISUAL_RGBA |
                 PIGLIT_GL_VISUAL_DOUBLE;
@@ -71,5 +79,9 @@ piglit_display(void)
         SUBTESTCONDITION(glIsTransformFeedback(ids[2]), pass,
                         "IsTransformFeedback()");

+       /* the default state is tested in the following piglit test:
+        *      arb_direct_state_access-gettransformfeedback
+        */
+
         return pass ? PIGLIT_PASS : PIGLIT_FAIL;
  }
--
2.3.0

_______________________________________________
Piglit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/piglit
_______________________________________________
Piglit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/piglit

_______________________________________________
Piglit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to