[mythtv] [PATCH] Add more views to mythgame

Barney Barumba b004908 at bsxl.com
Tue Apr 26 13:25:13 UTC 2005


Jon Wanderer wrote:

>>>Personally, I'm never too keen on hardcoding stuff like this - for 
>>>every person with 11,000 games there's probably someone else with 
>>>100 games or so who doesn't need all these different roots.
>>>      
>>>
>>I am.  I hate having quite so many config options.  There isn't enough 
>>time in the day to figure out how to configure half the stuff now, 
>>it's easier to just go into the code and hard code stuff sometimes...
>>
I'd suggest that config options should satisfy at least these two 
criteria (in addition to whatever configuration purposes they serve):

1. Once you've set them up as you like them, you shouldn't have to go 
there again.
2. They should have sensible defaults, so you can just fire it up and 
use it without having to bother with them at all.

If you have this, then people who have time to play with them can tweak 
the system exactly how they like it, and people that don't can just take 
the defaults and never go there.

>I agree. I put the patch forward because it's simple and enables
>selection on every game attribute without the existing hassle (exiting
>MythGame, going back into the configuration, resetting things and going
>back into MythGame).
>
That's exaclty why I refactored the code in the first place - because 
all that jumping in and out of settings was annoying me =). See point 1 
above.

> 
>Anyway, the patch is a small change that I think improves browsing in
>MythGame.
>
Sure - I have no problem at all with the patch - I was just saying that 
*personally* I prefer config options to hard coding.

> I'll take a poke around at some point to see how difficult it
>would be to add search functionality. 
>
Yes - a search function would be nice. I'm also thinking of a random 
option for when you just can't decide!

Cheers,
Barney




More information about the mythtv-dev mailing list