[mythtv-commits] Ticket #324: Patch to move LCD control to new mythlcdserver

[MythTV]

#324: Patch to move LCD control to new mythlcdserver
-------------------------+--------------------------------------------------
 Reporter:  Paul         |       Owner:  ijr 
     Type:  enhancement  |      Status:  new 
 Priority:  minor        |   Milestone:  0.19
Component:  mythtv       |     Version:  head
 Severity:  medium       |         Cc:               |  
-------------------------+--------------------------------------------------
 This patch fixes a couple of problems with the current
 implementation of the LCD output.

 1. If you have more than one program that creates an LCD object
 to update the LCD they each create a new set of screens and widgets
 which causes the LCD to display strange sequences of screens. The LCDd
 server isn't smart enough to recognise the screens were created using the
 same
 client name and screen names etc.

 2. If you start another process from the frontend like using an external
 player in
 MythVideo or MythDVD or start MythBrowser for example it causes the LCD
 display to freeze until control is passed back to the frontend.

 This patch fixes both problems by moving the control of the LCD to a
 separate process 'mythlcdserver'. Clients talk to the server though a
 socket.
 The public interface to the LCD class is the same so no need to change
 any of the plugins or mythtv to use it. The LCD class will start the
 mythlcdserver
 if necessary.

 There is another advantage to using a separate process to look after the
 LCD, it doesn't require the frontend or even the backend to be running for
 it to work so
 can be started early in the boot process if you want. There is even a
 parameter you
 can pass to the mythlcdserver that will display a start-up message for a
 set time or until a client is connected. Better than displaying the user
 unfriendly LCDproc
 server screen I think.

 There is two things I can't test - the set LED's stuff and the key press
 support.
 My LCD doesn't support either of them so if someone could test and report
 back.

 Should also fix #280 & #317.

 Paul

-- 
Ticket URL: <http://cvs.mythtv.org/trac/ticket/324>
MythTV <http://www.mythtv.org/>
MythTV