[Xcb] xproto: add any modifier for key?

Barton C Massey bart at cs.pdx.edu
Wed Aug 27 09:26:19 PDT 2008


In message <20080827101851.GF14183 at abydos.adm.naquadah.org> you wrote:
> 
> --N8ia4yKhAKKETby7
> Content-Type: text/plain; charset=utf-8
> Content-Disposition: inline
> Content-Transfer-Encoding: quoted-printable
> 
> At 1219202712 time_t, Barton C Massey wrote:
> > The protocol specification has both AnyButton and AnyModifier,
> > so your patch seems like the correct one rather than the
> > Xlib approach.  It's confused in the protocol by the fact
> > that the high bit in a mask is supposed to be reserved for
> > future unspecified "chaining".
> 
> And be careful that AnyButton is not a mask, it's 0L, whereas
> AnyModifier is a mask.
> AnyModifier in XCB is XCB_BUTTON_MASK_ANY and my patch add XCB_MOD_MASK_ANY
> as an identical mask and value.
> 
> > I'm not quite sure whether the protocol names should have
> > been mangled in this way in the first place, though: it
> > seems like XCB_ANY_MODIFIER and XCB_ANYBUTTON or somesuch
> > would be better?  Can someone comment on what they think of
> > all that?
> 
> IMHO, XCB_BUTTON_MASK_ANY should not exist and XCB_MOD_MASK_ANY should.
> 
> But having both can't harm.
> 
> I'll push that unless someone stop me in the next hours.

I need to look at the protocol spec again to understand what
the right thing to do with AnyButton is; Xlib seems a bit
confused here.

Thanks much for the patches!

   Bart


More information about the Xcb mailing list