msxml3: Add ISAXContentHandler_endPrefix event (1/7)

Alexandre Julliard julliard at winehq.org
Tue Aug 5 03:53:10 CDT 2008


Piotr Caban <piotr.caban at gmail.com> writes:

> +static HRESULT namespacePush(saxlocator *locator, int ns)
> +{
> +    if(locator->nsStackLast+1>locator->nsStackSize)
> +    {
> +        locator->nsStackSize += STACKALLOCATIONSIZE;
> +        locator->nsStack = HeapReAlloc(GetProcessHeap(), 0,
> +                locator->nsStack, locator->nsStackSize);
> +        if(!locator->nsStack) return E_FAIL;

You should preserve the existing stack on failure. Also the growth
should be by a multiple of the current size instead of a constant.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list