Hi, On Mon, May 09, 2011 at 06:34:09PM +0200, Cyril Brulebois wrote: > Sorry I'm late but: > > Daniel Stone <dan...@fooishbar.org> (05/05/2011): > > Replace: > > for (stuff; things; etc) { > > if (misc || other) { > > [...] > > } > > } > > with: > > for (stuff; things; etc) { > > if (!misc && !other) > > continue; > > [...] > > } > > that says nothing about that: > > > - if ((tmpd == dev) || (!IsMaster(tmpd) && GetMaster(tmpd, > > MASTER_KEYBOARD) == dev)) { > > + if (tmpd != dev && GetMaster(tmpd, MASTER_KEYBOARD) != dev) > > You're losing one part here; was that on purpose?
Yes. I could probably call it out in the commit message if that would help? Cheers, Daniel _______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel