[mythtv] [patch] Reorganized Setup

[Tako Schotanus]

Don't give up, this was surely meant as constructive criticism!

As long as the maintainers themselves aren't telling you "this will
never get in" there's always hope :-)

Just take a look at what people are saying because surely you'll agree
with some of it, then take a good look at the things you don't agree
with and think hard if they might not be right anyway or if there's some
middle ground to be found.

In the end it's much better to spend a bit more time on an accepted
patch then have wasted all that time on a patch that was already "almost
there"!! :-)

Cheers,
 -Tako

> -----Original Message-----
> From: mythtv-dev-bounces at mythtv.org 
> [mailto:mythtv-dev-bounces at mythtv.org] On Behalf Of Kenneth Aafløy
> Sent: dinsdag 20 april 2004 2:00
> To: Development of mythtv
> Subject: Re: [mythtv] [patch] Reorganized Setup
> 
> 
> On Tuesday 20 April 2004 01:44, J. Donavan Stanley wrote:
> > So far, I've only skimmed through things.  Here's my 
> initial impressions:
> >
> > 1) Overall it seems as though you've reduced the number of 
> steps in each
> > "wizard" by making more menu options.  In some cases this is a good
> > thing but in other it's a bit silly, several screens have only two
> > options on them, do they really warrant their own menu item 
> and dialog?
> > 2) It seems to me that the Size/Offset section belongs with other
> > appearance related items, possibly overscan as well.
> > 3) UDP notify port could probably be moved to someplace other than
> > playback, it's only valid during OSD but it really has nothing to do
> > with playback.
> > 4) The file setup_tv_rec_priorities.xml appears to be missing.
> >
> > The biggest drawback I see is #1.  I think you might have 
> gone too far
> > in your quest to reduce the wizard steps...
> 
> Bah, it was worth a shot, patch scrapped.
> 
> -- 
> 
> Kenneth Aafløy
> ke-aa at frisurf.no
> _______________________________________________
> mythtv-dev mailing list
> mythtv-dev at mythtv.org
> http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev
>