RFR: 8347392: Thread-unsafe implementation of c.s.j.scene.control.skin.Utils [v2]

[Kevin Rushforth]

On Fri, 7 Feb 2025 17:55:28 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

> Hmmm, to further make a point: options 2 and 3 may not be re-entrant, so they probably won't work.

Your solution is basically what I meant by option 2. The way you did it, always using the return value of a method that will return a "safe" instance to use, is reentrant. You are right that a simple approach to option 3 would not be reentrant.

I like the solution you chose.

-------------

PR Comment: https://git.openjdk.org/jfx/pull/1691#issuecomment-2643642428