Re: [PATCH] user32: Fix error handling in MapWindowPoints, ClientToScreen and ScreenToClient and add tests for them.

[Dmitry Timoshkov]

Christian Costa  wrote:
> -BOOL WINAPI ScreenToClient( HWND hwnd, LPPOINT lppnt )
> +BOOL WINAPI ScreenToClient( HWND wnd, LPPOINT point )

Please stick to commonly used 'hwnd' for a window handle, renaming 'lppnt'
to 'point' is also not justified.

[PATCH] user32: Fix error handling in MapWindowPoints, ClientToScreen and ScreenToClient and add tests for them.

[Christian Costa]

Le 20/10/2012 04:06, Dmitry Timoshkov a écrit :
> Christian Costa  wrote:
> > -BOOL WINAPI ScreenToClient( HWND hwnd, LPPOINT lppnt )
> > +BOOL WINAPI ScreenToClient( HWND wnd, LPPOINT point )
>
> Please stick to commonly used 'hwnd' for a window handle, renaming 'lppnt'
> to 'point' is also not justified.

I can revert them but I was told using Microsost variable naming was not 
encouraged.