[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