[mythtv] [PATCH] Limit the updates for nonexisting LCD
Kenneth Aafløy
lists at kenneth.aafloy.net
Fri Oct 8 22:12:52 UTC 2004
On Friday 08 October 2004 21:52, Isaac Richards wrote:
> On Friday 08 October 2004 03:33 pm, Kenneth Aafløy wrote:
> > On Friday 08 October 2004 11:54, Tako Schotanus wrote:
> > > Kenneth Aafløy wrote:
> > > > Also, after I had a look at this compile time option I must say I do
> > > > not understand why this was even made a compile time option. I would
> > > > say that what qualifies as a compile time option is a large part of
> > > > code that is either very unstable and/or contains external
> > > > dependecies that not every user would want or need to have. LcdDevice
> > > > is neither, right?
> > >
> > > Sorry, have no idea about the rationale for making it a compile time
> > > option.
> >
> > Do you have an oppinion on this Isaac?
>
> Not really - I think the current stuff could be done better, in that all
> the code in the various modules (eg, mythmusic) generally does a lot of
> extra work that it wouldn't have to if the lcd weren't present. As long as
> there's no additional dependencies, I don't have a problem getting rid of
> it being a compile-time option.
So I'll start making patches like:
if ( (LvdDevice *dev = gContext->GetLcdDevice()) )
{
....
}
which would not affect the current behavior at all, and then in the end send
the patch which really does not create and/or deletes the lcd object if it's
unable to connect to it's server.
> > So if I change the 250 in my patch to a 1000 and remove the secsTo()
> > stuff it would scale nicely as it would be updated less if the load is
> > high.
>
> I would just move the calcSliderPos bit into the existing code that makes
> it only check once a second, personally.
I was thinking why calculate time x is over y, z times a second for such a
minute task as updating the lcd display, which could live well with a bit of
a delay on very high load.
Kenneth
More information about the mythtv-dev
mailing list