[Piglit] [PATCH] amd_performance_monitor: Add braces around report macro.
Vinson Lee
vlee at freedesktop.org
Sat May 4 23:11:42 PDT 2013
The macro 'report' expands to multiple statements. Add braces to nest
all statements in expanded macro within if-statement.
Fixes "Nesting level does not match indentation" defect reported by
Coverity.
Signed-off-by: Vinson Lee <vlee at freedesktop.org>
---
tests/spec/amd_performance_monitor/api.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/tests/spec/amd_performance_monitor/api.c b/tests/spec/amd_performance_monitor/api.c
index 7b321cf..a7edf47 100644
--- a/tests/spec/amd_performance_monitor/api.c
+++ b/tests/spec/amd_performance_monitor/api.c
@@ -436,8 +436,9 @@ test_group_string_normal_buffer(unsigned valid_group)
/* Get the length; bail if unwritten to avoid huge allocations. */
glGetPerfMonitorGroupStringAMD(valid_group, 0, &length, NULL);
pass = pass && piglit_check_gl_error(GL_NO_ERROR);
- if (length == 0xd0d0d0d0)
+ if (length == 0xd0d0d0d0) {
report(false);
+ }
name = malloc(length + 1);
assert(name != NULL);
@@ -580,8 +581,9 @@ test_counter_string_normal_buffer(unsigned group, unsigned counter)
/* Get the length; bail if unwritten to avoid huge allocations. */
glGetPerfMonitorCounterStringAMD(group, counter, 0, &length, NULL);
pass = pass && piglit_check_gl_error(GL_NO_ERROR);
- if (length == 0xd0d0d0d0)
+ if (length == 0xd0d0d0d0) {
report(false);
+ }
name = malloc(length + 1);
assert(name != NULL);
--
1.8.1.3
More information about the Piglit
mailing list