[Piglit] [PATCH] varying-packing: add double support

Dave Airlie airlied at gmail.com
Fri Feb 20 15:05:11 PST 2015


On 21 February 2015 at 08:56, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
> On Fri, Feb 20, 2015 at 5:55 PM, Dave Airlie <airlied at gmail.com> wrote:
>> On 20 February 2015 at 20:13, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
>>> Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
>>
>> Looks good to me, did you mean to reindent or respace the types stuff?
>> makes it harder to see what this patch adds.
>
> Yeah, originally I added an extra field so I was going to have to do
> it anyways. But DOUBLE is longer than FLOAT, so it looks awkward if I
> don't do it. I can undo if it bugs you...
>

Nah just makes review a bit harder, clean code is nicer here.

I was actually looking to see if you had added something :-)

Dave.


More information about the Piglit mailing list