Re: user32: WM_SETFONT on button shouldn't repaint directly

2007-01-23 Thread Dmitry Timoshkov
"Clinton Stimpson" <[EMAIL PROTECTED]> wrote: -if (lParam) paint_button( hWnd, btn_type, ODA_DRAWENTIRE ); +if (LOWORD(lParam)) InvalidateRect(hWnd, NULL, TRUE); Your test doesn't show that the usage of LOWORD(lParam) is justified. -- Dmitry.

Re: user32: WM_SETFONT on button shouldn't repaint directly

2007-01-23 Thread Alexandre Julliard
Clinton Stimpson <[EMAIL PROTECTED]> writes: > Anything wrong with this patch? It hasn't been accepted. I suspect you need an UpdateWindow() too. To confirm this (or not) you should avoid the PeekMessage loop in your test. -- Alexandre Julliard [EMAIL PROTECTED]

Re: user32: WM_SETFONT on button shouldn't repaint directly

2007-01-22 Thread Clinton Stimpson
Anything wrong with this patch? It hasn't been accepted. Thanks, Clint Clinton Stimpson wrote: This patch includes a couple improvements over the patch submitted yesterday. Fix WM_SETFONT on a button to invalidate the rect instead of painting it directly. Test included. Thanks, Clinton