msi: Expand OLE automation interface parameter handling.
James Hawkins
truiken at gmail.com
Wed Feb 28 00:55:07 CST 2007
On 2/27/07, Misha Koshelev <mk144210 at bcm.tmc.edu> wrote:
> This applies on top of my other OLE automation patches. Added new
> checks:
>
Why don't you wait until the other patches are committed before
sending these new patches? It makes it hard to review, because I
don't have your other patches applied.
> - Parameter we are looking for is not passed?
> - Only set puArgErr if it is passed
> - Make sure pVarResult is valid before we write to it
> - Don't return S_FAIL. MSDN doesn't mention it in the Invoke help, so we
> won't use it.
>
You need to add tests for these items.
--
James Hawkins
More information about the wine-devel
mailing list