diff --git a/systray_darwin.m b/systray_darwin.m
index 485256d..3a28c73 100644
--- a/systray_darwin.m
+++ b/systray_darwin.m
@@ -127,7 +127,7 @@
     return;
   }
   menuItem = [menu itemAtIndex: existedMenuIndex];
-  [menuItem setHidden:NO];
+  [menuItem setHidden:FALSE];
 }
 
 - (void) quit
diff --git a/systray_linux.c b/systray_linux.c
index 7258613..64d7e8f 100644
--- a/systray_linux.c
+++ b/systray_linux.c
@@ -117,9 +117,10 @@ gboolean do_hide_menu_item(gpointer data) {
 		MenuItemNode* item = (MenuItemNode*)(it->data);
 		if(item->menu_id == mii->menu_id){
 			gtk_widget_hide(GTK_WIDGET(item->menu_item));
-			return;
+			break;
 		}
 	}
+	return FALSE;
 }
 
 // runs in main thread, should always return FALSE to prevent gtk to execute it again
@@ -130,9 +131,10 @@ gboolean do_show_menu_item(gpointer data) {
 		MenuItemNode* item = (MenuItemNode*)(it->data);
 		if(item->menu_id == mii->menu_id){
 			gtk_widget_show(GTK_WIDGET(item->menu_item));
-			return;
+			break;
 		}
 	}
+	return FALSE;
 }
 
 // runs in main thread, should always return FALSE to prevent gtk to execute it again