On 2013-05-06 13:24, Sergey Guralnik wrote:
Alexandre Julliard writes:
... It doesn't make sense to have all these complicated tests if we are going to always repaint anyway, which is essentially what your patch does. There has to be more to it than that.
This version uses the same logic as used before, but also it removes all redundant checks.
dlls/user32/tests/win.c | 1 - server/window.c | 32 ++++++-------------------------- 2 files changed, 6 insertions(+), 27 deletions(-)
Is there some problem with this patch?
Sergey Guralnik serhio@etersoft.ru writes:
On 2013-05-06 13:24, Sergey Guralnik wrote:
Alexandre Julliard writes:
... It doesn't make sense to have all these complicated tests if we are going to always repaint anyway, which is essentially what your patch does. There has to be more to it than that.
This version uses the same logic as used before, but also it removes all redundant checks.
dlls/user32/tests/win.c | 1 - server/window.c | 32 ++++++-------------------------- 2 files changed, 6 insertions(+), 27 deletions(-)
Is there some problem with this patch?
It seems a bit extreme to always redraw everything, especially since we go to the trouble of moving the existing bits. I'd like to see some more convincing tests.
-
Alexandre Julliard -
Sergey Guralnik