<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 20, 2016 at 3:11 PM, Aaryaman Vasishta <span dir="ltr"><<a href="mailto:jem456.vasishta@gmail.com" target="_blank">jem456.vasishta@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><div><div>On Mon, Jun 20, 2016 at 2:07 PM, Henri Verbeet <span dir="ltr"><<a href="mailto:hverbeet@gmail.com" target="_blank">hverbeet@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span>On 19 June 2016 at 21:21, Aaryaman Vasishta <<a href="mailto:jem456.vasishta@gmail.com" target="_blank">jem456.vasishta@gmail.com</a>> wrote:<br>
> There's a few test failures happening here because of QI and GetParent not<br>
> being updated for version 1. I am resending this patch along with other<br>
> changes. Sorry for not catching it earlier :/<br>
><br>
</span>Which tests are that? The changes make sense, but the previous version<br>
passed the tests for me.<br></blockquote></div></div><div>The GetParent tests for version 1 frames, and the QI tests afterwards for the same. The previous version included the updated GetParent and QI, but it slipped by after I added the stubs on top of the rearranged v2 functions. I have sent a v6 of this patch on wine-patches, it will show the changes that I meant.<br></div></div><br></div></div></blockquote><div>Specifically. pFrameTmp == pFrameC. pFrameTmp == pFrameP1, pFrameTmp == pFrameP2 tests etc are failing. Also, some vtable tests in test_frame_qi, after they're applied, fail as well. V6 of this patch fixes them. <br></div></div><br></div></div>