This is the mail archive of the
cygwin-xfree
mailing list for the Cygwin XFree86 project.
Re: bug report/suggested temp. patch: handling bursts of sent keys
- From: Jon TURNEY <jon dot turney at dronecode dot org dot uk>
- To: cygwin-xfree at cygwin dot com
- Cc: mark dot lillibridge at hp dot com, "Yaakov (Cygwin Ports)" <yselkowitz at users dot sourceforge dot net>
- Date: Mon, 29 Mar 2010 18:21:43 +0100
- Subject: Re: bug report/suggested temp. patch: handling bursts of sent keys
- References: <201001102040.o0AKenLB010951@mailhub-pa1.hpl.hp.com> <201001191817.o0JIHoS1030183@mailhub-pa1.hpl.hp.com> <6b9bc6ee1001230517v73231f5cma1503fa473a5d11c@mail.gmail.com> <201001232202.o0NM2HAl027114@mailhub-pa1.hpl.hp.com> <4B703361.3050304@dronecode.org.uk> <qmhy6iwriiw.fsf@ts-rhel4.hpl.hp.com> <4B840ABE.8080106@dronecode.org.uk> <qmh634r9coi.fsf@ts-rhel4.hpl.hp.com>
- Reply-to: cygwin-xfree at cygwin dot com
On 20/03/2010 16:32, Mark Lillibridge wrote:
On February 23, 2010, Jon Turney wrote:
On 13/02/2010 20:24, Mark Lillibridge wrote:
Jon wrote:
Thanks for the patch. Have you actually tested that this resolves your problem?
Yes. Of course, really, really large bursts will still fail, but
they should be very rare.
Perhaps you might try the attached patch, instead, and see if that helps.
Yes, your (previously) attached patch seems to solve the problem
without wasting memory or only working for a finite number of keys.
Nice job. What's the next step towards getting this patch added to the
source/sent upstream/to Xming?
Yaakov,
Can you pull this patch and include it in the next X server release, please.
Sadly this isn't quite a complete solution: there's still the problem in
multiwindow mode that Windows uses a modal loop when resizing/moving a window,
so the X server code gets no chance to do anything then. You can easily
demonstrate this by resizing the frame of an X window, moving the mouse
continuously for a number of seconds, and then notice that X window doesn't
react to the queued size changes until the mouse button is released.
The following changes since commit 579715f830fbbca9e1ecb17dc18176132f5969e7:
Rami Ylimaki (1):
os: Prevent backtrace from being stopped in noreturn functions.
are available in the git repository at:
git://anongit.freedesktop.org/~jturney/xserver for-yaakov
Jon TURNEY (1):
Cygwin/X: Process one Windows message per wakeup, rather than all of them.
hw/xwin/winblock.c | 23 ++++-------------------
hw/xwin/winwakeup.c | 4 ++--
2 files changed, 6 insertions(+), 21 deletions(-)
--
Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple
Problem reports: http://cygwin.com/problems.html
Documentation: http://x.cygwin.com/docs/
FAQ: http://x.cygwin.com/docs/faq/