Sergey Guralnik <ser...@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 julli...@winehq.org
