[Piglit] [PATCH 1/2] Modify shader attachment order in piglit_link_simple_program()

Paul Berry stereotype441 at gmail.com
Tue Nov 1 14:27:34 PDT 2011


Some tests (in particular, glsl-link-initializer-03.c) assume that
piglit_link_simple_program() attaches shaders in the same order that
they appear in the parameter list.  There's no good reason for
piglit_link_simple_program() to attach them in the opposite order.
---
 tests/util/piglit-shader.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/util/piglit-shader.c b/tests/util/piglit-shader.c
index c5c8c2d..3e0f935 100644
--- a/tests/util/piglit-shader.c
+++ b/tests/util/piglit-shader.c
@@ -240,10 +240,10 @@ GLint piglit_link_simple_program(GLint vs, GLint fs)
 	piglit_require_GLSL();
 
 	prog = piglit_CreateProgram();
-	if (fs)
-		piglit_AttachShader(prog, fs);
 	if (vs)
 		piglit_AttachShader(prog, vs);
+	if (fs)
+		piglit_AttachShader(prog, fs);
 	piglit_LinkProgram(prog);
 
 	if (!piglit_link_check_status(prog)) {
-- 
1.7.6.4



More information about the Piglit mailing list