Spore patch to fix UI and Part descriptions
Andrew Fenn
andrewfenn at gmail.com
Thu Aug 21 22:08:29 CDT 2008
On Thu, Aug 21, 2008 at 9:41 PM, Stefan Dösinger <stefan at codeweavers.com>wrote:
> These changes may be correct, but they need a test. Take a look at
> dlls/d3d9/tests/device.c to see how the other tests look. I guess the
> changes are likely to be correct, but they have to be tested
I'll add a test case and get back to you later then, thanks for the help.
On Thu, Aug 21, 2008 at 9:51 PM, H. Verbeet <hverbeet at gmail.com> wrote:
>
> Not in its current form, there's a C++ comment and trailing spaces in
> there, and the change in SetScissorRect() probably should test if the
> rectangle is empty. Also, if it tests for empty rectangles, it should
> probably also test for rectangles that are (partially) outside the
> screen, etc.
I'll change the code around and add the checking you mentioned.
Thanks for the info.
Regards,
Andrew
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://www.winehq.org/pipermail/wine-devel/attachments/20080822/d7882c12/attachment-0001.htm
More information about the wine-devel
mailing list