Fix some compiler warnings

Void function should actually be declared void. Not-reached switch
condition should still return something.

diff --git a/obt/prop.c b/obt/prop.c
index f7919d6c380a64d919d2d909a5f6cede435f5490..0cecccf4dfa6d5b160a78e515aeca185b2b462f5 100644 (file)
--- 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 3f4c4d64f72c15d5f1df8fcb9baa10045d772ab5..411e95c076c6f355c25b11a98b83979cc5d360c8 100644 (file)
--- 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;