[PATCH 4/4] glamor: Drop a bunch of glamor_priv == NULL checks.

Michel Dänzer michel at daenzer.net
Tue Jul 7 19:20:07 PDT 2015


On 08.07.2015 09:26, Eric Anholt wrote:
> 
> diff --git a/glamor/glamor_utils.h b/glamor/glamor_utils.h
> index 0a7de82..a923b7a 100644
> --- a/glamor/glamor_utils.h
> +++ b/glamor/glamor_utils.h
> @@ -756,8 +756,8 @@ glamor_translate_boxes(BoxPtr boxes, int nbox, int dx, int dy)
>  						|| _depth_ == 30	\
>  						|| _depth_ == 32)
>  
> -#define GLAMOR_PIXMAP_PRIV_IS_PICTURE(pixmap_priv) (pixmap_priv && pixmap_priv->is_picture == 1)
> -#define GLAMOR_PIXMAP_PRIV_HAS_FBO(pixmap_priv)    (pixmap_priv && pixmap_priv->gl_fbo == GLAMOR_FBO_NORMAL)
> +#define GLAMOR_PIXMAP_PRIV_IS_PICTURE(pixmap_priv) (pixmap_priv->is_picture == 1)
> +#define GLAMOR_PIXMAP_PRIV_HAS_FBO(pixmap_priv)    (pixmap_priv->gl_fbo == GLAMOR_FBO_NORMAL)

I guess these macros could be removed now, but that's for later.

The series is

Reviewed-by: Michel Dänzer <michel.daenzer at amd.com>


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer


More information about the xorg-devel mailing list