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 <[email protected]> > >> > >> Signed-off-by: Chase Douglas <[email protected]> > > > > if it's in touch.c, it should be prefixed with "Touch", so TouchAllow. other > > than that, Reviewed-by: Peter Hutterer <[email protected]> > > 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 _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
