Re: [PATCH v3 0/3] MR1825: kernelbase/path.c: UrlFixupW()

[Zebediah Figura (＠zfigura)]

Zebediah Figura (@zfigura) commented about dlls/kernelbase/path.c:
> */
>  HRESULT WINAPI UrlFixupW(PCWSTR url, PWSTR translatedUrl, DWORD maxChars)
>  {
> -    DWORD srcLen;
> +    DWORD srcLen, dstLen, len;
> +    DWORD colon_pos = 0, pos = 0;
> +    wchar_t *helper_str = NULL;
> +    PWSTR save_str = translatedUrl;
> We use WCHAR, not wchar_t.

It'd also be nice to use a consistent style for all of variable names in this function (wrt snake case vs camel case). In new code I think we prefer snake case.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/1825#note_19808