On Mon, Feb 16, 2015 at 9:13 AM, Martin Peres <martin.pe...@linux.intel.com> wrote: > Maybe this should be the job of the dispatch layer. > > Signed-off-by: Martin Peres <martin.pe...@linux.intel.com> > --- > src/mesa/main/multisample.c | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/src/mesa/main/multisample.c b/src/mesa/main/multisample.c > index 1f3fa0c..a0a659b 100644 > --- a/src/mesa/main/multisample.c > +++ b/src/mesa/main/multisample.c > @@ -150,6 +150,15 @@ GLenum > _mesa_check_sample_count(struct gl_context *ctx, GLenum target, > GLenum internalFormat, GLsizei samples) > { > + /* From the OpenGL core 3.0 spec, section 2.5:
I think this is a bit of a contradiction... there is no OpenGL 3.0 core spec. Core didn't become a thing until 3.1 afaik. > + * > + * "If a negative number is provided where an argument of type sizei or > + * sizeiptr is specified, the error INVALID VALUE is generated." > + */ > + if (samples < 0) { > + return GL_INVALID_VALUE; > + } > + > /* If ARB_internalformat_query is supported, then treat its highest > * returned sample count as the absolute maximum for this format; it is > * allowed to exceed MAX_SAMPLES. > -- > 2.3.0 > > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev