[mythtv-commits] Ticket #4877: Eject button on DVD drive disabled

[MythTV]

#4877: Eject button on DVD drive disabled
-------------------------------------------+--------------------------------
 Reporter:  Robin Gilks <g8ecj at gilks.org>  |        Owner:  awithers      
     Type:  defect                         |       Status:  infoneeded_new
 Priority:  minor                          |    Milestone:                
Component:  mythvideo                      |      Version:  0.21-fixes    
 Severity:  low                            |   Resolution:                
  Mlocked:  0                              |  
-------------------------------------------+--------------------------------

Comment(by Robin Gilks <g8ecj at gilks.org>):

 This may be a little too complex a log. The intention was that I'd restart
 the frontend with a DVD in the drive so it would be detected immediately,
 then eject and put in a picture CD and eject that and put in  an audio CD.

 The problem of the tray being closed on its own is now worse - it was
 doing it every time I used the Video/Eject navigation to eject the disk.
 This means that there are several instances of the DVD and Gallery plugins
 being loaded. Note that to change disks I knocked the disk out during the
 1/2 second the tray was open and allowed it to close empty. Bit of a pain
 not having an eject function in the Gallery plugin...

 After removing a DVD it just went back (and stayed) at the main menu
 whereas removing a picture CD caused it to re-enter the Gallery plugin
 which then complained about missing media!!

 Using the button on the drive only works when the drive is empty or
 contains an audio CD.

 Currently using:
 $ mythfrontend --version
 Please include all output in bug reports.
 MythTV Version   : 17639M
 MythTV Branch    : branches/release-0-21-fixes
 Library API      : 0.21.20080304-1
 Network Protocol : 40
 Options compiled in:
  linux release using_oss using_alsa using_backend using_dvb using_frontend
 using_ivtv using_joystick_menu using_lirc using_v4l using_x11 using_xrandr
 using_xv using_xvmc using_xvmcw using_xvmc_vld using_bindings_perl
 using_bindings_python using_opengl using_ffmpeg_threads

 with the following modification from the skamithi patch above from trunk:
 Index: libs/libmythtv/textsubtitleparser.cpp
 ===================================================================
 --- libs/libmythtv/textsubtitleparser.cpp       (revision 17639)
 +++ libs/libmythtv/textsubtitleparser.cpp       (working copy)
 @@ -112,7 +112,10 @@

      subtitle_t *loaded_subs = sub_read_file(&sub_data);
      if (!loaded_subs)
 -        return false;
 +    {
 +       fclose(sub_data.file_ptr);
 +       return false;
 +    }

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/4877#comment:5>
MythTV <http://www.mythtv.org/>
MythTV