[vlc-devel] [PATCH] macosx: Remove teletext case from MainMenu validation

Marvin Scholz epirat07 at gmail.com
Sun Jun 19 12:55:06 CEST 2016


It seems the teletext menu item validation special case is not
necessary at all anymore.
If the parent item is disabled, it does not seem to matter if the
subitems are enabled or not, as access to the subitems is not possible
when the parent is disabled.
---
 modules/gui/macosx/MainMenu.m | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/modules/gui/macosx/MainMenu.m b/modules/gui/macosx/MainMenu.m
index b026c1d..292343d 100644
--- a/modules/gui/macosx/MainMenu.m
+++ b/modules/gui/macosx/MainMenu.m
@@ -1635,16 +1635,6 @@ - (BOOL)validateMenuItem:(NSMenuItem *)mi
         }
     }
 
-    /* Special case for telx menu */
-    if (mi == _normal_window) {
-        bool b_telx = p_input && var_GetInteger(p_input, "teletext-es") >= 0;
-        [[_teletext submenu] setAutoenablesItems:NO];
-
-        for (NSMenuItem *subitem in _teletext.submenu.itemArray) {
-            [subitem setEnabled:b_telx];
-        }
-    }
-
     if (p_input)
         vlc_object_release(p_input);
 
-- 
2.7.4 (Apple Git-66)



More information about the vlc-devel mailing list