[Piglit] [PATCH] glean: remove line drawing code from fbo test
Brian Paul
brianp at vmware.com
Tue Aug 15 23:08:49 UTC 2017
On 08/04/2017 01:08 PM, Eric Anholt wrote:
> Brian Paul <brianp at vmware.com> writes:
>
>> The glean fbo test exercises rendering to 1D, 2D, 3D and cube images
>> attached to FBOs. For the 1D texture case, the rendering is done with
>> a line, rather than a polygon. Since the 1D image is one pixel high,
>> the slightest mis-rasterization of the line will cause it to be clipped
>> and the test to fail.
>>
>> Just fall through to the polygon drawing code to avoid that issue.
>> Less code too.
>
> If you just remove the whole switch statement around the POLYGON drawing
> code, this would get my r-b. :)
>
> I like this. Test line rasterization only in something that's about
> lines -- they're tricky to test well, and we've had plenty of trouble
> with this draw call.
>
My patch was already pushed. I'm posting a follow-up to do this clean-up.
-Brian
More information about the Piglit
mailing list