Re: [PATCH xcb/proto] screensaver: Use CARD32 encoding for ScreenSaverSuspend 'suspend'
Julien Cristau <[email protected]>
| Newsgroups | gmane.comp.freedesktop.xorg.devel,gmane.comp.freedesktop.xcb |
|---|---|
| Message-ID | <[email protected]> |
I think this belongs on xcb@. 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 <[email protected]> > --- > 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 > > _______________________________________________ > [email protected]: X.Org development > Archives: http://lists.x.org/archives/xorg-devel > Info: https://lists.x.org/mailman/listinfo/xorg-devel _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: https://lists.x.org/mailman/listinfo/xorg-devel