Building with clang, I'm seeing
 u_blitter.h:627:1: error: control may reach end of non-void function 
[-Werror,-Wreturn-type]

The util_blitter_get_color_format_for_zs() asserts for any
unhandled types, so we do not expect to reach the end of the
function here.

But provide a dummy return with an explicit assert above
to ensure we don't hit it with any future changes to the logic.

Cc: Rob Clark <robdcl...@chromium.org>
Cc: Emil Velikov <emil.l.veli...@gmail.com>
Cc: Amit Pundir <amit.pun...@linaro.org>
Cc: Sumit Semwal <sumit.sem...@linaro.org>
Cc: Alistair Strachan <astrac...@google.com>
Cc: Greg Hartman <ghart...@google.com>
Cc: Tapani Pälli <tapani.pa...@intel.com>
Cc: Marek Olšák <marek.ol...@amd.com>
Signed-off-by: John Stultz <john.stu...@linaro.org>
---
 src/gallium/auxiliary/util/u_blitter.h | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/gallium/auxiliary/util/u_blitter.h 
b/src/gallium/auxiliary/util/u_blitter.h
index 9e3fa55e648..7d6c3db64da 100644
--- a/src/gallium/auxiliary/util/u_blitter.h
+++ b/src/gallium/auxiliary/util/u_blitter.h
@@ -624,6 +624,9 @@ util_blitter_get_color_format_for_zs(enum pipe_format 
format)
    default:
       assert(0);
    }
+   assert(0);
+   /*XXX NEVER GET HERE*/
+   return PIPE_FORMAT_R32G32_UINT;
 }
 
 #ifdef __cplusplus
-- 
2.17.1

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to