diff options
author | Eric Anholt <[email protected]> | 2013-04-10 09:53:11 -0700 |
---|---|---|
committer | Eric Anholt <[email protected]> | 2013-04-29 11:41:33 -0700 |
commit | eac11996047521a5107d7877066b59db0a278d70 (patch) | |
tree | 4e079cf81292587b27897a3fe015c4ea9013ea6e /src | |
parent | 2534f0a57d27cf99b6eed5ab94cdbd96e36a6fb9 (diff) |
mesa: Add an extra clarifying set of braces to getter checking.
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.
Diffstat (limited to 'src')
-rw-r--r-- | src/mesa/main/get.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/mesa/main/get.c b/src/mesa/main/get.c index 6bc5f2976e0..73ad8fe5378 100644 --- a/src/mesa/main/get.c +++ b/src/mesa/main/get.c @@ -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, |