assert(0) is not guaranteed to not return since assert() in general can
be optimised out when building without debug / with optimisation.  This
breaks the build in clang, which warns/errors about the missing return.

Signed-off-by: David Lamparter <[email protected]>
---
Eating my own dog food, I'm now working with -Werror, and this warning
thus breaks my testbuilds on clang :)

As this is absolutely trivial, I've pushed it to master already.
---
 lib/command.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/lib/command.c b/lib/command.c
index c70391d..8ae27de 100644
--- a/lib/command.c
+++ b/lib/command.c
@@ -2018,6 +2018,7 @@ cmd_entry_function_desc (const char *src, struct 
cmd_token *token)
 
       default:
         assert(0);
+        return NULL;
     }
 }
 
-- 
2.0.4


_______________________________________________
Quagga-dev mailing list
[email protected]
https://lists.quagga.net/mailman/listinfo/quagga-dev

Reply via email to