
<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Hi Martin,<div class=""><br class=""></div><div class="">As far as my experience goes(which is less), the usage of SequencedEvents is relatively rare, and rarer even to generate a high volume of SequencedEvents.</div><div class="">So, I would be inclined to pursue the path of blocking the dispatch till the SequencedEvents are processed.</div><div class="">Probably Sergey/Phil are more qualified to comment on this, but this is my 2 cents!</div><div class=""><br class=""></div><div class="">Thanks,</div><div class="">Krishna<br class=""><div><br class=""><blockquote type="cite" class=""><div class="">On 18-Oct-2018, at 2:15 PM, Martin Balao <<a href="mailto:mbalao@redhat.com" class="">mbalao@redhat.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class=""><div dir="ltr" class="">Hi Laurent,<br class=""><div class="gmail_extra"><br class=""><div class="gmail_quote">On Wed, Oct 17, 2018 at 8:13 PM, Laurent BourgÃ¨s <span dir="ltr" class=""><<a href="mailto:bourges.laurent@gmail.com" target="_blank" class="">bourges.laurent@gmail.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto" class=""><div dir="auto" class="">If the behavior changed in your patch, it sounds more conservative to discard events (as before) if the present bug is still fixed.</div><div dir="auto" class="">It could be revisited later in another appropriate bug.</div><div dir="auto" class="">Is it a trivial change in your event filter ? I am looking forward trying this alternative sllution.</div></div></blockquote><div class=""><br class=""></div><div class="">I believe that this decision -whether we dispatch, block o completely change the approach- has to be taken as part of this bug. We would need to further investigate, particularly on the SequencedEvent clients side.</div><div class=""><br class=""></div><div class="">Discarding non-SequencedEvent events or holding them for the future is a trivial change. In fact, I tried it some days ago before proposing the latest version of the patch. What I noticed in your test at [1] was: events were injected at a high rate, the SequencedEvent.list list steadily grew, the EDT was very busy dispatching them -"blocked" like- and there was no room for dispatching other events keeping the GUI unresponsive -they were rejected by the waiting filter-. If I remember correctly, repainting after changing the counter was affected by this.</div><div class=""><br class=""></div><div class="">Regards,</div><div class="">Martin.-</div><div class=""><br class=""></div><div class="">--</div><div class="">[1] - <a href="http://mail.openjdk.java.net/pipermail/awt-dev/2018-October/014429.html" class="">http://mail.openjdk.java.net/pipermail/awt-dev/2018-October/014429.html</a></div></div></div></div></div>

</div></blockquote></div><br class=""></div></body></html>