Re: [PATCH 0/3] MR5264: winewayland.drv: part 13.2: More OpenGL functionality

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/winewayland.drv/opengl.c:
> -
>    return FALSE;
> - }
> -
> - if (!ctx || !ctx->draw)
> - {
> -
>    RtlSetLastWin32Error(ERROR_DC_NOT_FOUND);
> -
>    return FALSE;
> - }
> -
> - pthread_mutex_lock(&gl_object_mutex);
> - if ((ret = p_eglSwapInterval(egl_display, interval)))
> -
>    ctx->draw->swap_interval = interval;
> - else
> -
>    RtlSetLastWin32Error(ERROR_DC_NOT_FOUND);
> - ctx->draw->refresh_swap_interval = FALSE;
> - pthread_mutex_unlock(&gl_object_mutex);

Not sure to see why you need the lock here then if it's indeed safe to read it without a lock.