Am Freitag, 7. September 2007 18:29:02 schrieb Louis. Lenders:
Short answer: Patch is ok with me
Long answer: We can check if opengl supports vsync(which is what this flag does). WineD3D implements vsync using GLX_SGI_video_sync. So if this extension is supported, we should set the flag. If it is not supported, do not set it. Our WGL implementation does not forward the GLX extension yet, so we do not support vsync at the moment, so setting this flag means essentially lying to the app.
However, I think it is safe to pretend vsync support. The worst that may happen is that the game renders with a higher framerate, and the hardware consumes more power instead of beeing idle. I think both cases are better then not running the game at all.
There's one game which runs into internal race conditions if vsync does not work(need for speed 3), but that game doesn't even care to check that flag.