[mythtv-commits] mythtv commit: r27287 - in trunk/mythplugins/mythvideo by robertm

[mythtv at cvs.mythtv.org]

      Author: robertm
        Date: 2010-11-18 23:05:43 +0000 (Thu, 18 Nov 2010)
New Revision: 27287
   Changeset: http://svn.mythtv.org/trac/changeset/27287

Log:

Unify Blu-ray playback and DVD under a single menu item, 'Play Optical Disc.'  Unfortunately requires adding a setting in Utilities->Setup->Media Settings->Video Settings called 'Blu-ray Mountpoint.'  However, at the same time I am removing two settings, those pertaining to VCD playback.  VCD playback has never been natively supported in MythTV, all that these settings, and their associated code did was call mplayer.  What was done in hundreds of lines of code is easily done in six lines of menu theme code.  There are probably a few ways that this Play Optical Disc code could work a little better, but at least now it's in the UI and fairly consistent.  The code relies on your system to be auto-mounting your Blu-ray titles.  Welcome improvements would be something like a (cross-platform) command to make sure the disc gets mounted if not auto-mounted.

Modified:

   trunk/mythplugins/mythvideo/AUTHORS
   trunk/mythplugins/mythvideo/mythvideo/main.cpp
   trunk/mythplugins/mythvideo/mythvideo/playersettings.cpp
   trunk/mythplugins/mythvideo/mythvideo/playersettings.h
   trunk/mythplugins/mythvideo/theme/default-wide/video-ui.xml
   trunk/mythplugins/mythvideo/theme/default/video-ui.xml
   trunk/mythplugins/mythvideo/theme/menus/video_settings.xml
   trunk/mythplugins/mythvideo/theme/menus/videomenu.xml