<div dir="ltr">On 10 June 2013 15:26, Fabian Bieler <span dir="ltr"><<a href="mailto:fabianbieler@fastmail.fm" target="_blank">fabianbieler@fastmail.fm</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">It used to be piglit_width which works, too, as the framebuffer is hardcoded<br>
to 250x250 pixels.<br>
---<br>
 tests/shaders/shader_runner.c | 2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br></blockquote><div><br></div><div>s/heigth/height/ in the subject.<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<br>
diff --git a/tests/shaders/shader_runner.c b/tests/shaders/shader_runner.c<br>
index 7fc74aa..515fde1 100644<br>
--- a/tests/shaders/shader_runner.c<br>
+++ b/tests/shaders/shader_runner.c<br>
@@ -1751,7 +1751,7 @@ piglit_display(void)<br>
                                  c + 0, c + 1,<br>
                                  c + 2, c + 3, c + 4, c + 5) == 6) {<br>
                        x = c[0] * piglit_width;<br>
-                       y = c[1] * piglit_width;<br>
+                       y = c[1] * piglit_height;<br>
                        if (x >= piglit_width)<br>
                                x = piglit_width - 1;<br>
                        if (y >= piglit_height)<br></blockquote><div><br></div><div>It looks like a similar change needs to be made about 20 lines below, in the handling of "relative probe rgb".<br><br></div>
<div>With those changes, this patch is:<br><br>Reviewed-by: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<span class=""><font color="#888888">--<br>
1.8.1.2<br>
<br>
_______________________________________________<br>
Piglit mailing list<br>
<a href="mailto:Piglit@lists.freedesktop.org">Piglit@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/piglit" target="_blank">http://lists.freedesktop.org/mailman/listinfo/piglit</a><br>
</font></span></blockquote></div><br></div></div>