followed stippi's suggestion and implemented SetEnabled more cleanly

git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@15989 a95241bf-73f2-0310-859d-f6bbb57e9c96
2006-01-16 22:49:05 +00:00
parent dd180e6378
commit 0c7450238f
1 changed files with 6 additions and 9 deletions
@@ -528,18 +528,15 @@ BMenu::SetTargetForItems(BMessenger messenger)
 void
 BMenu::SetEnabled(bool enabled)
 {
+	if (fEnabled == enabled)
+		return;
+
 	fEnabled = enabled;
-
-	
-	if (fSuper) {
-		// Can't use fSuper->SetEnabled() here, as
-		// it would call SetEnabled() on us again, thus
-		// entering an infinite loop
-		fSuper->fEnabled = enabled;
-	}
-
 	for (int32 i = 0; i < CountItems(); i++)
 		ItemAt(i)->SetEnabled(enabled);
+	
+	if (fSuper)
+		fSuper->SetEnabled(enabled);
 }