mesa: Add an extra clarifying set of braces to getter checking. 74/6274/1
authorEric Anholt <eric@anholt.net>
Wed, 10 Apr 2013 16:53:11 +0000 (09:53 -0700)
committerEric Anholt <eric@anholt.net>
Mon, 29 Apr 2013 18:41:33 +0000 (11:41 -0700)
For this multi-page single statement, my thought the end was to that the
next block was mis-indented, rather than that the dropped indentation
actually indicated the end of the loop.

src/mesa/main/get.c

index 6bc5f29..73ad8fe 100644 (file)
@@ -905,7 +905,7 @@ check_extra(struct gl_context *ctx, const char *func, const struct value_desc *d
    GLboolean api_found = GL_FALSE;
    const int *e;
 
-   for (e = d->extra; *e != EXTRA_END; e++)
+   for (e = d->extra; *e != EXTRA_END; e++) {
       switch (*e) {
       case EXTRA_VERSION_30:
          api_check = GL_TRUE;
@@ -987,6 +987,7 @@ check_extra(struct gl_context *ctx, const char *func, const struct value_desc *d
            api_found = GL_TRUE;
         break;
       }
+   }
 
    if (api_check && !api_found) {
       _mesa_error(ctx, GL_INVALID_ENUM, "%s(pname=%s)", func,