There is no need to offset the source rectangle in the reversedirection before scrolling

Rein Klazes wijn at wanadoo.nl
Thu Dec 15 05:49:38 CST 2005


On Wed, 14 Dec 2005 22:42:28 +0800, you wrote:


>Well, the committed patch 1) does fix a visual glitch in the app I'm working on
>and 2) passes all the current tests.

1) You know that doesn't make it right. I see it causing visual glitches
in *three* programs here, girotel,  native hh.exe and wine's built-in
notepad, which does not make it wrong either
2) Indeed, you managed to break functionality that never broke before.

>
>There are a couple of problems with your tests and fixes

It shows a problem. If you could create a test, or otherwise point to a
way to reproduce your problem I would not be arguing but looking for a
way to fix them all.

>: it tests
>child windows with CS_PARENTDC style set, and it doesn't test the case
>when clipping rect is NULL. In my case both cases are not fulfilled and
>therefore we see different results.

I still feel you do not understand what you 'fixed':

- clipping is plain MSDN specified functionality. Any undocumented
restrictions of the dc to a CS_PARENTDC inhered one is unlikely;
- if the *special* case cliprect==NULL is not handled correctly, then
the correct fix is unlikely to be not to clip in *all* cases.

Rein.



More information about the wine-devel mailing list