[PATCH, try 2] ole32: fix wrong timeout check

Alexander Dorofeyev alexd4 at inbox.lv
Mon Dec 24 04:23:06 CST 2007


Thank you for suggestion, I will note that. Patch updated.

Alexandre Julliard wrote:
    > Alexander Dorofeyev <alexd4 at inbox.lv> writes:
    >
    >> Yeah these calculations are a headache. I could probably write
    >>
    >> now < start_time || now >= start_time + dwTimeout
    >
    > The right way is to write
    >
    >  now - start_time > dwTimeout
    >
    > using unsigned variables everywhere. Then you don't have to worry about
    > overflows, and you don't even need to special case INFINITE.
    >


---
     dlls/ole32/compobj.c |    2 +-
     1 files changed, 1 insertions(+), 1 deletions(-)



-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0de9941f9b9d31bf5df397b25991767564e39df2.diff
Type: text/x-patch
Size: 482 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-patches/attachments/20071224/398f6d3c/attachment.bin 


More information about the wine-patches mailing list