RFR: 8290844: Add Skin.install() method [v2]

Andy Goryachev angorya at openjdk.org
Thu Aug 11 19:35:04 UTC 2022


On Thu, 11 Aug 2022 13:17:43 GMT, Jeanette Winzenburg <fastegal at openjdk.org> wrote:

> hmm ... probably I'm missing something but I don't see how that would be the case for the subset of core skins that have to be refactored and moved their installation into the install() method.

You are right - some of the core skins will have to be refactored.  As a result, any skin that user derived from those _might need_ to change as well.  I shall update the CSR.

This is a price to pay for fixing a design error, I think.  Either fix it now, or forever stay with memory leaks and resulting bugs.

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

PR: https://git.openjdk.org/jfx/pull/845


More information about the openjfx-dev mailing list