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/

Reply via email to