On Jul 24, 2013, at 9:41 PM, Ken Thomases wrote:
On Jul 24, 2013, at 9:34 PM, Charles Davis wrote:
Try 2: Don't duplicate a constant string; we might need to add more legacy extensions. (Hopefully not. ;)
Thanks for making that change. Looks good to me.
Actually, on further thought I realize that we shouldn't be advertising WGL_EXT_swap_control unconditionally. It should be controlled by the AllowVerticalSync registry setting / allow_vsync variable. See load_extensions(). If vsync is disallowed, the corresponding functions are left as NULL in the OpenGL function table, so clients may crash if WGL_EXT_swap_control is advertised.
Regards, Ken