<div dir="ltr">Just to clarify for others: All the patches in the previous email are yet to be pushed - I made a mistake in specifying which patches are already upstream (where I said 0000-0005 are already upstream) two of those patches are not yet upstream, which I've included in the patchset I just sent.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 5, 2015 at 1:19 AM, 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>As discussed on IRC, here's the patches rebased on top of the current upstream's HEAD (as of 8/5/15).<br><br><br></div>Jam<br></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 4, 2015 at 10:02 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>The patch numbers start from 6 because I haven't updated my origin yet. Patches 0 - 5 have already been pushed upstream.<br><br></div>Cheers,<br></div>Jam<div><div><br><div><div><div><div class="gmail_extra"><div class="gmail_quote">On Tue, Aug 4, 2015 at 10:01 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Okay, here's the updated patch with width and height removed from device_init and quite a bit of useless NULL checks and redundant if's removed (as they're no longer needed once the implementations are added) I'll be following a similar approach while sending over CreateDeviceFromD3D's implementation.<br><br></div>I've also removed some redundancy in device_init too, hopefully threre shouldn't be any leaks this time.<br></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 4, 2015 at 3:00 AM, Stefan Dösinger <span dir="ltr"><<a href="mailto:stefandoesinger@gmail.com" target="_blank">stefandoesinger@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA1<br>
<br>
</span><span>Am 2015-08-03 um 23:24 schrieb Aaryaman Vasishta:<br>
> This could still lead to a redundant check for DDSCAPS_3DDEVICE in<br>
>  the case of CreateDeviceFromClipper, as we're creating the surface<br>
>  internally so we know that this flag will be set always. Is this<br>
> overhead acceptable? If yes then I have no problem in shifting the<br>
>  check there.<br>
</span>Yeah, I think that's fine. Better than duplicating the code in two places.<br>
<br>
(And no, I don't think it's worth adding a parameter to skip the check)<br>
<span><br>
<br>
-----BEGIN PGP SIGNATURE-----<br>
Version: GnuPG v2<br>
<br>
</span>iQIcBAEBAgAGBQJVv914AAoJEN0/YqbEcdMw/ZcP/iFB/y+65ok1hQwIleuqtrWQ<br>
wDuTNJvibf7heqXCitoiaebb5QSB8HA5zfrnaFu+10XrzYNX7eCy7k2kXMi1GTGQ<br>
a9i8pAfhcOrPbG9OLUL3g7whkj5A8obrwrXvplm5HwrT3aE0Hrg9xpobB7IrVtrZ<br>
pke5/UWfy9G3ZjoLY+X+uSx5ANDT9DzAjnCqyDQq9zRl32ICB1u2pAlXuw3I0jA0<br>
Popf3cRqY7RvKfz2G+I3nE37eKGCWV8M/uNn14DDDYgNwI48LfMTVajDN45KUkbZ<br>
A/OkusMdzQ4uDI+QyxHk6pPq85zFzb6/vo1ucvf//XxS/5bJrofyoKUGuuraH1H5<br>
l/Pyd956XyW9st457eIOW181OoyWUph8KWXSTUEo79z1BdnjP+UtC7P2bIn/V0MN<br>
PBJeAFnSGrio5Iya8dg8XMg7De2yWP2pOyga+yl8KAkth4ynqE2/Mi5/VdYXJPR/<br>
+hIEb6WE81CTrZSrfRsVCpSDNvYswtFncyk6MvqsObfddlKD7OU4c1FluWiZ7j6z<br>
SjPrqOUmK1IfxhJyPnyjgGmjdDv10d7FhFmI7S8MDM1OcTSlSOPGYgx42STAH4ss<br>
cIsIiizRRVAnuOBjGAL5NxQg5Bg07am1QHsju5XcjKOYc/oOF6xaUtoivOSTylxE<br>
0SUk2FjWQMA7H03ScK5U<br>
=tn/g<br>
-----END PGP SIGNATURE-----<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div></div></div></div></div></div></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>