On Thu, 17 Oct 2019 15:28:28 GMT, mruzicka 
<github.com+562920+mruzi...@openjdk.org> wrote:

> Secondary event loop introduced as a means of synchronization with the JavaFX 
> Application thread in [1] never terminates as the SecondaryLoop.exit() call 
> is not reached because the thread is blocked in the SecondaryLoop.enter() 
> call.
> This patch fixes the problem by submitting the UI work (including the call to 
> the SecondaryLoop.exit() method) before entering the secondary loop.
> 
> [1] 
> https://github.com/openjdk/jfx/commit/7cf2dfa0b3c5bfd0f1a2de36d46b62f7e9e256c4
> 
> ----------------
> 
> Commits:
>  - 9483ccde: 8231372: Correctly terminate secondary event loop in 
> JFXPanel.setScene()
> 
> Changes: https://git.openjdk.java.net/jfx/pull/16/files
>  Webrev: https://webrevs.openjdk.java.net/jfx/16/webrev.00
>   Issue: https://bugs.openjdk.java.net/browse/JDK-8231372
>   Stats: 6 lines in 1 file changed: 1 ins; 2 del; 3 mod
>   Patch: https://git.openjdk.java.net/jfx/pull/16.diff
>   Fetch: git fetch https://git.openjdk.java.net/jfx pull/16/head:pull/16

@kevinrushforth I believe you should be able to move forward with the review of 
this PR now as I've filed a bug report for the problem and signed the OCA as 
required. (As I did not know I needed to have my github account associated with 
the OCA, I did not include it in my details. I've asked Dalibor Topic from the 
Java SE PM team to help me have that fixed or get in touch with you to confirm 
my signing of the OCA.)

PR: https://git.openjdk.java.net/jfx/pull/16

Reply via email to