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.