Re: [PATCH v2 0/5] MR3062: wine{alsa,coreaudio,oss,pulse}: Move get_application_name into mmdevapi.

[Huw Davies (＠huw)]

Huw Davies (@huw) commented about dlls/mmdevapi/client.c:
> -
>    goto skip;
> -
> - if (!VerQueryValueW(data, L"\VarFileInfo\Translation", (LPVOID *)&translate, &translate_size))
> -
>    goto skip;
> -
> - /* No translations found. */
> - if (translate_size < sizeof(LANGANDCODEPAGE))
> -
>    goto skip;
> -
> - /* The following code will try to find the best translation. We first search for an
> -
> * exact match of the language, then a match of the language PRIMARYLANGID, then we
> -
> * search for a LANG_NEUTRAL match, and if that still doesn't work we pick the
> -
> * first entry which contains a proper productname. */
> - locale = GetThreadLocale();
> -
> - for (i = 0; i < translate_size / sizeof(LANGANDCODEPAGE); ++i) {

Likewise `i++` -> `++i` (I personally prefer the former anyway, but in any case there's no need to change it from the original).

However, I don't mind if brace placements are fixed up so that the function is consistent.