On Wed, 2017-01-11 at 09:02 -0800, Keith Packard wrote: > Adam Jackson <[email protected]> writes: > > > 80 columns max, please. > > Wrapped.
D'oh, this was in two places but I only commented on the one. > > The conditional is always true, which makes me wonder what > > conditional > > you meant to write instead. > > Good catch; I was just reading the cases where listen_to_client could > return FALSE and yet the client should not be marked ready, and not > looking at the line in AttendClient which was zeroing the > COMM_IGNORED bit. > > Here's a second version: lgtm, merged: remote: E: failed to find patch for rev 785053d033e73d2deb0ded4b97eabfd881991978. remote: I: 0 patch(es) updated to state Accepted. To ssh://git.freedesktop.org/git/xorg/xserver 436da93..785053d master -> master Patch not found in patchwork at minimum because I fixed up the long comment line in include/dixstruct.h as well. Probably also because patchwork sucks at patches sent as attachments. We've got a few serious bugfixes in master since 1.19 at this point, I'll cherry-pick those back and push out a 1.19.1. - ajax _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: https://lists.x.org/mailman/listinfo/xorg-devel
