[mythtv] [Patch] Dynamic Dialogs

Isaac Richards ijr at po.cwru.edu
Wed Apr 16 15:31:38 EDT 2003


On Wednesday 16 April 2003 01:48 pm, ben wrote:
> I'm working creating menu based inputs in the osd.  This is a little
> patch that will make the dialog functions dynamic.  This patch changes
> the dialog function to require a QStringList of options and a dialog
> type.  I wanted to go ahead and let people try this and see if  these
> changes were acceptable to Isaac before I did the rest of the menu
> stuff.

You're leaking the QStringList -- just pass it in by reference and don't 
create a new QStringList object.  Also, the intention of the 'basedialog' and 
sending in a different name (for each of the dialogs) is so multiple dialogs 
can be shown at once, if the need for that ever arises.

Isaac



More information about the mythtv-dev mailing list