Re: [PATCH v2] winex11: Use GLX_OML_sync_control to synchronise buffer swaps with X11DRV_FLUSH_GL_DRAWABLE requests.

[Jacek Caban]

Hi Henri,

On 23.03.2017 23:58, Henri Verbeet wrote:
> diff --git a/dlls/winex11.drv/init.c b/dlls/winex11.drv/init.c
> index af2fd92..7efd603 100644
> --- a/dlls/winex11.drv/init.c
> +++ b/dlls/winex11.drv/init.c
> @@ -375,9 +375,6 @@ static INT X11DRV_ExtEscape( PHYSDEV dev, INT escape, INT in_count, LPCVOID in_d
>                      RECT rect = physDev->dc_rect;
>                  OffsetRect( &rect, -physDev->dc_rect.left, -physDev->dc_rect.top );
> -
>                /* The GL drawable may be lagged behind if we don't flush first, so
> -
>                 * flush the display make sure we copy up-to-date data */
> -
>                XFlush( gdi_display );

Shouldn't this be called if GLX_OML_sync_control is not available? It
seems to me that it should either depend on extension availability or be
a separated patch.

Thanks,

Jacek