On Wed, Oct 29, 2014 at 5:05 PM, Jose Fonseca <[email protected]> wrote:
> I've commented on the bug, but just noticed this review now after commenting.
>
>> Does piglit have a place where wrapper functions like this should live?
>
> I think that would be tests/util/piglit-util.h
>
> But I suspect this change might break MSVC 2013 (which probably defines or 
> declares the function)...
>
> Like I said to Vinson, unless there is a major reason, I'd rather bump the 
> minimum MSVC version required to build piglit to MSVC 2013, instead of having 
> to deal older MSVC 2012 versions.

That sounds great. Can we do the same with Mesa as well?
_______________________________________________
Piglit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to