[Piglit] [PATCH 08/12] util/piglit-vbo: do not use 'and' in a conditional statement

Ilia Mirkin imirkin at alum.mit.edu
Tue Aug 12 09:30:42 PDT 2014


On Tue, Aug 12, 2014 at 1:18 PM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> I'm not entirely sure how piglit build with gcc as is, yet VC compiler
> seems very unhappy about this.

http://www.open-std.org/jtc1/sc22/wg14/www/docs/n1124.pdf

See 7.9 Alternate spellings <iso646.h>. I guess that gets included by
gcc somehow.

>
> Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>

Reviewed-by: Ilia Mirkin <imirkin at alum.mit.edu>

> ---
>  tests/util/piglit-vbo.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/util/piglit-vbo.cpp b/tests/util/piglit-vbo.cpp
> index 8a6ef90..19fddde 100644
> --- a/tests/util/piglit-vbo.cpp
> +++ b/tests/util/piglit-vbo.cpp
> @@ -284,7 +284,7 @@ vertex_attrib_description::setup(size_t *offset, size_t stride) const
>                 glVertexAttribPointer(this->index, this->count,
>                                       this->data_type, GL_FALSE, stride,
>                                       (void *) *offset);
> -       } else if (piglit_is_gles() and piglit_get_gl_version() < 30) {
> +       } else if (piglit_is_gles() && piglit_get_gl_version() < 30) {
>                 fprintf(stderr,"vertex_attrib_description fail. no int support\n");
>         } else {
>                 glVertexAttribIPointer(this->index, this->count,
> --
> 2.0.2
>
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit


More information about the Piglit mailing list