mshtml: fix MSVC build in htmllocation.c
bunglehead at gmail.com
Thu Nov 17 13:41:20 CST 2011
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
> 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.
More information about the wine-devel