macdrv: implement getting and setting the screen saver state, version 6

C.W. Betts computers57 at hotmail.com
Tue Jan 22 17:33:07 CST 2013


Yet another fix.
On Jan 22, 2013, at 12:57 PM, Ken Thomases <ken at codeweavers.com> wrote:

> On Jan 22, 2013, at 12:22 PM, C.W. Betts wrote:
> 
>> This version should fix the problems you mentioned
> 
> Sorry about this – I should have thought about this last time – but I think you shouldn't fall through in the SPI_SETSCREENSAVEACTIVE case.  It's asking for trouble in the future, if anybody adds another case to the switch statement.
> 
> I think it would be better to just explicitly return FALSE and put in a comment explaining why (like you suggested for the X11 driver).
> 
> Thanks for your persistence!
> 
> -Ken
> 
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-macdrv-implement-SystemParametersInfo-for-the-Mac-Wi.patch
Type: application/octet-stream
Size: 5561 bytes
Desc: not available
URL: <http://www.winehq.org/pipermail/wine-patches/attachments/20130122/b0a8a539/attachment.obj>


More information about the wine-patches mailing list