On Tue, Apr 11, 2017 at 6:52 PM, Ilia Mirkin <imir...@alum.mit.edu> wrote:
> On Tue, Apr 11, 2017 at 12:17 PM, Marek Olšák <mar...@gmail.com> wrote:
>> On Mon, Apr 10, 2017 at 11:12 PM, Ilia Mirkin <imir...@alum.mit.edu> wrote:
>>> This appears to be nop.shader_test almost verbatim...
>>
>> But the difference is clear: nop.shader_test ends up using a constant
>> buffer for vertex data, while this tests uses a vertex buffer.
>
> Ah OK. Well, I don't care that much about the naming, but it just
> seems sad to end up with a bunch of nop/nop2/nop75/etc shaders which
> are all the same but subtly different. I'd recommend trying for a more
> descriptive name. Either way, this seems like a legal test, so
>
> Reviewed-by: Ilia Mirkin <imir...@alum.mit.edu>

I can rename all the tests to *trivial-*, but I wouldn't bother
sending it to this list.

Marek
_______________________________________________
Piglit mailing list
Piglit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to