Re: [PATCH]winecfg: Double click in dlls list to edit overides. (try 4)

[Nikolay Sivov]

On 29.06.2015 12:35, Hao Peng wrote:
> Add a feature to winecfg. User can double click in dlls list to edit
> overides of selected item.
>
> superseded all earlier patches.
>
> programs/winecfg/libraries.c | 14 ++++++++++++++
>   1 file changed, 14 insertions(+)
> -
>            case CBN_DBLCLK:
> -
>                    if (LOWORD(wParam) == IDC_DLLS_LIST)
> -
>                    {
> -
>                        int index;
> -
>                        POINT p;
> -
>                        if (GetCursorPos(&p) && ScreenToClient((HWND)lParam, &p))
> -
>                        {
> -
>                            index = SendDlgItemMessageW(hDlg, IDC_DLLS_LIST,
> -
>                                    LB_ITEMFROMPOINT, 0, MAKELPARAM(p.x, p.y));
> -
>                            if (!HIWORD(index))
> -
>                                on_edit_click(hDlg);
> -
>                        }
> -
>                    }
> -
>                break;

Correct me if I'm wrong, but I don't think you get CBN_DBLCLK unless 
clicked on an item. Quick test with ControlSpy on Windows confirms that. 
And by the time you get CBN_DBLCLK something is always selected I think, 
which makes all that cursor complexity redundant.