Re: [PATCH 1/6] msi: Make MsiGetProperty() RPC-compatible.

[Dmitry Timoshkov]

Zebediah Figura  wrote:
> - r = MsiGetPropertyW(hinst, property, empty, &size);
> - if (r == ERROR_MORE_DATA)
> - {
> -
>    *value = midl_user_allocate(++size * sizeof(WCHAR));

While this construct works as expected could you please avoid this kind
of convoluted thing to make it look less tricky?
> -
>    if (!*value)
> -
>        return ERROR_OUTOFMEMORY;
> -
>    r = MsiGetPropertyW(hinst, property, *value, &size);
> - }
> - return r;
>
> }