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