<div dir="ltr"><div class="gmail_extra">Hi,<br>2015-10-03 1:21 GMT+08:00 Henri Verbeet <<a href="mailto:hverbeet@gmail.com" target="_blank">hverbeet@gmail.com</a>>:<br>><br>> On 2 October 2015 at 18:26, Ken Thomases <<a href="mailto:ken@codeweavers.com" target="_blank">ken@codeweavers.com</a>> wrote:<br>> > On Oct 2, 2015, at 2:59 AM, Jactry Zeng <<a href="mailto:jzeng@codeweavers.com" target="_blank">jzeng@codeweavers.com</a>> wrote:<br>> ><br>> >> +    return FlashWindow( pfwi->hwnd, pwfi->dwFlags );<br>> ><br>> > This has a typo. "pwfi" should be "pfwi".  You've fixed it in the next patch, which is why it compiles for you, but you have to not introduce it in the first place. :)<br>> ><br>> It would probably be even better to use a real variable name instead<br>> of something that looks like 4 random characters.<br><br>Thanks! :)</div><div class="gmail_extra">I rename it to "pfinfo" in the newer version.<br><br clear="all"><div><br></div>-- <br><div><div dir="ltr"><div>Regards,<br></div><div>Jactry Zeng</div><div><br></div></div></div>
</div></div>