From 30f8293685963e71f35e1beaf2672bf4729905be Mon Sep 17 00:00:00 2001
From: Ox Cart <ox.to.a.cart@gmail.com>
Date: Tue, 12 Sep 2017 05:43:24 -0500
Subject: [PATCH] Code review fixes

---
 systray_darwin.m | 2 +-
 systray_linux.c  | 6 ++++--
 2 files changed, 5 insertions(+), 3 deletions(-)

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