https://bugs.documentfoundation.org/show_bug.cgi?id=170740

--- Comment #22 from Patrick (volunteer) <[email protected]> ---
(In reply to [email protected] from comment #21)
> I think the patch is just doing pointer comparison on the NSEvents? In my
> case there are two NSEvent pointers getting alternately returned, though
> they both contain exactly the same data. So I did this:

Interesting. I am only seeing one left mouse up events when live resizing via
the bottom right corner of the window and none when Option-dragging a window.

I do agree that pointer comparison isn't going to work as I found that
reposting an event and then immediately fetching it from the native event queue
returns a different pointer so like NSApp makes a copy of the event somewhere.

What happens if you replace the following line in my patch:

                    if ( bRepost )

with the following?:

                    if ( pLastRepostedEvent )

I will take another look at what is occurring in tdf#155092, but what the
repost code is trying to do is to stop recursive event dispatching until the
current event that made the recursive call is finished dispatching. Basically,
this gives the LibreOffice internal resizing and repainting timers a chance to
catch up with the native window's size changes before dispatching the left
mouse up and any events after it.

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to