[PATCH 2/5] Move AllowTouch to dix/touch.c

Peter Hutterer peter.hutterer at who-t.net
Fri Feb 3 00:09:33 PST 2012


On Fri, Feb 03, 2012 at 06:36:55AM +0100, Chase Douglas wrote:
> On 02/03/2012 06:30 AM, Peter Hutterer wrote:
> > On Thu, Feb 02, 2012 at 04:57:55PM -0800, Chase Douglas wrote:
> >> From: Chase Douglas <chase.douglas at ubuntu.com>
> >>
> >> Signed-off-by: Chase Douglas <chase.douglas at canonical.com>
> > 
> > if it's in touch.c, it should be prefixed with "Touch", so TouchAllow. other
> > than that, Reviewed-by: Peter Hutterer <peter.hutterer at who-t.net>
> 
> The other function this is patterned off of is AllowSome(), which in
> retrospect is a bit ambiguous, in dix/events.c. Do you think we should
> keep that pattern or rename this TouchAllow()?

tbh, I'd rather have all the touch related functions prefixed, rename it to
TouchAcceptReject() if you want to to be more explicit.

Cheers,
  Peter


More information about the xorg-devel mailing list