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

Marvin Scholz git at videolan.org
Sun Jun 26 22:56:32 CEST 2016


vlc | branch: master | Marvin Scholz <epirat07 at gmail.com> | Sun Jun 19 12:55:06 2016 +0200| [601c7b718c8d6218ae1fb1fc1c267c1353582652] | 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=601c7b718c8d6218ae1fb1fc1c267c1353582652
---

 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