[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