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

[James Hawkins]

On 5/15/07, Misha Koshelev  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).