Re: [Piglit] [PATCH] arb_sample_shading-builtin-gl-sample-position: Fix VS 2015 build.

2015-07-21 Thread Brian Paul
On 07/21/2015 06:24 PM, Vinson Lee wrote: builtin-gl-sample-position.cpp(81): error C3688: invalid literal suffix 'MAX_NUM_SAMPLES_STR'; literal operator or literal operator template 'operator ""MAX_NUM_SAMPLES_STR' not found Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91419 Signed-

[Piglit] [PATCH] arb_sample_shading-builtin-gl-sample-position: Fix VS 2015 build.

2015-07-21 Thread Vinson Lee
builtin-gl-sample-position.cpp(81): error C3688: invalid literal suffix 'MAX_NUM_SAMPLES_STR'; literal operator or literal operator template 'operator ""MAX_NUM_SAMPLES_STR' not found Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91419 Signed-off-by: Vinson Lee --- tests/spec/arb_samp

[Piglit] [Bug 91419] New: builtin-gl-sample-position.cpp(81): error C3688: invalid literal suffix 'MAX_NUM_SAMPLES_STR'

2015-07-21 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=91419 Bug ID: 91419 Summary: builtin-gl-sample-position.cpp(81): error C3688: invalid literal suffix 'MAX_NUM_SAMPLES_STR' Product: piglit Version: unspecified Hardware: x86-64 (

Re: [Piglit] [PATCH] Test that interpolateAtSample returns different results

2015-07-21 Thread Dave Airlie
On 22 July 2015 at 02:13, Neil Roberts wrote: > Adds a new test for interpolateAtSample which captures the results > from the function with each different sample value and ensures that > they are all different. This can be done either with a constant or > non-constant value depending on whether a

[Piglit] [PATCH] Test that interpolateAtSample returns different results

2015-07-21 Thread Neil Roberts
Adds a new test for interpolateAtSample which captures the results from the function with each different sample value and ensures that they are all different. This can be done either with a constant or non-constant value depending on whether a command line argument is given. The other tests for in

Re: [Piglit] [PATCH] egl_khr_fence_sync: Remove unused variable.

2015-07-21 Thread Timothy Arceri
Reviewed-by: Timothy Arceri On Mon, 2015-07-20 at 20:50 -0700, Vinson Lee wrote: > Fix compiler warnings. > > egl_khr_fence_sync.c:70:36: warning: tentative array definition assumed to > have one element > static const struct piglit_subtest subtests[]; >^ > e