On Wed, Oct 09, 2013 at 05:16:18PM -0500, Strake wrote: > Segfault otherwise. > > From 7bc2543ee36e46fedfea3f0af6042be3667e88d2 Mon Sep 17 00:00:00 2001 > From: Strake <[email protected]> > Date: Wed, 9 Oct 2013 11:36:34 -0500 > Subject: [PATCH] unbreak KdEnableInput >
please sign off your patches, see http://www.x.org/wiki/Development/Documentation/SubmittingPatches/ > --- > hw/kdrive/src/kinput.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/hw/kdrive/src/kinput.c b/hw/kdrive/src/kinput.c > index 527c7a2..c49f2e0 100644 > --- a/hw/kdrive/src/kinput.c > +++ b/hw/kdrive/src/kinput.c > @@ -320,8 +320,8 @@ KdEnableInput(void) > > /* reset screen saver */ > ev.any.time = GetTimeInMillis(); > - NoticeEventTime(&ev, pi->dixdev); > - NoticeEventTime(&ev, ki->dixdev); > + if (kdPointers) NoticeEventTime(&ev, kdPointers ->dixdev); no space before -> > + if (kdKeyboards) NoticeEventTime(&ev, kdKeyboards->dixdev); shouldn't this be moved up into the respective loop? otherwise you're only updating the event time for the first device in the list. Cheers, Peter > > OsReleaseSIGIO(); > } > -- > 1.7.11.1 _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
