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?
>
> Thanks for the comment. I'm not sure I really see how use of a
> preincrement operator is that convoluted, but if it's felt strongly
> about I'll change it.

Probably I should have said not obvious instead, sorry for the "strong"
incantation :)