[PATCH 1/6] msi: automation: Condense a few loops.

[James Hawkins]

On 5/15/07, Misha Koshelev <mk144210 at bcm.edu> wrote:
> ---
>  dlls/msi/automation.c |   30 ++++++++++++------------------
>  1 files changed, 12 insertions(+), 18 deletions(-)
>

--- a/dlls/msi/automation.c

+++ b/dlls/msi/automation.c

@@ -580,16 +580,13 @@ static HRESULT WINAPI RecordImpl_Invoke(

                 if (FAILED(hr)) return hr;

                 V_VT(pVarResult) = VT_BSTR;

                 V_BSTR(pVarResult) = NULL;

-		ret = MsiRecordGetStringW(This->msiHandle, V_I4(&varg0), NULL, &dwLen);

-                if (ret == ERROR_SUCCESS)

+		if ((ret = MsiRecordGetStringW(This->msiHandle, V_I4(&varg0), NULL,
&dwLen)) == ERROR_SUCCESS)

                 {

Did you mean to mix tabs and spaces in your own file?  You should
probably stick with one or the other (preferably spaces to keep with
the rest of msi).

-- 
James Hawkins