[mythtv] [PATCH] Display Resolution switching (Version 3)

John Patrick Poet john at BlueSkyTours.com
Tue Aug 3 21:06:08 EDT 2004


Isaac Richards wrote:

>On Thursday 29 July 2004 12:07 am, John Patrick Poet wrote:
>  
>
>>This is version 3 of my Display Resolution switching patch.
>>
>>The patch enables automatic switching to different display resolutions
>>based on the dimensions of the video being played.  If USING_XRANDR is
>>enabled in settings.pro, a new config screen will show up under
>>[Appearance].  This config screen allows you to specify the display
>>resolution to use for the GUI, a default video display resolution, and
>>two "override" video display resolutions.
>>    
>>
>
>This is looking much better, but still has some issues I'd like to see taken 
>care of before I commit:
>
>- It won't compile at all without xrandr
>- It won't compile at all if using_x11 isn't defined
>- Why is everything in DisplayRes static?  I don't see a reason for that..
>- Why is the DisplayRes class a public member of the context?  I think it'd 
>live best as a singleton class (see some recent mediamonitor changes), but at 
>the least it should be a private member with an accessor function.
>
>Other suggestions:
>- I'd make the display res class grab its own settings, and not feed em in 
>through Activate() & addRes().
>- I would _really_ prefer it if your patch followed the rest of the style 
>conventions ('if' is not a function, 4 space indents, etc).  Otherwise, I 
>have to spend time fixing things as I apply it.
>
>Isaac
>  
>

Third times the charm?

Here is version three of the Display Resolution Switching patch.  I 
believe it addresses all of your concerns.

John

-------------- next part --------------
A non-text attachment was scrubbed...
Name: DisplayResV3b.patch.bz2
Type: application/x-bzip2
Size: 10603 bytes
Desc: not available
Url : http://mythtv.org/pipermail/mythtv-dev/attachments/20040803/8d915866/DisplayResV3b.patch.bin


More information about the mythtv-dev mailing list