Re: [Mesa-dev] [PATCH v2] amd_performance_monitor: Fix multi-statement macro 'report'.

2013-05-07 Thread Kenneth Graunke
On 05/07/2013 02:04 PM, Vinson Lee wrote: Fixes "Nesting level does not match indentation" defect reported by Coverity. That has to be the stupidest defect I've ever heard of. But your patch looks reasonable nonetheless - most people use do-while blocks like that, so it's probably a good ide

[Mesa-dev] [PATCH v2] amd_performance_monitor: Fix multi-statement macro 'report'.

2013-05-07 Thread Vinson Lee
Fixes "Nesting level does not match indentation" defect reported by Coverity. Signed-off-by: Vinson Lee --- 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_mo