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

Jacek Caban jacek at codeweavers.com
Fri Mar 24 07:02:12 CDT 2017


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




More information about the wine-devel mailing list