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

epirat07@gmail.com git at videolan.org
Sun Jun 26 22:55:25 CEST 2016


vlc | branch: master | epirat07 at gmail.com <epirat07 at gmail.com> | Sun Jun 19 12:55:06 2016 +0200| [e961f02bfbce28a0e6e28ef93efff288d9504cb0] | committer: Jean-Baptiste Kempf

macosx: Remove teletext case from MainMenu validation

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.

Signed-off-by: Jean-Baptiste Kempf <jb at videolan.org>

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=e961f02bfbce28a0e6e28ef93efff288d9504cb0
---

 modules/gui/macosx/MainMenu.m |   11 -----------
 1 file changed, 11 deletions(-)

diff --git a/modules/gui/macosx/MainMenu.m b/modules/gui/macosx/MainMenu.m
index cfb679b..030aa48 100644
--- a/modules/gui/macosx/MainMenu.m
+++ b/modules/gui/macosx/MainMenu.m
@@ -1630,17 +1630,6 @@
             bEnabled = _openSubtitleFile.isEnabled;
     }
 
-    /* Special case for telx menu */
-    if ([title isEqualToString: _NS("Normal Size")]) {
-        NSMenuItem *item = [[mi menu] itemWithTitle:_NS("Teletext")];
-        bool b_telx = p_input && var_GetInteger(p_input, "teletext-es") >= 0;
-
-        [[item submenu] setAutoenablesItems:NO];
-
-        for (int k=0; k < [[item submenu] numberOfItems]; k++)
-            [[[item submenu] itemAtIndex:k] setEnabled: b_telx];
-    }
-
     if (p_input)
         vlc_object_release(p_input);
 



More information about the vlc-commits mailing list