[prev in list] [next in list] [prev in thread] [next in thread]
List: wine-devel
Subject: Re: PATCH: User32: Replicate Windows behavior of WM_SETTEXT handler
From: Vitaliy Margolen <wine-devel () kievinfo ! com>
Date: 2011-01-28 15:11:06
Message-ID: 4D42DC8A.5090303 () kievinfo ! com
[Download RAW message or body]
On 01/27/2011 11:51 PM, Alexander LAW wrote:
>
>
> + if (style & WS_VISIBLE) {
> + HDC hdc = GetDC(hWnd);
> + HBRUSH hbrush;
> + RECT client, rc;
> + HWND parent = GetParent(hWnd);
Please follow the style of the file - curly brace goes on a separate line.
Please if you can't set your editor to have tab character equal 8 spaces -
don't use tabs! Use space characters only.
> + if (!parent) parent = hWnd;
> + UINT ctlMessage=((btn_type==BS_PUSHBUTTON ||
> btn_type==BS_DEFPUSHBUTTON || btn_type==BS_PUSHLIKE)?WM_CTLCOLORBTN:
> WM_CTLCOLORSTATIC);
This is not c++ inline declarations are not allowed.
Please cut this line down it's too long.
Also please add corresponding test to msg.c tests to validate the change.
Vitaliy.
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic