Re: mshtml: fix MSVC build in htmllocation.c

[Nikolay Sivov]

On 11/17/2011 15:18, Thomas Faber wrote:
> This removes the use of two Variable Length Arrays.
> Instead of filling a local buffer, then allocating a new one using
> SysAllocString[Len], the strings are now directly written to the latter
> buffer.
> Note that the 'len' variable in hunk 2 does not need to count in the
> terminating null in this case because SysAllocStringLen automatically
> adds one.
> -        WCHAR buf[url.dwSchemeLength + 1];
> +        WCHAR *buf;
> +        buf = *p = SysAllocStringLen(NULL, url.dwSchemeLength + 1);
>           memcpy(buf, url.lpszScheme, url.dwSchemeLength * sizeof(WCHAR));
>           buf[url.dwSchemeLength] = ':';
> -        *p = SysAllocStringLen(buf, url.dwSchemeLength + 1);
> You don't need 'buf' variable after that.