Re: [Mesa-dev] [PATCH 2/2] egl-nok-swap-region: probe front not back buffer

2012-09-20 Thread Brian Paul
On 09/20/2012 07:21 AM, Robert Bragg wrote: The egl-nok-swap-buffer was mistakenly reading the back buffer to test the correctness of eglSwapRegionNOK but once the second glClear() to red has been issued the whole back buffer is red and so the test would always passed since it only probed points

Re: [Mesa-dev] [PATCH 2/2] egl-nok-swap-region: probe front not back buffer

2012-09-20 Thread Robert Bragg
Ah, I only just realized there is a separate list for piglit tests, sorry. Hopefully it's ok to review these here this time, but I'll remember to send piglit patches to the right list in the future. regards, - Robert On Thu, Sep 20, 2012 at 2:21 PM, Robert Bragg wrote: > The egl-nok-swap-buffer

[Mesa-dev] [PATCH 2/2] egl-nok-swap-region: probe front not back buffer

2012-09-20 Thread Robert Bragg
The egl-nok-swap-buffer was mistakenly reading the back buffer to test the correctness of eglSwapRegionNOK but once the second glClear() to red has been issued the whole back buffer is red and so the test would always passed since it only probed points it expected to be red anyway. This patch now