<div dir="ltr">I agree with removing the width and height params and handling the checks from inside device_init itself. I'll put it accordingly.<br><br><div><div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
You can move the DDSCAPS_3DDEVICE check from d3drm1_CreateDeviceFromSurface into device_init as well, that way you don't need to fetch the surface description twice.<br></blockquote><div>This could still lead to a redundant check for DDSCAPS_3DDEVICE in the case of CreateDeviceFromClipper, as we're creating the surface internally so we know that this flag will be set always. Is this overhead acceptable? If yes then I have no problem in shifting the check there.<br> <br></div></div></div></div></div></div>