all repos — fluxbox @ 05a52116dfa39a07bc5739340eb1a84660c96e25

custom fork of the fluxbox windowmanager 

MenuItem::isEnabled() is sufficient to decide Menu::isItemSelectable()

MenuSeparator is never enabled since it returns always 'false' in its
implementation of isEnabled(). no need to use RTTI imho.
Mathias Gumz akira at fluxbox dot org
commit

05a52116dfa39a07bc5739340eb1a84660c96e25

parent

bb480030f54947b19b761f37dd3c4756b361b5e1

1 files changed, 1 insertions(+), 3 deletions(-)

jump to
M src/FbTk/Menu.cc → src/FbTk/Menu.cc 
@@ -842,9 +842,7 @@ 
     if (index >= menuitems.size()) return false;
 
     const MenuItem *item = find(index);
-    return (!item ||
-        (typeid(*item) == typeid(FbTk::MenuSeparator)) ||
-        !item->isEnabled()) ? false : true;
+    return (!item || !item->isEnabled()) ? false : true;
 }