[Piglit] [PATCH] texwrap: test wrapping texture offsets
Brian Paul
brianp at vmware.com
Wed May 27 05:59:28 PDT 2015
On 05/27/2015 03:25 AM, Dave Airlie wrote:
>>> case GL_MIRROR_CLAMP_EXT:
>>> @@ -862,6 +868,10 @@ static void draw(const struct format_desc *format,
>>> scale[0] = scale[1] = scale[2] = scale[3] = 1.0/((1ull <<
>>> (bits-1))-1);
>>> glUseProgram(prog_int);
>>> glUniform4fv(int_scale_loc, 1, scale);
>>> + if (texture_offset)
>>> + glUniform1f(int_use_offset_loc, 1.0);
>>> + else
>>> + glUniform1f(int_use_offset_loc, 0.0);
>>
>>
>> Is it legal to set boolean uniforms with a float? I would expect that
>> glUniform1i() would be required here. In any case, this could boil down to:
>> glUniform1f(int_use_offset_loc, texture_offset);
>
> Yes its legal but Uniform1i with texture_offset is cleaner alright.
> I'll fix that up.
>
> and I've fixed up the tabs, can I consider that an R-b?
>
> Thanks,
> Dave.
>
Reviewed-by: Brian Paul <brianp at vmware.com>
Thanks.
More information about the Piglit
mailing list