Re: [PATCH 0/1] MR4794: comctl32/combo: Add tests for keypresses showing search functionality

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/comctl32/tests/combo.c:
>  DestroyWindow(combo);
>
> }
>
> +static void get_selected_value(HWND combo, char* selected)
> +{
> - int index;
> - selected[0] = 0;
> - index = SendMessageA(combo, CB_GETCURSEL, 0, 0);
> - SendMessageA(combo, CB_GETLBTEXT, index, (LPARAM)selected);
>
> +}
> -
>
> +static void test_combo_keypresses(BOOL sort)
> +{
> - HWND combo = create_combobox(CBS_DROPDOWNLIST | (sort ? CBS_SORT : 0));

If this really needs a function argument, which is questionable, it might as well use style bit directly as that argument.