On 10 March 2011 20:22, Adam Martinson amartinson@codeweavers.com wrote:
context_validate_onscreen_formats(device, context, NULL);
/* It's silly to call context_validate_onscreen_formats() with a NULL depth_stencil. */
Sigh. I've asked you a very simple question about the performance impact of these changes *twice* now, and you ignored the question both times.
Unless you have evidence that the compiler fails to eliminate the call by itself, and this actually makes any kind of difference, consistency is more important. The same goes for the IWineD3DDeviceImpl_MarkStateDirty() change, though at least in that case there's somewhat of a remote chance of it making a difference.