[mythtv] [PATCH] Picture Adjust
Ben Levitt
levittben at yahoo.com
Sun Oct 12 14:04:44 EDT 2003
Ah, okay.
I renamed things in a more generic way, and changed
OSDSet::Display() to take the same osdFunctionalType
parameter as OSDSet::DisplayFor() so you can display
any OSDSet and give it a specific type.
Right now only OSD::StartPause() passes in a type to
OSDSet::Display() or OSDSet::DisplayFor(), but if
OSD::SetChannumText(), OSD::SetInfoText(),
OSD::SetSettingsText() etc wanted to pass in a type
they could.
And anyone with a HandleOSDClosed(int) slot can sign
up to get OSDClosed(int) signals.
(The list wouldn't let me post a 40k+ attachment, so
the patch is gzipped...)
Ben
--- Isaac Richards <ijr at po.cwru.edu> wrote:
> On Saturday 11 October 2003 08:12 pm, Ben Levitt
> wrote:
> > Isaac, I'd love to see this go in to 0.12. :] If
> > there's anything else I can do to help that
> happen,
> > please let me know.
>
> I just don't like calling the signal
> 'OSDPauseClosed', stuff like the
> 'currentOSDPauseType' variable in OSDSet, and the
> OSDPauseType enum in osd.h.
> Something like sending a signal when a given OSD
> container is done displaying
> should be generic, not tied to this specific
> feature.
>
> Isaac
__________________________________
Do you Yahoo!?
The New Yahoo! Shopping - with improved product search
http://shopping.yahoo.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pictureAdjust4.patch.gz
Type: application/gzip
Size: 6130 bytes
Desc: pictureAdjust4.patch.gz
Url : http://mythtv.org/pipermail/mythtv-dev/attachments/20031012/e26ac5d6/pictureAdjust4.patch.bin
More information about the mythtv-dev
mailing list