[PATCH xcb/proto] screensaver: Use CARD32 encoding for ScreenSaverSuspend 'suspend'

Julien Cristau jcristau at debian.org
Tue Mar 20 19:56:30 UTC 2018


I think this belongs on xcb at .

Cheers,
Julien

On Mon, Mar 12, 2018 at 12:03:19 -0700, Keith Packard wrote:

> This was set to BOOL, but the protocol headers used Bool, presumably a
> 32-bit type. We're switching everything to CARD32 as the best option
> for compatibility.
> 
> Signed-off-by: Keith Packard <keithp at keithp.com>
> ---
>  src/screensaver.xml | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/src/screensaver.xml b/src/screensaver.xml
> index 8d5abb4..c546f94 100644
> --- a/src/screensaver.xml
> +++ b/src/screensaver.xml
> @@ -168,8 +168,7 @@ Draft Standard Version 1.1
>    
>    <!-- version 1.1 -->
>    <request name="Suspend" opcode="5">
> -    <field type="BOOL" name="suspend" />
> -    <pad bytes="3" />
> +    <field type="CARD32" name="suspend" />
>    </request>
>  
>    <!-- Events -->
> -- 
> 2.16.2
> 
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: https://lists.x.org/mailman/listinfo/xorg-devel


More information about the xorg-devel mailing list