On Fri, Jul 3, 2009 at 13:41:05 +1000, Peter Hutterer wrote: > case XI_ProximityIn: > case XI_ProximityOut: > + if (!dev->proximity) > + { > + client->errorValue = ev->u.u.type; > + return BadValue; > + } > + break; > break; No need to add a second break here :) Cheers, Julien