[mythtv] [PATCH] Massive LCD Support Update

Isaac Richards ijr at po.cwru.edu
Mon Aug 18 19:36:33 EDT 2003

On Monday 18 August 2003 06:13 pm, DanM wrote:
> I matched the indentation in lcddevice.* per the recomendation, I
> presume you changed it from using tab to using spaces?  What bug did you
> fix in themedmenu?  I'll do a clean checkout tonight and make sure
> everything still works fine.

Yeah, I got tired of the existing indentation style in there while I was 
applying your patch, so I fixed it to be the One True Indentation Style.  Bug 
in themedmenu was you were assuming all rows had the same number of columes.  
Just a minor little thing, really.  Oh, and I should mention that I didn't 
end up using the additional lcd*.cpp files -- I just incorporated them into 
the lcddevice header, as the two classes were just simple data types.

> My future plans for this is make the LCD stuff into a daemon and make
> lcddevice connect to the socket , so there isn't any contention between
> say MTD holding the display, preventing mythfrontend from updating it;
> mythbackend can turn on a LED or 3 when something important happens (Say
> it start recording).  Does all this sound fine?  Or do you have a
> suggestion as to a better way of doing this?

That sounds fine to me, but as I don't have a LCD display, I'm rather 
ambivalent about it all =)


More information about the mythtv-dev mailing list