[Mesa-dev] [PATCH 04/10] swrast: remove non-applicable GLX_SWAP_COPY_OML comment

Ian Romanick idr at freedesktop.org
Tue Dec 12 00:19:22 UTC 2017


On 12/07/2017 09:07 AM, Emil Velikov wrote:
> From: Emil Velikov <emil.velikov at collabora.com>
> 
> Noticed while skimming for GLX_ instances i the dri codebase.

                                            in

With that fixed, this patch is also

Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

> Comment is completely off and was in such a state since day 1.
> 
> Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
> ---
>  src/mesa/drivers/dri/swrast/swrast.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/src/mesa/drivers/dri/swrast/swrast.c b/src/mesa/drivers/dri/swrast/swrast.c
> index 4bee01ad308..8084be64393 100644
> --- a/src/mesa/drivers/dri/swrast/swrast.c
> +++ b/src/mesa/drivers/dri/swrast/swrast.c
> @@ -223,9 +223,6 @@ swrastFillInModes(__DRIscreen *psp,
>      unsigned back_buffer_factor;
>      mesa_format format;
>  
> -    /* GLX_SWAP_COPY_OML is only supported because the Intel driver doesn't
> -     * support pageflipping at all.
> -     */
>      static const GLenum back_buffer_modes[] = {
>  	__DRI_ATTRIB_SWAP_NONE, __DRI_ATTRIB_SWAP_UNDEFINED
>      };
> 



More information about the mesa-dev mailing list