[Mesa-dev] [PATCH mesa] i965/blorp: add missing braces

Eric Engestrom eric.engestrom at imgtec.com
Wed Jul 6 16:36:54 UTC 2016


Signed-off-by: Eric Engestrom <eric.engestrom at imgtec.com>
---

I know nothing about blorp, but GCC6 noticed the weird indentation, and my best
guess looking at the code is that there are missing braces.

  CC       gen7_blorp.lo
gen7_blorp.c: In function ‘gen7_blorp_exec’:
gen7_blorp.c:797:4: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation]
    if (params->wm_prog_data)
    ^~
gen7_blorp.c:800:7: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘if’
       gen7_blorp_emit_constant_ps_disable(brw);
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

If this is wrong, then the second line should be de-indented.

---
 src/mesa/drivers/dri/i965/gen7_blorp.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/mesa/drivers/dri/i965/gen7_blorp.c b/src/mesa/drivers/dri/i965/gen7_blorp.c
index 7201549..f40e445 100644
--- a/src/mesa/drivers/dri/i965/gen7_blorp.c
+++ b/src/mesa/drivers/dri/i965/gen7_blorp.c
@@ -794,10 +794,11 @@ gen7_blorp_exec(struct brw_context *brw,
    gen6_blorp_emit_clip_disable(brw);
    gen7_blorp_emit_sf_config(brw, params);
    gen7_blorp_emit_wm_config(brw, params);
-   if (params->wm_prog_data)
+   if (params->wm_prog_data) {
       gen7_blorp_emit_binding_table_pointers_ps(brw, wm_bind_bo_offset);
 
       gen7_blorp_emit_constant_ps_disable(brw);
+   }
 
    if (params->src.mt) {
       const uint32_t sampler_offset =
-- 
2.9.0



More information about the mesa-dev mailing list