[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