[PATCH 1/2] oleaut32/tests: Add test for ITypeInfo2_fnGetContainingTypeLib [try 2]
Huw Davies
huw at codeweavers.com
Thu Nov 8 08:55:15 CST 2012
On Thu, Nov 08, 2012 at 06:13:28PM +0400, Tatyana Fokina wrote:
> + hr = CreateTypeLib2(SYS_WIN32, jsdeb, (ICreateTypeLib2 **)&ctl2);
> + ok_ole_success(hr, CreateTypeLib2);
> +
> + hr = ICreateTypeLib2_CreateTypeInfo(ctl2, jsname, TKIND_DISPATCH, &cti);
> + ok_ole_success(hr, ICreateTypeLib2_CreateTypeInfo);
Please don't use obscure strings like 'jsdeb.tlb' use something
like 'test.tlb'. You don't need the casts here or further down.
> +
> + hr = ICreateTypeInfo_QueryInterface(cti, &IID_ITypeInfo2, (void**)&ti2);
> + ok_ole_success(hr, ICreateTypeInfo2_QueryInterface);
> + if (SUCCEEDED(hr))
> + {
There's no need to put this inside an if block. QI isn't going to fail.
Also your indentation is first four spaces, then two - keep it at four.
Huw.
More information about the wine-devel
mailing list