<div dir="ltr">Hi Andrew,<div><br></div><div>Thanks for review! I sent another try.</div></div><div class="gmail_extra"><br><div class="gmail_quote">2017-03-27 21:49 GMT+08:00 Andrew Eikum <span dir="ltr"><<a href="mailto:aeikum@codeweavers.com" target="_blank">aeikum@codeweavers.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sun, Mar 26, 2017 at 04:31:55PM +0800, Jactry Zeng wrote:<br>
> +    res = ICreateDevEnum_<wbr>CreateClassEnumerator(create_<wbr>devenum, &CLSID_LegacyAmFilterCategory, &enum_moniker0, 0);<br>
> +    ok(SUCCEEDED(res), "Cannot create enum moniker (res = %x)\n", res);<br>
> +    res = ICreateDevEnum_<wbr>CreateClassEnumerator(create_<wbr>devenum, &CLSID_LegacyAmFilterCategory, &enum_moniker1, 0);<br>
> +    ok(SUCCEEDED(res), "Cannot create enum moniker (res = %x)\n", res);<br>
> +    if (!res)<br>
> +    {<br>
> +        if (!IEnumMoniker_Next(enum_<wbr>moniker0, 1, &moniker0, NULL) && !IEnumMoniker_Next(enum_<wbr>moniker1, 1, &moniker1, NULL))<br>
> +        {<br>
> +            res = IMoniker_IsEqual(moniker0, moniker1);<br>
> +            ok(!res, "IMoniker_IsEqual should failed (res = %x)\n", res);<br>
> +<br>
> +            res = IMoniker_IsEqual(moniker1, moniker0);<br>
> +            ok(!res, "IMoniker_IsEqual should failed (res = %x)\n", res);<br>
> +<br>
> +            IMoniker_Release(moniker0);<br>
> +            IMoniker_Release(moniker1);<br>
> +        }<br>
<br>
These patches look fine to me except the ok() message is wrong on these two.<br>
<br>
Also, I think it's easier to read code that explicitly compares against S_OK or<br>
uses SUCCEEDED() instead of using !.<br>
<span class="HOEnZb"><font color="#888888"><br>
Andrew<br>
<br>
<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Jactry Zeng</div><div><br></div></div></div>
</div>