From bcfe59fce27bb6a7aa7bc9dde2ad461dfaf68816 Mon Sep 17 00:00:00 2001 From: Dana Jansens Date: Sun, 11 Aug 2013 14:39:26 -0400 Subject: [PATCH] Fix some compiler warnings Void function should actually be declared void. Not-reached switch condition should still return something. --- obt/prop.c | 1 + openbox/actions/if.c | 8 ++++---- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/obt/prop.c b/obt/prop.c index f7919d6c..0cecccf4 100644 --- a/obt/prop.c +++ b/obt/prop.c @@ -314,6 +314,7 @@ static gboolean get_text_property(Window win, Atom prop, return tprop->encoding == OBT_PROP_ATOM(UTF8_STRING); default: g_assert_not_reached(); + return FALSE; } } diff --git a/openbox/actions/if.c b/openbox/actions/if.c index 3f4c4d64..411e95c0 100644 --- a/openbox/actions/if.c +++ b/openbox/actions/if.c @@ -46,10 +46,10 @@ void action_if_startup(void) actions_register("If", setup_func, free_func, run_func); } -static inline set_bool(xmlNodePtr node, - const char *name, - gboolean *on, - gboolean *off) +static inline void set_bool(xmlNodePtr node, + const char *name, + gboolean *on, + gboolean *off) { xmlNodePtr n; -- 2.45.2