On Fri, Nov 14, 2008 at 01:28:33PM +1100, Daniel Stone wrote:
> grep for 'long-standing misunderstanding': Wacom will always define a
> keyboard feedback class, but not necessarily a key class. XkbHandleBell
> expects a key class to exist as well as a keyboard feedback class, so if
> we have a Wacom tablet with no keys, a bell will cause the server to
> explode. I guess strengthening the test to (!dev->kbdfeed || !dev->key)
> should solve this.
(Tangentially, not sure why it defines feedback classes, as it doesn't
do anything with keyboard feedback, bells, or LED feedback. Oh well.)
On Fri, Nov 14, 2008 at 01:28:33PM +1100, Daniel Stone wrote:
> grep for 'long-standing misunderstanding': Wacom will always define a
> keyboard feedback class, but not necessarily a key class. XkbHandleBell
> expects a key class to exist as well as a keyboard feedback class, so if
> we have a Wacom tablet with no keys, a bell will cause the server to
> explode. I guess strengthening the test to (!dev->kbdfeed || !dev->key)
> should solve this.
(Tangentially, not sure why it defines feedback classes, as it doesn't
do anything with keyboard feedback, bells, or LED feedback. Oh well.)