Re: [Piglit] Fwd: [PATCH 1/2] port ttexrect.cpp to piglit from glean
On 02/07/2013 08:35 AM, Brian Paul wrote: OK, I thought you had git-write ability. I'll commit/push soon. Actually, I found a few more issues with the code. The piglit_swap_buffers() call should be moved after the probing code. And you should use piglit_present_results() instead. After a swapbuffers, the contents of the back buffer are undefined so you need to probe first. I think that the probe loop can be replaced by a call to piglit_probe_image_rgb(). -Brian ___ Piglit mailing list Piglit@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/piglit
Re: [Piglit] Fwd: [PATCH 1/2] port ttexrect.cpp to piglit from glean
OK, I thought you had git-write ability. I'll commit/push soon. -Brian On 02/07/2013 08:23 AM, Tom Gall wrote: Hi Brian, It has your reviewed by. Ideally I was looking for it to be committed unless of course there's something egregious about it that we both missed. On Thu, Feb 7, 2013 at 9:04 AM, Brian Paul wrote: LGTM. Are you asking for another review or do you need someone to commit/push it for you? -Brian On 02/06/2013 03:38 PM, Tom Gall wrote: It was a little over two weeks ago when v2 of these 2 patches to port ttexrect to piglit and remove it from glean were posted. It has a Reviewed-by: Brian Paul Any reason why this shouldn't go in? Thanks. -- Forwarded message -- From: Brian Paul Date: Mon, Jan 21, 2013 at 4:46 PM Subject: Re: [Piglit] [PATCH 1/2] port ttexrect.cpp to piglit from glean To: Tom Gall Cc: piglit@lists.freedesktop.org, patc...@linaro.org On 01/21/2013 01:43 PM, Tom Gall wrote: This ported test goes into tests/spec/arb_texture_rectangle. It tests the ARB_texture_rectangle extension. Create a 255x127 texture of varying colors and bind it as a GL_ARB_texture_recangle target. Draw that rectangle to the window, and check that the texture was drawn correctly. The common failure to be caught with this test is not adjusting the non-normalized coordinates on hardware that expects normalized coordinates. To all.tests, add new texrect_simple_arb_texrect v2: add PIGLIT_GL_VISUAL_DOUBLE to the window_visual, move various setup bits to piglit_init and add forgotten all.tests change. Signed-off-by: Tom Gall For both: Reviewed-by: Brian Paul -- Regards, Tom "Where's the kaboom!? There was supposed to be an earth-shattering kaboom!" Marvin Martian Tech Lead, Graphics Working Group | Linaro.org │ Open source software for ARM SoCs w) tom.gall att linaro.org h) tom_gall att mac.com ___ Piglit mailing list Piglit@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/piglit -- Regards, Tom "Where's the kaboom!? There was supposed to be an earth-shattering kaboom!" Marvin Martian Tech Lead, Graphics Working Group | Linaro.org │ Open source software for ARM SoCs w) tom.gall att linaro.org h) tom_gall att mac.com ___ Piglit mailing list Piglit@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/piglit
Re: [Piglit] Fwd: [PATCH 1/2] port ttexrect.cpp to piglit from glean
Hi Brian, It has your reviewed by. Ideally I was looking for it to be committed unless of course there's something egregious about it that we both missed. On Thu, Feb 7, 2013 at 9:04 AM, Brian Paul wrote: > LGTM. Are you asking for another review or do you need someone to > commit/push it for you? > > -Brian > > > On 02/06/2013 03:38 PM, Tom Gall wrote: >> >> It was a little over two weeks ago when v2 of these 2 patches to port >> ttexrect to piglit and remove it from glean were posted. >> >> It has a Reviewed-by: Brian Paul >> >> Any reason why this shouldn't go in? >> >> Thanks. >> >> -- Forwarded message -- >> From: Brian Paul >> Date: Mon, Jan 21, 2013 at 4:46 PM >> Subject: Re: [Piglit] [PATCH 1/2] port ttexrect.cpp to piglit from glean >> To: Tom Gall >> Cc: piglit@lists.freedesktop.org, patc...@linaro.org >> >> >> On 01/21/2013 01:43 PM, Tom Gall wrote: >>> >>> >>> This ported test goes into tests/spec/arb_texture_rectangle. It >>> tests the ARB_texture_rectangle extension. >>> >>> Create a 255x127 texture of varying colors and bind it as a >>> GL_ARB_texture_recangle target. Draw that rectangle to the window, and >>> check that the texture was drawn correctly. The common failure to be >>> caught with this test is not adjusting the non-normalized coordinates on >>> hardware that expects normalized coordinates. >>> >>> To all.tests, add new texrect_simple_arb_texrect >>> >>> v2: add PIGLIT_GL_VISUAL_DOUBLE to the window_visual, move various >>> setup bits to piglit_init and add forgotten all.tests change. >>> >>> Signed-off-by: Tom Gall >> >> >> >> For both: >> Reviewed-by: Brian Paul >> >> >> -- >> Regards, >> Tom >> >> "Where's the kaboom!? There was supposed to be an earth-shattering >> kaboom!" Marvin Martian >> Tech Lead, Graphics Working Group | Linaro.org │ Open source software >> for ARM SoCs >> w) tom.gall att linaro.org >> h) tom_gall att mac.com >> ___ >> Piglit mailing list >> Piglit@lists.freedesktop.org >> http://lists.freedesktop.org/mailman/listinfo/piglit > > -- Regards, Tom "Where's the kaboom!? There was supposed to be an earth-shattering kaboom!" Marvin Martian Tech Lead, Graphics Working Group | Linaro.org │ Open source software for ARM SoCs w) tom.gall att linaro.org h) tom_gall att mac.com ___ Piglit mailing list Piglit@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/piglit
Re: [Piglit] Fwd: [PATCH 1/2] port ttexrect.cpp to piglit from glean
LGTM. Are you asking for another review or do you need someone to commit/push it for you? -Brian On 02/06/2013 03:38 PM, Tom Gall wrote: It was a little over two weeks ago when v2 of these 2 patches to port ttexrect to piglit and remove it from glean were posted. It has a Reviewed-by: Brian Paul Any reason why this shouldn't go in? Thanks. -- Forwarded message -- From: Brian Paul Date: Mon, Jan 21, 2013 at 4:46 PM Subject: Re: [Piglit] [PATCH 1/2] port ttexrect.cpp to piglit from glean To: Tom Gall Cc: piglit@lists.freedesktop.org, patc...@linaro.org On 01/21/2013 01:43 PM, Tom Gall wrote: This ported test goes into tests/spec/arb_texture_rectangle. It tests the ARB_texture_rectangle extension. Create a 255x127 texture of varying colors and bind it as a GL_ARB_texture_recangle target. Draw that rectangle to the window, and check that the texture was drawn correctly. The common failure to be caught with this test is not adjusting the non-normalized coordinates on hardware that expects normalized coordinates. To all.tests, add new texrect_simple_arb_texrect v2: add PIGLIT_GL_VISUAL_DOUBLE to the window_visual, move various setup bits to piglit_init and add forgotten all.tests change. Signed-off-by: Tom Gall For both: Reviewed-by: Brian Paul -- Regards, Tom "Where's the kaboom!? There was supposed to be an earth-shattering kaboom!" Marvin Martian Tech Lead, Graphics Working Group | Linaro.org │ Open source software for ARM SoCs w) tom.gall att linaro.org h) tom_gall att mac.com ___ Piglit mailing list Piglit@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/piglit ___ Piglit mailing list Piglit@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/piglit